kern/152768: [mfi] Weird check in mfi(4)
John Baldwin
jhb at freebsd.org
Mon Dec 6 17:00:23 UTC 2010
The following reply was made to PR kern/152768; it has been noted by GNATS.
From: John Baldwin <jhb at freebsd.org>
To: Scott Long <scottl at samsco.org>
Cc: bug-followup at freebsd.org,
saw at online.de
Subject: Re: kern/152768: [mfi] Weird check in mfi(4)
Date: Mon, 6 Dec 2010 11:59:38 -0500
On Monday, December 06, 2010 11:45:32 am Scott Long wrote:
> John,
>
> Had a chance to review this, and the patch looks reasonable. This code was
basically a copy-and-paste from the aac driver, which will also need a similar
change.
>
> Scott
Ok, does this look right for aac?
Index: aac_cam.c
===================================================================
--- aac_cam.c (revision 216122)
+++ aac_cam.c (working copy)
@@ -587,7 +587,8 @@
(device == T_PROCESSOR) ||
(sc->flags & AAC_FLAGS_CAM_PASSONLY))
ccb->csio.data_ptr[0] =
- ((device & 0xe0) | T_NODEVICE);
+ ((ccb->csio.data_ptr[0] & 0xe0) |
+ T_NODEVICE);
} else if (ccb->ccb_h.status == CAM_SEL_TIMEOUT &&
ccb->ccb_h.target_lun != 0) {
/* fix for INQUIRYs on Lun>0 */
--
John Baldwin
More information about the freebsd-bugs
mailing list