d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

When editing a mode page on a tape drive, do not clear the device

Browse Source 
specific parameter.

Tape drives include write protect (WP), Buffered Mode and Speed
settings in the device-specific parameter.  Clearing this
parameter on a mode select can have the effect of turning off
write protect or buffered mode, or changing the speed setting of
the tape drive.

Disks report DPO/FUA support via the device specific parameter
for MODE SENSE, but the bit is reserved for MODE SELECT.  So we
clear this for disks (and other non-tape devices) to avoid
potential errors from the target device.

sbin/camcontrol/modeedit.c:
	Clear the device-specific parameter in the mode page
	header if we're not operating on a tape drive.

MFC after:	3 days
Sponsored by:	Spectra Logic
This commit is contained in:
Kenneth D. Merry 2017-05-05 21:29:28 +00:00
parent 6790499792
commit 1e773aeb99
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=317854
1 changed files with 14 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
sbin/camcontrol/modeedit.c
View File

@ -629,8 +629,21 @@ editlist_save(struct cam_device *device, int dbd, int pc, int page,
/* Recalculate headers & offsets. */
mh->data_length = 0; /* Reserved for MODE SELECT command. */
mh->dev_spec = 0; /* Clear device-specific parameters. */
mh->blk_desc_len = 0; /* No block descriptors. */
/*
* Tape drives include write protect (WP), Buffered Mode and Speed
* settings in the device-specific parameter. Clearing this
* parameter on a mode select can have the effect of turning off
* write protect or buffered mode, or changing the speed setting of
* the tape drive.
*
* Disks report DPO/FUA support via the device specific parameter
* for MODE SENSE, but the bit is reserved for MODE SELECT. So we
* clear this for disks (and other non-tape devices) to avoid
* potential errors from the target device.
*/
if (device->pd_type != T_SEQUENTIAL)
mh->dev_spec = 0;
mph = MODE_PAGE_HEADER(mh);
mph->page_code &= ~SMPH_PS; /* Reserved for MODE SELECT command. */