From owner-cvs-src@FreeBSD.ORG Tue Jan 31 00:33:53 2006 Return-Path: X-Original-To: cvs-src@freebsd.org Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 234D116A420; Tue, 31 Jan 2006 00:33:53 +0000 (GMT) (envelope-from flz@xbsd.org) Received: from smtp.xbsd.org (xbsd.org [82.233.2.192]) by mx1.FreeBSD.org (Postfix) with ESMTP id 70D3543D45; Tue, 31 Jan 2006 00:33:52 +0000 (GMT) (envelope-from flz@xbsd.org) Received: from localhost (localhost.xbsd.org [127.0.0.1]) by smtp.xbsd.org (Postfix) with ESMTP id 437621143B; Tue, 31 Jan 2006 01:33:51 +0100 (CET) Received: from smtp.xbsd.org ([127.0.0.1]) by localhost (srv1.xbsd.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 39253-03; Tue, 31 Jan 2006 01:33:40 +0100 (CET) Received: from cream.xbsd.org (cream.xbsd.org [192.168.42.6]) by smtp.xbsd.org (Postfix) with ESMTP id 8A37D1172F; Tue, 31 Jan 2006 01:33:39 +0100 (CET) From: Florent Thoumie To: Nate Lawson Date: Tue, 31 Jan 2006 01:33:27 +0100 User-Agent: KMail/1.8.2 References: <20060130202806.DCC7916A4CA@hub.freebsd.org> <200601310108.27007.flz@xbsd.org> <43DEAE5A.4010904@root.org> In-Reply-To: <43DEAE5A.4010904@root.org> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart6122666.2qVWMitRIr"; protocol="application/pgp-signature"; micalg=pgp-sha1 Content-Transfer-Encoding: 7bit Message-Id: <200601310133.34152.flz@xbsd.org> X-Virus-Scanned: amavisd-new at xbsd.org Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/cam/scsi scsi_da.c src/sys/dev/usb umass.c usbdevs X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 31 Jan 2006 00:33:53 -0000 --nextPart6122666.2qVWMitRIr Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline On Tuesday 31 January 2006 01:24, Nate Lawson wrote: > Florent Thoumie wrote: > > On Tuesday 31 January 2006 00:48, Nate Lawson wrote: > >>Florent Thoumie wrote: > >>>flz 2006-01-30 20:27:44 UTC > >>> > >>> FreeBSD src repository (ports committer) > >>> > >>> Modified files: > >>> sys/cam/scsi scsi_da.c > >>> sys/dev/usb umass.c usbdevs > >>> Log: > >>> - Add a scsi_da.c and a umass.c quirk for Genesys 6-in-1 Card Reader. > >>> > >>>--- src/sys/cam/scsi/scsi_da.c:1.185 Thu Jan 26 00:35:53 2006 > >>>+++ src/sys/cam/scsi/scsi_da.c Mon Jan 30 20:27:44 2006 > >>>@@ -427,6 +427,14 @@ > >>> {T_DIRECT, SIP_MEDIA_REMOVABLE, "*" , "USB DISK*", > >>> "*"}, /*quirks*/ DA_Q_NO_SYNC_CACHE > >>> }, > >>>+ { > >>>+ /* > >>>+ * Genesys 6-in-1 Card Reader > >>>+ * No PR, reported by anders > >>>+ */ > >>>+ {T_DIRECT, SIP_MEDIA_REMOVABLE, "Generic*", "STORAGE DEVICE*", > >>>+ "*"}, /*quirks*/ DA_Q_NO_SYNC_CACHE > >>>+ }, > >>> }; > >> > >>I think it's bad to deviate from the policy we've established for > >>handling quirks. The reason why we need a PR is to track the following > >>info: > >> > >># Output of "camcontrol inquiry yourdevice" > >># Manufacturer name, model number, etc. > >># Transport type (FC, SCSI, USB, Firewire) > >># Output from dmesg for failed attach attempts > >># Output from dmesg for successful attach attempts (after quirk added) > >># Output of "usbdevs -v" with device attached > >># Valid email address > >> > >>An email address or posting is not enough since the user may discard the > >>device later and it's tough to tell from the quirk itself exactly what > >>device was causing the problem. Case in point: the quirk "Generic > >>STORAGE DEVICE" you've added could match any number of devices. Was > >>this only a particular Genesys product or do all their products have > >>this issue? Without a way to track this info, it becomes impossible to > >>decide. PRs are the best current way to track things. > >> > >>http://root.org/~nate/freebsd/scsi/quirks.html > >> > >>Please stick to the policy of collecting the necessary information to > >>help future maintainers. > > > > Hum sorry, I thought I had enough information from anders' mail [1] > > The email is definitely better than the comment in the code. > > > I've used "*" as revision match since that's what is used for all other > > entries, whether particular revisions are mentioned in the comment or > > not. > > I'm not concerned about the revision. I'm concerned about the vendor > (Generic*) and device name (STORAGE DEVICE*). Why are the *'s needed? Seemed common practice reading the other entries. > (Again, a PR would help track this kind of conversation as shown in > previous PRs about quirks. Submitters often match way too much.) > > > Do you want me to create a PR just for tracking purposes? > > [1] http://docs.freebsd.org/cgi/mid.cgi?20060116193024.GA95183 > > That would be nice, especially since some of the requested info is > missing (dmesg, usbdevs -v). However, if you cited a email in the > commit msg (maybe SMTP Message-ID) such that we could find it in the > future, that would probably be enough. I'm not trying to create a > bureaucracy, just make sure we don't lose information like we used to on > why a quirk was added in the first place. I only mentioned the freebsd-usb mailing list. I'll contact Anders to get=20 additional details and I (or he) will fill a PR so that we can add it to th= e=20 comment. It seems a lot of devices are concerned by the sync cache problem, would it= be=20 harmful to just remove this part of the code or could there be a way to=20 detect if the device supports it or not? =2D-=20 =46lorent Thoumie flz@FreeBSD.org =46reeBSD Committer --nextPart6122666.2qVWMitRIr Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2 (FreeBSD) iD8DBQBD3rBeMxEkbVFH3PQRAn7lAJwMjgFcxny+rw8OfH+aUBl20SKjAgCeOF4B 3+5SQLEFyg5pp8Q4xzgQaVo= =JdFv -----END PGP SIGNATURE----- --nextPart6122666.2qVWMitRIr--