Re: [PATCH v3] usb: core: Add "quirks" parameter for usbcore

From: Matthew Wilcox
Date: Sun Feb 25 2018 - 10:18:38 EST


On Sun, Feb 25, 2018 at 08:38:33PM +0800, Kai-Heng Feng wrote:
> v2: Use in-kernel tolower() function.

... why are you using tolower at all?

You've got 13 quirks already; you may need to use upper case as well
before too long.

> + quirk = vmalloc(sizeof(struct quirk_entry));

vmalloc? For a struct that's 24 bytes? You know that allocates an
entire 4k page, right?