[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20191219143422.GJ4113@kitsune.suse.cz>
Date: Thu, 19 Dec 2019 15:34:22 +0100
From: Michal Suchánek <msuchanek@...e.de>
To: "Martin K. Petersen" <martin.petersen@...cle.com>
Cc: linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
"James E.J. Bottomley" <jejb@...ux.ibm.com>,
Jens Axboe <axboe@...nel.dk>
Subject: Re: [PATCH] scsi: blacklist: add VMware ESXi cdrom - broken tray
emulation
On Tue, Dec 17, 2019 at 05:28:28PM -0500, Martin K. Petersen wrote:
>
> Michal,
>
> > + {"VMware", "VMware", NULL, BLIST_NO_MATCH_VENDOR | BLIST_NO_TRAY},
>
> 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.
Thanks
Michal
Powered by blists - more mailing lists