search for: class_id_any

...> > A lot of entries call add_wildcard() here, should we? Sorry for the > > basic questions, I haven't played in this code. Thanks, > > > It's really good question. My understanding is we won't have a module > that can deal with all kinds of classes like CLASS_ID_ANY. So there's > probably no need for the wildcard. The comment for add_wildcard() indicates future extension, so it's hard to know what we might need in the future until we do need it. The majority of modules.alias entries on my laptop (even if I exclude pci aliases) end with a wildcard...

On Wed, 23 Oct 2019 21:07:48 +0800 Jason Wang <jasowang at redhat.com> wrote: > Add support to parse mdev class id table. > > Reviewed-by: Parav Pandit <parav at mellanox.com> > Signed-off-by: Jason Wang <jasowang at redhat.com> > --- > drivers/vfio/mdev/vfio_mdev.c | 2 ++ > scripts/mod/devicetable-offsets.c | 3 +++ > scripts/mod/file2alias.c

On Wed, 23 Oct 2019 21:07:48 +0800 Jason Wang <jasowang at redhat.com> wrote: > Add support to parse mdev class id table. > > Reviewed-by: Parav Pandit <parav at mellanox.com> > Signed-off-by: Jason Wang <jasowang at redhat.com> > --- > drivers/vfio/mdev/vfio_mdev.c | 2 ++ > scripts/mod/devicetable-offsets.c | 3 +++ > scripts/mod/file2alias.c

...ias, "mdev:c%02X", id); > A lot of entries call add_wildcard() here, should we? Sorry for the > basic questions, I haven't played in this code. Thanks, It's really good question. My understanding is we won't have a module that can deal with all kinds of classes like CLASS_ID_ANY. So there's probably no need for the wildcard. Thanks > > Alex > >> + return 1; >> +} >> + >> /* Does namelen bytes of name exactly match the symbol? */ >> static bool sym_is(const char *name, unsigned namelen, const char *symbol) >> { >...