Feature #2665

sd.conf should be able to override physical-block-size

Added by George Wilson almost 2 years ago. Updated 11 months ago.

Status:Resolved Start date:2012-05-01
Priority:Normal Due date:
Assignee:George Wilson % Done:

0%
Category:driver - device drivers Spent time: -
Target version:-
Difficulty:Medium Tags:needs-triage

Description

PSARC 2008/769 advertised the ability to override the physical-block-size by using the sd-config-list. Unfortunately this portion of the project was not implemented. This feature proposes to implement this missing functionality. This would allow administrators to update sd.conf by adding a simple VID/PID tuple in the sd-config-list. An example would look like this:

sd.conf:

sd-config-list = "DGC     RAID", "physical-block-size:4096";

The format of the VID/PID tuple uses the following format:

"012345670123456789012345" 
"|-VID--||-----PID------|"

This example overrides the physical-block-size for all DGC RAID devices to use 4K. ZFS would be able to consume this value to determine the proper ashift value for its vdevs.

Related issues

related to illumos gate - Bug #3220: noncompliant devices break sd-config-list New 2012-09-24
related to illumos gate - Bug #453: ZPOOL_CONFIG_ASHIFT not set correctly for drives with lar... New 2010-12-06

History

#1
Updated by Eric Schrock almost 2 years ago

  • Status changed from New to Resolved

changeset: 13680:2bd022a765e2
tag: tip
user: George Wilson <>
date: Thu May 03 05:49:19 2012 -0700

description:
2665 sd.conf should be able to override physical-block-size
2671 zpool import should not fail if vdev ashift has increased
Reviewed by: Adam Leventhal <>
Reviewed by: Eric Schrock <>
Reviewed by: Richard Elling <>
Reviewed by: Gordon Ross <>
Reviewed by: Garrett D'Amore <>
Approved by: Richard Lowe <>

modified:
usr/src/cmd/zpool/zpool_main.c
usr/src/uts/common/fs/zfs/vdev.c
usr/src/uts/common/io/scsi/targets/sd.c

#2
Updated by Albert Lee about 1 year ago

For future reference, this command can be used to retrieve the vendor and product ID strings for all sd devices:

echo "::walk sd_state | ::grep '.!=0' | ::print struct sd_lun un_sd | ::print struct \\
 scsi_device sd_inq | ::print struct scsi_inquiry inq_vid inq_pid" | mdb -k

#3
Updated by Albert Lee about 1 year ago

Remember, you need to pad the vendor, product string to the length shown with spaces.

Padding is actually not necessary. Extra spaces (not tabs!) are ignored.

Asterisks have special meaning if they appear at the beginning and end of the ID string only. sd searches the product ID (not the vendor ID) for the substring enclosed by asterisks.
Examples:
  • "*ST9320*" will match any vendor ID and product ID "ST9320423AS".
  • "**" will match any vendor ID and product ID.

#4
Updated by Jens Elkner 11 months ago

Albert Lee wrote:

... command can be used to retrieve the vendor and product ID strings for all sd devices ...

http://iws.cs.uni-magdeburg.de/~elkner/s11/etc/sdprops.sh may also help to check/lookup current settings.

NOTE: As usual the script requires {man,log}.kshlib from http://iws.cs.uni-magdeburg.de/~elkner/s11/etc/ to be in the same dir or anywhere else in the $PATH

Also available in: Atom PDF