From owner-svn-src-head@FreeBSD.ORG Wed Jun 12 18:26:36 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 820ECB73; Wed, 12 Jun 2013 18:26:36 +0000 (UTC) (envelope-from gjb@FreeBSD.org) Received: from onyx.glenbarber.us (onyx.glenbarber.us [IPv6:2607:fc50:1000:c200::face]) by mx1.freebsd.org (Postfix) with ESMTP id 58F501944; Wed, 12 Jun 2013 18:26:36 +0000 (UTC) Received: from glenbarber.us (70.15.88.86.res-cmts.sewb.ptd.net [70.15.88.86]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) (Authenticated sender: gjb) by onyx.glenbarber.us (Postfix) with ESMTPSA id 8ED6F23F848; Wed, 12 Jun 2013 14:26:31 -0400 (EDT) DKIM-Filter: OpenDKIM Filter v2.8.3 onyx.glenbarber.us 8ED6F23F848 Authentication-Results: onyx.glenbarber.us; dkim=none reason="no signature"; dkim-adsp=none Date: Wed, 12 Jun 2013 14:26:29 -0400 From: Glen Barber To: Alexander Motin Subject: Re: svn commit: r251654 - in head/sys: cam cam/scsi geom Message-ID: <20130612182629.GK1558@glenbarber.us> References: <201306121336.r5CDaKUF007665@svn.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="N/GrjenRD+RJfyz+" Content-Disposition: inline In-Reply-To: <201306121336.r5CDaKUF007665@svn.freebsd.org> X-Operating-System: FreeBSD 10.0-CURRENT amd64 User-Agent: Mutt/1.5.21 (2010-09-15) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 12 Jun 2013 18:26:36 -0000 --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+--