Messages in this thread | | | Date | Tue, 27 Oct 2020 07:52:17 +0000 | From | Christoph Hellwig <> | Subject | Re: [PATCH v2] PCI: check also dynamic IDs for duplicate in new_id_store() |
| |
On Mon, Oct 26, 2020 at 11:57:10AM +0800, Zhenzhong Duan wrote: > When a device ID data is writen to /sys/bus/pci/drivers/.../new_id, > only static ID table is checked for duplicate and multiple dynamic ID > entries of same kind are allowed to exist in a dynamic linked list. > > Fix it by calling pci_match_device() which checks both dynamic and static > IDs. > > After fix, it shows below result which is expected. > > echo "1af4:1000" > /sys/bus/pci/drivers/vfio-pci/new_id > echo "1af4:1000" > /sys/bus/pci/drivers/vfio-pci/new_id > -bash: echo: write error: File exists > > Drop the static specifier and add a prototype to avoid build error. > > Signed-off-by: Zhenzhong Duan <zhenzhong.duan@gmail.com>
Looks good,
Reviewed-by: Christoph Hellwig <hch@lst.de>
| |