Subject: Re: panic: dequeued wrong buf in -current
To: Manuel Bouyer <bouyer@antioche.eu.org>
From: Jason Thorpe <thorpej@wasabisystems.com>
List: tech-kern
Date: 09/08/2004 08:35:13
--Apple-Mail-3-386396975
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset=US-ASCII; format=flowed
On Sep 7, 2004, at 2:12 PM, Manuel Bouyer wrote:
> I noone object I'll commit this in a couple of days, and request a
> pullup to
> 2.0 of all commits related to dealing with scsipi_xfer shortage.
int
-atapi_scsipi_cmd(struct scsipi_periph *periph,
+atapi_scsipi_cmd(struct scsipi_periph *periph, struct scsipi_xfer *_xs,
struct scsipi_generic *scsipi_cmd, int cmdlen, void *data, size_t
datalen,
int retries, int timeout, struct buf *bp, int flags)
{
- struct scsipi_xfer *xs;
+ struct scsipi_xfer *xs = _xs;
What's the point of that? Why not just name the parameter "xs" and
eliminate the automatic variable?
-- Jason R. Thorpe <thorpej@wasabisystems.com>
--Apple-Mail-3-386396975
content-type: application/pgp-signature; x-mac-type=70674453;
name=PGP.sig
content-description: This is a digitally signed message part
content-disposition: inline; filename=PGP.sig
content-transfer-encoding: 7bit
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (Darwin)
iD8DBQFBPyaxOpVKkaBm8XkRAq4hAKDC8I/odDC3BRMPtDxlAWy9QsGljwCbBJoh
ttN1d5BTCux3Jd9mluoKZC4=
=kIJL
-----END PGP SIGNATURE-----
--Apple-Mail-3-386396975--