On Thu, Dec 19, 2019 at 06:31:12PM -0500, Martin K. Petersen wrote: > > Michal, > > >> Please don't introduce a blist flag to work around deficiencies in the > >> matching interface. I suggest you tweak the matching functions so they > >> handle a NULL vendor string correctly. > > > > I don't think that will work with the interface for dynamically adding > > entries through sysfs. > > Please make it work :) > > There's nothing conceptually wrong with being able to do: > > echo ":Model:Flags" > /proc/scsi/device_info > > We keep running into issues where the same device needs to be listed > many times because it gets branded by different vendors. > Is there any description of what the format is supposed to be? >From the current code it looks like comma separated list of blacklist entries that may be optionally quoted in some way. The quoting is basically ignored but it is not clear if the inidividual entries are supposed to be quoted or the whole thing. Thanks Michal
