21/07/2020 14:13, Parav Pandit: > From: David Marchand <david.march...@redhat.com> > > On Tue, Jul 21, 2020 at 1:19 PM Parav Pandit <pa...@mellanox.com> wrote: > > > > This way, net/mlx5 and vdpa/mlx5 will pass their id_map to the mlx5 > > > > pci driver whether it is registered to the pci bus or not yet. > > > > > > And here pci_id_table will not be const *. > > > Is that ok? > > > > This is already the case in the current patch. > > I see nothing wrong with it. > > The pci code expects this pointer to be const and will it treat it as such. > > > o.k. > > Gaetan, Ferruh, Thomas, > Can you please ack as well?
Yes of course it is OK updating the PCI table of the common layer in runtime. The most important is to keep the fixed PCI table of the PMDs the same as registered for pmdinfo usage. > > On the other hand, you may remove the pci table exported by the common > > driver as it is useless. > > pmdinfogen only parses the .o and extracts the table symbol, so it won't > > load > > other mlx drivers to generate the .pmd.c file. > > Ok. One more nit: please don't use "class" as variable name for drivers.
