Subject: PR/34202 CVS commit: [netbsd-4] src/sys/dev
To: None <martin@NetBSD.org, gnats-admin@netbsd.org,>
From: Pavel Cahyna <pavel@netbsd.org>
List: netbsd-bugs
Date: 01/15/2007 22:20:02
The following reply was made to PR kern/34202; it has been noted by GNATS.
From: Pavel Cahyna <pavel@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc:
Subject: PR/34202 CVS commit: [netbsd-4] src/sys/dev
Date: Mon, 15 Jan 2007 22:15:14 +0000 (UTC)
Module Name: src
Committed By: pavel
Date: Mon Jan 15 22:15:14 UTC 2007
Modified Files:
src/sys/dev/ic [netbsd-4]: wdc.c
src/sys/dev/scsipi [netbsd-4]: cd.c
Log Message:
Pull up following revision(s) (requested by martin in ticket #371):
sys/dev/scsipi/cd.c: revision 1.261
sys/dev/ic/wdc.c: revision 1.244
If __NO_STRICT_ALIGNMENT is not defined, take special care to do PIO
transfers with proper alignement - this removes alignement restrictions
of the buffer pointer passed.
Thanks to bouyer, reinoud, jdc for helping with the code, and special
thanks to Dave Huang for testing.
The previous fix for PR 34202 was wrong (some sizeof used the union, not
the command block) - but instead of fixing this, we can now revert it
completley: the real fix has been applied to ../ic/wdc.c in rev. 1.244,
removing the alignement constraints.
To generate a diff of this commit:
cvs rdiff -r1.243 -r1.243.2.1 src/sys/dev/ic/wdc.c
cvs rdiff -r1.260 -r1.260.2.1 src/sys/dev/scsipi/cd.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.