Date: Wed, 12 Jun 2013 14:26:29 -0400 From: Glen Barber <gjb@FreeBSD.org> To: Alexander Motin <mav@FreeBSD.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r251654 - in head/sys: cam cam/scsi geom Message-ID: <20130612182629.GK1558@glenbarber.us> In-Reply-To: <201306121336.r5CDaKUF007665@svn.freebsd.org> References: <201306121336.r5CDaKUF007665@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--N/GrjenRD+RJfyz+ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jun 12, 2013 at 01:36:20PM +0000, Alexander Motin wrote: > Author: mav > Date: Wed Jun 12 13:36:20 2013 > New Revision: 251654 > URL: http://svnweb.freebsd.org/changeset/base/251654 >=20 > Log: > Make CAM return and GEOM DISK pass through new GEOM::lunid attribute. > =20 > SPC-4 specification states that serial number may be property of device, > but not a specific logical unit. People reported about FC storages usi= ng > serial number in that way, making it unusable for purposes of LUN multi= path > detection. SPC-4 states that designators associated with logical unit = =66rom > the VPD page 83h "Device Identification" should be used for that purpos= e. > Report first of them in the new attribute in such preference order: NAA, > EUI-64, T10 and SCSI name string. > =20 > While there, make GEOM DISK properly report GEOM::ident in XML output a= lso > using d_getattr() method, if available. This fixes serial numbers repo= rting > for SCSI disks in `geom disk list` output and confxml. > =20 I think this change breaks the build. cc -O -pipe -DRESCUE -std=3Dgnu99 -Qunused-arguments -Wsystem-headers -We= rror -Wall -Wno-format- y2k -W -Wno-unused-parameter -Wstrict-prototypes -Wmissing-prototypes -Wpoi= nter-arith -Wno-unini tialized -Wno-pointer-sign -Wno-empty-body -Wno-string-plus-int -Wno-tautol= ogical-compare -Wno-u nused-value -Wno-parentheses-equality -Wno-unused-function -Wno-conversion = -c /src/sbin/camcontrol/camcontrol.c /src/sbin/camcontrol/camcontrol.c:6955:13: error: incompatible pointer types assigning to 'uint8_t *' (aka 'unsigned char *') from 'struct scsi_vp= d_id_descriptor *' [-Werror,-Wincompatible-pointer-types] item_addr =3D scsi_get_devid(item->device_id, ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 error generated. *** Error code 1 Stop. make: stopped in /src/sbin/camcontrol *** Error code 1 Glen --N/GrjenRD+RJfyz+ Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (FreeBSD) iQEcBAEBCAAGBQJRuL1VAAoJEFJPDDeguUajSKkIALnbFD7mMVZMX93PK8D8B9Mz 3EsYfL4uPptiNpF/QnMUIIAUrB8j+gIAfAEA075O4ml5/kLfDTHBxiDNppIArvOu E9XX6llJAklFgFAyFjSLPb+PQhjzyEjZLpmRtcOTR8+c9XrlmfhwqjaVIrGVFKlX ytez3g7PhF4JtZPFayYyMUIzfDeuSOGQl0+9Pzi/qS0nQoJuYu4GyQ9T0tQLjnO+ UoTSFw40aB8WDytFgHKIQgR4TK+AJG63YRthDes2hsTSgOgj2hXcr7dXkglQVytu 98Sw04uYnQDmFQnpNPT9xedBelaih3+mA/hkm4jGQTJG6Yvz39JpKV+wcRS88cg= =/lxB -----END PGP SIGNATURE----- --N/GrjenRD+RJfyz+--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130612182629.GK1558>