From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Garzik Subject: Re: [PATCH 2.6.20-rc3] fix broken retval test in sr_block_ioctl Date: Tue, 02 Jan 2007 17:32:16 -0500 Message-ID: <459ADD70.9030402@pobox.com> References: <200701021724.39613.liml@rtr.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <200701021724.39613.liml@rtr.ca> Sender: linux-scsi-owner@vger.kernel.org To: Mark Lord Cc: Tejun Heo , IDE/ATA development list , linux-scsi@vger.kernel.org List-Id: linux-ide@vger.kernel.org Mark Lord wrote: > With recent kernels (and possibly much older ones, too), > ioctl() calls for ATAPI devices never make it to the driver layers. > > The reason for this is a borked return code test in drivers/scsi/sr.c. > This patch fixes it. > > Signed-off-by: Mark Lord > > --- > --- old/drivers/scsi/sr.c 2006-11-29 16:57:37.000000000 -0500 > +++ linux/drivers/scsi/sr.c 2007-01-02 16:40:33.000000000 -0500 > @@ -468,7 +468,7 @@ > } > > ret = cdrom_ioctl(file, &cd->cdi, inode, cmd, arg); > - if (ret != ENOSYS) > + if (ret != -ENOSYS) I think Tejun posted the same patch earlier today. ACK to either patch, of course. Jeff