[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH 09/13] u2f-passthru: put it into the 'usb' category
From: |
ganqixin |
Subject: |
RE: [PATCH 09/13] u2f-passthru: put it into the 'usb' category |
Date: |
Tue, 17 Nov 2020 13:40:23 +0000 |
> -----Original Message-----
> From: Gerd Hoffmann [mailto:kraxel@redhat.com]
> Sent: Tuesday, November 17, 2020 2:59 PM
> To: Philippe Mathieu-Daudé <philmd@redhat.com>
> Cc: ganqixin <ganqixin@huawei.com>; qemu-devel@nongnu.org;
> qemu-trivial@nongnu.org; Marc-André Lureau
> <marcandre.lureau@redhat.com>; thuth@redhat.com; Chenqun (kuhn)
> <kuhn.chenqun@huawei.com>; Zhanghailiang
> <zhang.zhanghailiang@huawei.com>; Eduardo Habkost
> <ehabkost@redhat.com>
> Subject: Re: [PATCH 09/13] u2f-passthru: put it into the 'usb' category
>
> On Mon, Nov 16, 2020 at 03:04:52PM +0100, Philippe Mathieu-Daudé
> wrote:
> > On 11/15/20 7:48 PM, Gan Qixin wrote:
> > > The category of the u2f-passthru device is not set, put it into the 'usb'
> > > category.
> >
> > I guess we discussed this with Thomas 1 or 2 years ago but I don't
> > remember. I think it was about using set_bits() so devices can appear
> > in multiple categories.
> >
> > Gerd, do you know what is the point of the "usb" category for
> > management apps? This is a bus accepting multiple better categorized
> > devices (display, storage, input, network, sound).
>
> Right now we have the host adapters collected in the usb category.
> Maybe we should rename the category to make that clear.
>
> And, yes, tagging the u2f devices as "usb" doesn't look useful.
>
> > Unrelated but multiple devices are related to SECURITY.
> > Maybe it is time to introduce the DEVICE_CATEGORY_SECURITY?
>
> Either that, or place it in the "misc" category.
Thanks for your advice, I will put it into 'misc' category.
Gan Qixin
- [PATCH 05/13] mc146818rtc: put it into the 'misc' category, (continued)
- [PATCH 05/13] mc146818rtc: put it into the 'misc' category, Gan Qixin, 2020/11/16
- [PATCH 03/13] vmmouse: put it into the 'input' category, Gan Qixin, 2020/11/16
- [PATCH 02/13] virtio-pmem: put it into the 'storage' category, Gan Qixin, 2020/11/16
- [PATCH 04/13] nvdimm: put it into the 'storage' category, Gan Qixin, 2020/11/16
- [PATCH 07/13] tpm: put some tpm devices into the correct category, Gan Qixin, 2020/11/16
- [PATCH 09/13] u2f-passthru: put it into the 'usb' category, Gan Qixin, 2020/11/16
- [PATCH 08/13] AMDVI-PCI: put it into the 'misc' category, Gan Qixin, 2020/11/16
- [PATCH 11/13] spitz: put some Spitz-family devices into the correct category, Gan Qixin, 2020/11/16
- [PATCH 10/13] tosa-ssp: put it into the 'misc' category, Gan Qixin, 2020/11/16
- [PATCH 06/13] ipmi: put some ipmi devices into the correct category, Gan Qixin, 2020/11/16
- [PATCH 12/13] SPI flash devices: put them into the 'storage' category, Gan Qixin, 2020/11/16
- [PATCH 13/13] bcm2835_cprman: put some peripherals of bcm2835 cprman into the 'misc' category, Gan Qixin, 2020/11/16