From fukui@fanet.net  Thu Jun 17 02:35:01 2010
Return-Path: <fukui@fanet.net>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 10A1A1065674
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 17 Jun 2010 02:35:01 +0000 (UTC)
	(envelope-from fukui@fanet.net)
Received: from mail.fanet.net (home.fanet.net [202.229.191.26])
	by mx1.freebsd.org (Postfix) with ESMTP id B58218FC14
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 17 Jun 2010 02:35:00 +0000 (UTC)
Received: from home.fanet.net
	by mail.fanet.net (8.14.3+3.5Wbeta/8.14.4) with ESMTP id o5H2MVC2039397
	(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO);
	Thu, 17 Jun 2010 11:22:31 +0900 (JST)
	(envelope-from fukui@fanet.net)
Received: (from fukui@localhost)
	by home.fanet.net (8.14.3+3.5Wbeta/8.14.4/Submit) id o5H2MUfs039396;
	Thu, 17 Jun 2010 11:22:30 +0900 (JST)
	(envelope-from fukui)
Message-Id: <201006170222.o5H2MUfs039396@home.fanet.net>
Date: Thu, 17 Jun 2010 11:22:30 +0900 (JST)
From: Masayuki FUKUI <fukui.FreeBSD@fanet.net>
To: FreeBSD-gnats-submit@freebsd.org
Cc: mav@freebsd.org
Subject: hptiop(4) (HighPoint RocketRAID 3xxx/4xxx driver) broken on FreeBSD 8.1-PRERELEASE
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         147929
>Category:       kern
>Synopsis:       [hptiop] HighPoint RocketRAID 3xxx/4xxx driver broken on FreeBSD 8.1-PRERELEASE
>Confidential:   no
>Severity:       critical
>Priority:       high
>Responsible:    mav
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Jun 17 02:40:03 UTC 2010
>Closed-Date:    Mon Jun 28 16:04:04 UTC 2010
>Last-Modified:  Mon Jun 28 16:04:04 UTC 2010
>Originator:     Masayuki FUKUI
>Release:        FreeBSD 8.1-PRERELEASE amd64
>Organization:
>Environment:
System: FreeBSD master 8.1-PRERELEASE FreeBSD 8.1-PRERELEASE #0: Sat Jun  5 11:07:11 JST 2010     root@master:/usr/src/sys/amd64/compile/master  amd64
>Description:
Fails to boot FreeBSD 8.1-PRERELEASE from RAID volume under hptiop(4) (HighPoint RocketRAID 3xxx/4xxx RAID controller).
RAID controller seems to be detected normally as hptiop(4). However, RAID volume is not detected at all.

hptiop(4) had operated normally on FreeBSD 7.2-RELEASE.


dmesg:

...
<snip>
...
hptiop0: adapter at PCI 1:0:0, IRQ 16
hptiop0: <RocketRAID 3510 SATA Controller
> mem 0xfdf00000-0xfdffffff irq 16 at device 0.0 on pci1
hptiop0: 0 RocketRAID 3xxx/4xxx controller driver v1.3 (010208)
xpt_dev_async_default called
hptiop0: [GIANT-LOCKED]
hptiop0: [ITHREAD]
...
<snip>
...
SMP: AP CPU #1 Launched!
SMP: AP CPU #3 Launched!
SMP: AP CPU #2 Launched!
Trying to mount root from ufs:/dev/da0p2
ROOT MOUNT ERROR:
If you have invalid mount options, reboot, and first try the following from
the loader prompt:

     set vfs.root.mountfrom.options=rw

and then remove invalid mount options from /etc/fstab.

Loader variables:
vfs.root.mountfrom=ufs:/dev/da0p2
vfs.root.mountfrom.options=rw,noatime

Manual root filesystem specification:
  <fstype>:<device>  Mount <device> using filesystem <fstype>
...
<snip>
...
mountroot> 

>How-To-Repeat:
Try to boot FreeBSD 8.1-PRERELEASE with RAID volume under HighPoint RocketRAID 3xxx/4xxx.
>Fix:
I do not understand the cause though I am debugging.

During boot FreeBSD 7.2-RELEASE, xpt_scan_lun() (in src/sys/cam/cam_xpt.c) seems to be called immediately after hptiop(4) is detected. 
However, during boot FreeBSD 8.1-PRERELEASE, scsi_scan_lun() (in src/sys/cam/scsi/scsi_xpt.c) seems never to be called after hptiop(4) is detected.
Probing RAID volumes seems not to start at all.

I hope this problem solves by the time FreeBSD 8.1 is released.
>Release-Note:
>Audit-Trail:

From: Alexander Motin <mav@FreeBSD.org>
To: bug-followup@FreeBSD.org, fukui.FreeBSD@fanet.net
Cc:  
Subject: Re: i386/147929: hptiop(4) (HighPoint RocketRAID 3xxx/4xxx driver)
 broken on FreeBSD 8.1-PRERELEASE
Date: Sat, 19 Jun 2010 15:18:23 +0300

 This is a multi-part message in MIME format.
 --------------000202070107020705030200
 Content-Type: text/plain; charset=KOI8-R
 Content-Transfer-Encoding: 7bit
 
 Hello.
 
 Please try attached patch.
 
 -- 
 Alexander Motin
 
 --------------000202070107020705030200
 Content-Type: text/plain;
  name="hptiop.trans.patch"
 Content-Transfer-Encoding: 7bit
 Content-Disposition: inline;
  filename="hptiop.trans.patch"
 
 --- hptiop.c.prev	2010-06-18 21:39:47.000000000 +0300
 +++ hptiop.c	2010-06-19 15:15:26.000000000 +0300
 @@ -1823,6 +1823,10 @@ scsi_done:
  		strncpy(cpi->sim_vid, "FreeBSD", SIM_IDLEN);
  		strncpy(cpi->hba_vid, "HPT   ", HBA_IDLEN);
  		strncpy(cpi->dev_name, cam_sim_name(sim), DEV_IDLEN);
 +		cpi->transport = XPORT_SPI;
 +		cpi->transport_version = 2;
 +		cpi->protocol = PROTO_SCSI;
 +		cpi->protocol_version = SCSI_REV_2;
  		cpi->ccb_h.status = CAM_REQ_CMP;
  		break;
  	}
 
 --------------000202070107020705030200--

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: i386/147929: commit references a PR
Date: Sat, 19 Jun 2010 13:41:43 +0000 (UTC)

 Author: mav
 Date: Sat Jun 19 13:41:17 2010
 New Revision: 209340
 URL: http://svn.freebsd.org/changeset/base/209340
 
 Log:
   Report transport type in XPT_PATH_INQ.
   
   PR:		i386/147929
   MFC after:	3 days
 
 Modified:
   head/sys/dev/hptiop/hptiop.c
 
 Modified: head/sys/dev/hptiop/hptiop.c
 ==============================================================================
 --- head/sys/dev/hptiop/hptiop.c	Sat Jun 19 13:09:42 2010	(r209339)
 +++ head/sys/dev/hptiop/hptiop.c	Sat Jun 19 13:41:17 2010	(r209340)
 @@ -1823,6 +1823,10 @@ scsi_done:
  		strncpy(cpi->sim_vid, "FreeBSD", SIM_IDLEN);
  		strncpy(cpi->hba_vid, "HPT   ", HBA_IDLEN);
  		strncpy(cpi->dev_name, cam_sim_name(sim), DEV_IDLEN);
 +		cpi->transport = XPORT_SPI;
 +		cpi->transport_version = 2;
 +		cpi->protocol = PROTO_SCSI;
 +		cpi->protocol_version = SCSI_REV_2;
  		cpi->ccb_h.status = CAM_REQ_CMP;
  		break;
  	}
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 

From: Masayuki FUKUI <fukui.FreeBSD@fanet.net>
To: mav@FreeBSD.org
Cc: bug-followup@FreeBSD.org
Subject: Re: i386/147929: hptiop(4) (HighPoint RocketRAID 3xxx/4xxx driver) broken on FreeBSD 8.1-PRERELEASE
Date: Sat, 19 Jun 2010 23:44:57 +0900

 Thank you, Alex-san.
 
 Afterwards, I kept debugging, and I tried the same patch by chance yesterday.
 The RAID volume was normally detected as da(4).
 It works fine.
 
 I hope for fixing this probrem by the time FreeBSD 8.1 is released.
 
State-Changed-From-To: open->patched 
State-Changed-By: linimon 
State-Changed-When: Mon Jun 21 03:26:18 UTC 2010 
State-Changed-Why:  
reclassify and mark as patched. 


Responsible-Changed-From-To: freebsd-i386->mav 
Responsible-Changed-By: linimon 
Responsible-Changed-When: Mon Jun 21 03:26:18 UTC 2010 
Responsible-Changed-Why:  

http://www.freebsd.org/cgi/query-pr.cgi?pr=147929 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/147929: commit references a PR
Date: Tue, 22 Jun 2010 19:23:13 +0000 (UTC)

 Author: mav
 Date: Tue Jun 22 19:22:34 2010
 New Revision: 209438
 URL: http://svn.freebsd.org/changeset/base/209438
 
 Log:
   MFC r209340:
   Report transport type in XPT_PATH_INQ.
   
   PR:		i386/147929
 
 Modified:
   stable/8/sys/dev/hptiop/hptiop.c
 Directory Properties:
   stable/8/sys/   (props changed)
   stable/8/sys/amd64/include/xen/   (props changed)
   stable/8/sys/cddl/contrib/opensolaris/   (props changed)
   stable/8/sys/contrib/dev/acpica/   (props changed)
   stable/8/sys/contrib/pf/   (props changed)
   stable/8/sys/dev/ixgbe/   (props changed)
   stable/8/sys/dev/xen/xenpci/   (props changed)
   stable/8/sys/geom/sched/   (props changed)
 
 Modified: stable/8/sys/dev/hptiop/hptiop.c
 ==============================================================================
 --- stable/8/sys/dev/hptiop/hptiop.c	Tue Jun 22 19:17:57 2010	(r209437)
 +++ stable/8/sys/dev/hptiop/hptiop.c	Tue Jun 22 19:22:34 2010	(r209438)
 @@ -1823,6 +1823,10 @@ scsi_done:
  		strncpy(cpi->sim_vid, "FreeBSD", SIM_IDLEN);
  		strncpy(cpi->hba_vid, "HPT   ", HBA_IDLEN);
  		strncpy(cpi->dev_name, cam_sim_name(sim), DEV_IDLEN);
 +		cpi->transport = XPORT_SPI;
 +		cpi->transport_version = 2;
 +		cpi->protocol = PROTO_SCSI;
 +		cpi->protocol_version = SCSI_REV_2;
  		cpi->ccb_h.status = CAM_REQ_CMP;
  		break;
  	}
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/147929: commit references a PR
Date: Wed, 23 Jun 2010 10:05:16 +0000 (UTC)

 Author: mav
 Date: Wed Jun 23 10:04:48 2010
 New Revision: 209457
 URL: http://svn.freebsd.org/changeset/base/209457
 
 Log:
   MFC r209340:
   Report transport type in XPT_PATH_INQ.
   
   PR:		i386/147929
   Approved by:	re (kib)
 
 Modified:
   releng/8.1/sys/dev/hptiop/hptiop.c
 Directory Properties:
   releng/8.1/sys/   (props changed)
   releng/8.1/sys/amd64/include/xen/   (props changed)
   releng/8.1/sys/cddl/contrib/opensolaris/   (props changed)
   releng/8.1/sys/contrib/dev/acpica/   (props changed)
   releng/8.1/sys/contrib/pf/   (props changed)
   releng/8.1/sys/dev/xen/xenpci/   (props changed)
   releng/8.1/sys/geom/sched/   (props changed)
 
 Modified: releng/8.1/sys/dev/hptiop/hptiop.c
 ==============================================================================
 --- releng/8.1/sys/dev/hptiop/hptiop.c	Wed Jun 23 05:43:21 2010	(r209456)
 +++ releng/8.1/sys/dev/hptiop/hptiop.c	Wed Jun 23 10:04:48 2010	(r209457)
 @@ -1823,6 +1823,10 @@ scsi_done:
  		strncpy(cpi->sim_vid, "FreeBSD", SIM_IDLEN);
  		strncpy(cpi->hba_vid, "HPT   ", HBA_IDLEN);
  		strncpy(cpi->dev_name, cam_sim_name(sim), DEV_IDLEN);
 +		cpi->transport = XPORT_SPI;
 +		cpi->transport_version = 2;
 +		cpi->protocol = PROTO_SCSI;
 +		cpi->protocol_version = SCSI_REV_2;
  		cpi->ccb_h.status = CAM_REQ_CMP;
  		break;
  	}
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: patched->closed 
State-Changed-By: mav 
State-Changed-When: Mon Jun 28 16:03:29 UTC 2010 
State-Changed-Why:  
Patch merged to 8.1-RELEASE. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147929 
>Unformatted:
