RE: Write-protect checks

RE: Write-protect checks

[Salyzyn, Mark]

I have an interest in this being checked for fixed media as well, we
want to expose the underlying physical array components, but read &
write protected to protect the array contents. Christoph asked for us to
use the scsi layer for this check rather than spoofing it in the driver.

The other alternative is to add yet another device flag to enforce this.

Sincerely -- Mark Salyzyn

> -----Original Message-----
> From: linux-scsi-owner@vger.kernel.org 
> [mailto:linux-scsi-owner@vger.kernel.org] On Behalf Of Alan Stern
> Sent: Thursday, December 08, 2005 1:22 PM
> To: SCSI development list; Alan Cox
> Cc: Dmitry Antipov
> Subject: Write-protect checks
> 
> 
> On Thu, 8 Dec 2005, Alan Cox wrote:
> 
> > On Iau, 2005-12-08 at 11:00 -0500, Alan Stern wrote:
> > > module parameter.  For some reason (don't ask me why -- I 
> don't know) the
> > > SCSI core doesn't check a disk device's write-protect 
> status unless the
> > > device is marked as removable.
> > 
> > Have you asked the scsi maintainers why this occurs. Having 
> had a quick
> > look over the specs I have here I can't see why this is done either.
> 
> No, I haven't.
> 
> Okay guys.  Does anyone know why sd.c calls 
> sd_read_write_protect_flag 
> only for devices with removable media?
> 
> Alan Stern
> 
> -
> To unsubscribe from this list: send the line "unsubscribe 
> linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

Re: Write-protect checks

[Douglas Gilbert]

Salyzyn, Mark wrote:
> I have an interest in this being checked for fixed media as well, we
> want to expose the underlying physical array components, but read &
> write protected to protect the array contents.

Mark,
How does a device indicate that it is read protected?
[apart from bouncing READ commands with data protect]
SBC-3 has no similar mechanism to the WP bit in the
device specific parameter field returned by MODE SENSE.

IMO sd should check and respect the WP bit for all
devices (whether removable or not). If the sd driver
finds out that a device is read and write protected,
perhaps it should detach itself.

Doug Gilbert

 Christoph asked for us to
> use the scsi layer for this check rather than spoofing it in the driver.
> 
> The other alternative is to add yet another device flag to enforce this.
> 
> Sincerely -- Mark Salyzyn
> 
> 
>>-----Original Message-----
>>From: linux-scsi-owner@vger.kernel.org 
>>[mailto:linux-scsi-owner@vger.kernel.org] On Behalf Of Alan Stern
>>Sent: Thursday, December 08, 2005 1:22 PM
>>To: SCSI development list; Alan Cox
>>Cc: Dmitry Antipov
>>Subject: Write-protect checks
>>
>>
>>On Thu, 8 Dec 2005, Alan Cox wrote:
>>
>>
>>>On Iau, 2005-12-08 at 11:00 -0500, Alan Stern wrote:
>>>
>>>>module parameter.  For some reason (don't ask me why -- I 
>>
>>don't know) the
>>
>>>>SCSI core doesn't check a disk device's write-protect 
>>
>>status unless the
>>
>>>>device is marked as removable.
>>>
>>>Have you asked the scsi maintainers why this occurs. Having 
>>
>>had a quick
>>
>>>look over the specs I have here I can't see why this is done either.
>>
>>No, I haven't.
>>
>>Okay guys.  Does anyone know why sd.c calls 
>>sd_read_write_protect_flag 
>>only for devices with removable media?
>>
>>Alan Stern
>>
>>-
>>To unsubscribe from this list: send the line "unsubscribe 
>>linux-scsi" in
>>the body of a message to majordomo@vger.kernel.org
>>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

[PATCH] Always do write-protect check

[Alan Stern]

James:

Since nobody has offered an explanation for why the sd driver makes a 
write-protect check only for devices with removable media, I'm submitting 
this patch (as620) to get rid of the removable-media test.

Alan Stern



Signed-off-by: Alan Stern <stern@rowland.harvard.edu>

---

Index: usb-2.6/drivers/scsi/sd.c
===================================================================
--- usb-2.6.orig/drivers/scsi/sd.c
+++ usb-2.6/drivers/scsi/sd.c
@@ -1509,9 +1509,7 @@ static int sd_revalidate_disk(struct gen
 	 */
 	if (sdkp->media_present) {
 		sd_read_capacity(sdkp, disk->disk_name, buffer);
-		if (sdp->removable)
-			sd_read_write_protect_flag(sdkp, disk->disk_name,
-						   buffer);
+		sd_read_write_protect_flag(sdkp, disk->disk_name, buffer);
 		sd_read_cache_type(sdkp, disk->disk_name, buffer);
 	}
 		

Re: [PATCH] Always do write-protect check

[James Bottomley]

On Fri, 2005-12-09 at 11:34 -0500, Alan Stern wrote:
> Since nobody has offered an explanation for why the sd driver makes a 
> write-protect check only for devices with removable media, I'm submitting 
> this patch (as620) to get rid of the removable-media test.

OK ... I'll put this in scsi-misc for post 2.6.15.  I suspect the reason
is that many devices get confused or lie about it, but there's only one
sure way to ascertain if this is true or not ...

If it turns out to be a problem, I'll back it out before 2.6.16.

James