From nobody@FreeBSD.org  Tue Sep  3 18:01:39 2013
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115])
	(using TLSv1 with cipher ADH-AES256-SHA (256/256 bits))
	(No client certificate requested)
	by hub.freebsd.org (Postfix) with ESMTP id 11A2E778
	for <freebsd-gnats-submit@FreeBSD.org>; Tue,  3 Sep 2013 18:01:39 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from oldred.freebsd.org (oldred.freebsd.org [8.8.178.121])
	(using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))
	(No client certificate requested)
	by mx1.freebsd.org (Postfix) with ESMTPS id F3B21262B
	for <freebsd-gnats-submit@FreeBSD.org>; Tue,  3 Sep 2013 18:01:38 +0000 (UTC)
Received: from oldred.freebsd.org ([127.0.1.6])
	by oldred.freebsd.org (8.14.5/8.14.7) with ESMTP id r83I1cVo058856
	for <freebsd-gnats-submit@FreeBSD.org>; Tue, 3 Sep 2013 18:01:38 GMT
	(envelope-from nobody@oldred.freebsd.org)
Received: (from nobody@localhost)
	by oldred.freebsd.org (8.14.5/8.14.5/Submit) id r83I1cHO058831;
	Tue, 3 Sep 2013 18:01:38 GMT
	(envelope-from nobody)
Message-Id: <201309031801.r83I1cHO058831@oldred.freebsd.org>
Date: Tue, 3 Sep 2013 18:01:38 GMT
From: Xiguang Wang <kurapica@gmail.com>
To: freebsd-gnats-submit@FreeBSD.org
Subject: mps driver broken in 9.2 pre-release
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         181784
>Category:       kern
>Synopsis:       [mps] mps driver broken in 9.2 pre-release [regression]
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    ken
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Tue Sep 03 18:10:02 UTC 2013
>Closed-Date:    Mon Sep 23 22:10:04 UTC 2013
>Last-Modified:  Mon Sep 23 22:10:04 UTC 2013
>Originator:     Xiguang Wang
>Release:        9-STABLE (r255183)
>Organization:
Peking University
>Environment:
FreeBSD SVR1 9.2-PRERELEASE FreeBSD 9.2-PRERELEASE #26: Sun Aug 25 16:21:53 CST 2013     wangxg@XTYLAB_SVR1:/usr/obj/usr/src/sys/GENERIC  amd64
>Description:
I just upgraded my server(Dell T110 with SAS2008 raid controller, 1
hardware RAID-1 array) to r255103. Last time I upgraded it was at Aug.
25(see Environment), and it works fine.

After I installed the kernel(r255183), it cannot boot and stuck at
mountroot prompt. It says that "mounting from ufs:/dev/da0p2 failed
with error 19".

I tried typing "?" to list bootable devices, but only CD-ROM is on list,
seems /dev/da0 is not recognized by the new kernel.

I then booted kernel.old, and everying is OK. So it is not my hardware
problem.

I think the mps driver MFCed has some bug in it. If you need more details,
please contact me via email. I normally reply in 24h.

Thank you.
>How-To-Repeat:
Find a SAS2008 card, connect a harddisk to it, install the lastest version
of 9-STABLE and boot.
>Fix:
None.

>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-bugs->ken 
Responsible-Changed-By: linimon 
Responsible-Changed-When: Tue Sep 3 22:51:58 UTC 2013 
Responsible-Changed-Why:  
Ken, you seem to have done most of the large changes here recently, so 
could you take a look, please?  Thanks. 

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

From: "Kenneth D. Merry" <ken@freebsd.org>
To: Xiguang Wang <kurapica@gmail.com>
Cc: freebsd-gnats-submit@freebsd.org
Subject: Re: kern/181784: mps driver broken in 9.2 pre-release
Date: Tue, 3 Sep 2013 20:17:14 -0600

 On Tue, Sep 03, 2013 at 18:01:38 +0000, Xiguang Wang wrote:
 > 
 > >Number:         181784
 > >Category:       kern
 > >Synopsis:       mps driver broken in 9.2 pre-release
 > >Confidential:   no
 > >Severity:       non-critical
 > >Priority:       low
 > >Responsible:    freebsd-bugs
 > >State:          open
 > >Quarter:        
 > >Keywords:       
 > >Date-Required:
 > >Class:          sw-bug
 > >Submitter-Id:   current-users
 > >Arrival-Date:   Tue Sep 03 18:10:02 UTC 2013
 > >Closed-Date:
 > >Last-Modified:
 > >Originator:     Xiguang Wang
 > >Release:        9-STABLE (r255183)
 > >Organization:
 > Peking University
 > >Environment:
 > FreeBSD SVR1 9.2-PRERELEASE FreeBSD 9.2-PRERELEASE #26: Sun Aug 25 16:21:53 CST 2013     wangxg@XTYLAB_SVR1:/usr/obj/usr/src/sys/GENERIC  amd64
 > >Description:
 > I just upgraded my server(Dell T110 with SAS2008 raid controller, 1 hardware RAID-1 array) to r255103. Last time I upgraded it was at Aug. 25(see Environment), and it works fine.
 > 
 > After I installed the kernel(r255183), it cannot boot and stuck at mountroot prompt. It says that "mounting from ufs:/dev/da0p2 failed with error 19".
 > 
 > I tried typing "?" to list bootable devices, but only CD-ROM is on list, seems /dev/da0 is not recognized by the new kernel.
 > 
 > I then booted kernel.old, and everying is OK. So it is not my hardware problem.
 > 
 > I think the mps driver MFCed has some bug in it. If you need more details, please contact me via email. I normally reply in 24h.
 > 
 > Thank you.
 > >How-To-Repeat:
 > Find a SAS2008 card, connect a harddisk to it, install the lastest version of 9-STABLE and boot.
 
 Please provide the full dmesg for the box in question, both with the old
 kernel and the new kernel.  You may need to do this via serial console for
 the new kernel.
 
 A picture is acceptable if you can get the mps(4) driver messages in the
 picture.
 
 You might also want to try the mpslsi(4) driver from LSI's web site, and
 see whether that works properly.  If it does, then the issue is a
 regression in FreeBSD.  If it does not, then LSI's version of the driver
 also has the problem.
 
 If you have the problem with LSI's driver, then it is something they need
 to fix.
 
 Ken
 -- 
 Kenneth Merry
 ken@FreeBSD.ORG

From: Kurapica <kurapica@gmail.com>
To: "Kenneth D. Merry" <ken@freebsd.org>
Cc: freebsd-gnats-submit <freebsd-gnats-submit@freebsd.org>
Subject: Re: kern/181784: mps driver broken in 9.2 pre-release
Date: Fri, 6 Sep 2013 19:31:15 +0800

 --bcaec5196d7708428b04e5b56218
 Content-Type: text/plain; charset=ISO-8859-1
 
 dmesg of the two boxes is attached here:
 
 http://162.105.164.41/dmesg.zip
 
 I found that the new driver does not probe mps0:0:1:0 where my raid(da0) is
 attached.
 
 I'll try to test LSI's driver in days.
 
 
 Kurapica
 
 
 2013/9/4 Kenneth D. Merry <ken@freebsd.org>
 
 > On Tue, Sep 03, 2013 at 18:01:38 +0000, Xiguang Wang wrote:
 > >
 > > >Number:         181784
 > > >Category:       kern
 > > >Synopsis:       mps driver broken in 9.2 pre-release
 > > >Confidential:   no
 > > >Severity:       non-critical
 > > >Priority:       low
 > > >Responsible:    freebsd-bugs
 > > >State:          open
 > > >Quarter:
 > > >Keywords:
 > > >Date-Required:
 > > >Class:          sw-bug
 > > >Submitter-Id:   current-users
 > > >Arrival-Date:   Tue Sep 03 18:10:02 UTC 2013
 > > >Closed-Date:
 > > >Last-Modified:
 > > >Originator:     Xiguang Wang
 > > >Release:        9-STABLE (r255183)
 > > >Organization:
 > > Peking University
 > > >Environment:
 > > FreeBSD SVR1 9.2-PRERELEASE FreeBSD 9.2-PRERELEASE #26: Sun Aug 25
 > 16:21:53 CST 2013     wangxg@XTYLAB_SVR1:/usr/obj/usr/src/sys/GENERIC
 >  amd64
 > > >Description:
 > > I just upgraded my server(Dell T110 with SAS2008 raid controller, 1
 > hardware RAID-1 array) to r255103. Last time I upgraded it was at Aug.
 > 25(see Environment), and it works fine.
 > >
 > > After I installed the kernel(r255183), it cannot boot and stuck at
 > mountroot prompt. It says that "mounting from ufs:/dev/da0p2 failed with
 > error 19".
 > >
 > > I tried typing "?" to list bootable devices, but only CD-ROM is on list,
 > seems /dev/da0 is not recognized by the new kernel.
 > >
 > > I then booted kernel.old, and everying is OK. So it is not my hardware
 > problem.
 > >
 > > I think the mps driver MFCed has some bug in it. If you need more
 > details, please contact me via email. I normally reply in 24h.
 > >
 > > Thank you.
 > > >How-To-Repeat:
 > > Find a SAS2008 card, connect a harddisk to it, install the lastest
 > version of 9-STABLE and boot.
 >
 > Please provide the full dmesg for the box in question, both with the old
 > kernel and the new kernel.  You may need to do this via serial console for
 > the new kernel.
 >
 > A picture is acceptable if you can get the mps(4) driver messages in the
 > picture.
 >
 > You might also want to try the mpslsi(4) driver from LSI's web site, and
 > see whether that works properly.  If it does, then the issue is a
 > regression in FreeBSD.  If it does not, then LSI's version of the driver
 > also has the problem.
 >
 > If you have the problem with LSI's driver, then it is something they need
 > to fix.
 >
 > Ken
 > --
 > Kenneth Merry
 > ken@FreeBSD.ORG
 >
 
 --bcaec5196d7708428b04e5b56218
 Content-Type: text/html; charset=ISO-8859-1
 Content-Transfer-Encoding: quoted-printable
 
 <div dir=3D"ltr"><span style=3D"font-family:arial,sans-serif;font-size:14px=
 ">dmesg of the two boxes is attached here:</span><div><font face=3D"arial, =
 sans-serif"><span style=3D"font-size:14px"><br></span></font></div><div><a =
 href=3D"http://162.105.164.41/dmesg.zip">http://162.105.164.41/dmesg.zip</a=
 ></div>
 
 <div><div><div style=3D"font-family:arial,sans-serif;font-size:14px"><br></=
 div><div style=3D"font-family:arial,sans-serif;font-size:14px">I found that=
  the new driver does not probe mps0:0:1:0 where my raid(da0) is attached.</=
 div>
 
 <div style=3D"font-family:arial,sans-serif;font-size:14px"><br></div><div s=
 tyle=3D"font-family:arial,sans-serif;font-size:14px">I&#39;ll try to test L=
 SI&#39;s driver in days.</div><div style=3D"font-family:arial,sans-serif;fo=
 nt-size:14px">
 
 <br></div><div style=3D"font-family:arial,sans-serif;font-size:14px"><br></=
 div><div style=3D"font-family:arial,sans-serif;font-size:14px">Kurapica</di=
 v></div></div></div><div class=3D"gmail_extra"><br><br><div class=3D"gmail_=
 quote">
 
 2013/9/4 Kenneth D. Merry <span dir=3D"ltr">&lt;<a href=3D"mailto:ken@freeb=
 sd.org" target=3D"_blank">ken@freebsd.org</a>&gt;</span><br><blockquote cla=
 ss=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;pa=
 dding-left:1ex">
 
 On Tue, Sep 03, 2013 at 18:01:38 +0000, Xiguang Wang wrote:<br>
 &gt;<br>
 &gt; &gt;Number: =A0 =A0 =A0 =A0 181784<br>
 &gt; &gt;Category: =A0 =A0 =A0 kern<br>
 <div class=3D"im">&gt; &gt;Synopsis: =A0 =A0 =A0 mps driver broken in 9.2 p=
 re-release<br>
 </div>&gt; &gt;Confidential: =A0 no<br>
 &gt; &gt;Severity: =A0 =A0 =A0 non-critical<br>
 &gt; &gt;Priority: =A0 =A0 =A0 low<br>
 &gt; &gt;Responsible: =A0 =A0freebsd-bugs<br>
 &gt; &gt;State: =A0 =A0 =A0 =A0 =A0open<br>
 &gt; &gt;Quarter:<br>
 &gt; &gt;Keywords:<br>
 &gt; &gt;Date-Required:<br>
 &gt; &gt;Class: =A0 =A0 =A0 =A0 =A0sw-bug<br>
 &gt; &gt;Submitter-Id: =A0 current-users<br>
 <div class=3D"im">&gt; &gt;Arrival-Date: =A0 Tue Sep 03 18:10:02 UTC 2013<b=
 r>
 </div>&gt; &gt;Closed-Date:<br>
 &gt; &gt;Last-Modified:<br>
 &gt; &gt;Originator: =A0 =A0 Xiguang Wang<br>
 &gt; &gt;Release: =A0 =A0 =A0 =A09-STABLE (r255183)<br>
 &gt; &gt;Organization:<br>
 &gt; Peking University<br>
 &gt; &gt;Environment:<br>
 &gt; FreeBSD SVR1 9.2-PRERELEASE FreeBSD 9.2-PRERELEASE #26: Sun Aug 25 16:=
 21:53 CST 2013 =A0 =A0 wangxg@XTYLAB_SVR1:/usr/obj/usr/src/sys/GENERIC =A0a=
 md64<br>
 &gt; &gt;Description:<br>
 &gt; I just upgraded my server(Dell T110 with SAS2008 raid controller, 1 ha=
 rdware RAID-1 array) to r255103. Last time I upgraded it was at Aug. 25(see=
  Environment), and it works fine.<br>
 &gt;<br>
 &gt; After I installed the kernel(r255183), it cannot boot and stuck at mou=
 ntroot prompt. It says that &quot;mounting from ufs:/dev/da0p2 failed with =
 error 19&quot;.<br>
 &gt;<br>
 &gt; I tried typing &quot;?&quot; to list bootable devices, but only CD-ROM=
  is on list, seems /dev/da0 is not recognized by the new kernel.<br>
 &gt;<br>
 &gt; I then booted kernel.old, and everying is OK. So it is not my hardware=
  problem.<br>
 &gt;<br>
 &gt; I think the mps driver MFCed has some bug in it. If you need more deta=
 ils, please contact me via email. I normally reply in 24h.<br>
 &gt;<br>
 &gt; Thank you.<br>
 &gt; &gt;How-To-Repeat:<br>
 &gt; Find a SAS2008 card, connect a harddisk to it, install the lastest ver=
 sion of 9-STABLE and boot.<br>
 <br>
 Please provide the full dmesg for the box in question, both with the old<br=
 >
 kernel and the new kernel. =A0You may need to do this via serial console fo=
 r<br>
 the new kernel.<br>
 <br>
 A picture is acceptable if you can get the mps(4) driver messages in the<br=
 >
 picture.<br>
 <br>
 You might also want to try the mpslsi(4) driver from LSI&#39;s web site, an=
 d<br>
 see whether that works properly. =A0If it does, then the issue is a<br>
 regression in FreeBSD. =A0If it does not, then LSI&#39;s version of the dri=
 ver<br>
 also has the problem.<br>
 <br>
 If you have the problem with LSI&#39;s driver, then it is something they ne=
 ed<br>
 to fix.<br>
 <br>
 Ken<br>
 <span class=3D"HOEnZb"><font color=3D"#888888">--<br>
 Kenneth Merry<br>
 ken@FreeBSD.ORG<br>
 </font></span></blockquote></div><br></div>
 
 --bcaec5196d7708428b04e5b56218--

From: "Kenneth D. Merry" <ken@freebsd.org>
To: Kurapica <kurapica@gmail.com>
Cc: freebsd-gnats-submit <freebsd-gnats-submit@freebsd.org>
Subject: Re: kern/181784: mps driver broken in 9.2 pre-release
Date: Fri, 6 Sep 2013 17:06:03 -0600

 On Fri, Sep 06, 2013 at 19:31:15 +0800, Kurapica wrote:
 > dmesg of the two boxes is attached here:
 > 
 > http://162.105.164.41/dmesg.zip
 > 
 > I found that the new driver does not probe mps0:0:1:0 where my raid(da0) is
 > attached.
 > 
 > I'll try to test LSI's driver in days.
 
 I can't download the dmesg.  fetch reports the file is truncated at varying
 points:
 
 dmesg.zip                                      32% of 4625 kB  136 kBps 00m22s
 fetch: dmesg.zip appears to be truncated: 1539799/4736036 bytes
 
 Can you email it to me?
 
 Thanks,
 
 Ken
 -- 
 Kenneth Merry
 ken@FreeBSD.ORG

From: Kurapica <kurapica@gmail.com>
To: "Kenneth D. Merry" <ken@freebsd.org>
Cc: freebsd-gnats-submit <freebsd-gnats-submit@freebsd.org>
Subject: Re: kern/181784: mps driver broken in 9.2 pre-release
Date: Sat, 7 Sep 2013 22:22:10 +0800

 --089e01160098a3654804e5cbe288
 Content-Type: text/plain; charset=ISO-8859-1
 
 I tried to email the package as a attachment, but the mailer  said "550
 5.7.1 Please keep viruses to yourself"
 
 So how can I share the file to you? I'm in China and most of file share
 site(such as dropbox) is filtered by GFW.
 
 
 2013/9/7 Kenneth D. Merry <ken@freebsd.org>
 
 > On Fri, Sep 06, 2013 at 19:31:15 +0800, Kurapica wrote:
 > > dmesg of the two boxes is attached here:
 > >
 > > http://162.105.164.41/dmesg.zip
 > >
 > > I found that the new driver does not probe mps0:0:1:0 where my raid(da0)
 > is
 > > attached.
 > >
 > > I'll try to test LSI's driver in days.
 >
 > I can't download the dmesg.  fetch reports the file is truncated at varying
 > points:
 >
 > dmesg.zip                                      32% of 4625 kB  136 kBps
 > 00m22s
 > fetch: dmesg.zip appears to be truncated: 1539799/4736036 bytes
 >
 > Can you email it to me?
 >
 > Thanks,
 >
 > Ken
 > --
 > Kenneth Merry
 > ken@FreeBSD.ORG
 >
 
 --089e01160098a3654804e5cbe288
 Content-Type: text/html; charset=ISO-8859-1
 Content-Transfer-Encoding: quoted-printable
 
 <div dir=3D"ltr">I tried to email the package as a attachment, but the mail=
 er =A0said &quot;<span style=3D"font-family:arial,sans-serif;font-size:14px=
 ">550 5.7.1 Please keep viruses to yourself</span>&quot;<div><br></div><div=
 >So how can I share the file to you? I&#39;m in China and most of file shar=
 e site(such as dropbox) is filtered by GFW.</div>
 
 </div><div class=3D"gmail_extra"><br><br><div class=3D"gmail_quote">2013/9/=
 7 Kenneth D. Merry <span dir=3D"ltr">&lt;<a href=3D"mailto:ken@freebsd.org"=
  target=3D"_blank">ken@freebsd.org</a>&gt;</span><br><blockquote class=3D"g=
 mail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-l=
 eft:1ex">
 
 <div class=3D"im">On Fri, Sep 06, 2013 at 19:31:15 +0800, Kurapica wrote:<b=
 r>
 &gt; dmesg of the two boxes is attached here:<br>
 &gt;<br>
 &gt; <a href=3D"http://162.105.164.41/dmesg.zip" target=3D"_blank">http://1=
 62.105.164.41/dmesg.zip</a><br>
 &gt;<br>
 &gt; I found that the new driver does not probe mps0:0:1:0 where my raid(da=
 0) is<br>
 &gt; attached.<br>
 &gt;<br>
 &gt; I&#39;ll try to test LSI&#39;s driver in days.<br>
 <br>
 </div>I can&#39;t download the dmesg. =A0fetch reports the file is truncate=
 d at varying<br>
 points:<br>
 <br>
 dmesg.zip =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =
 =A0 =A0 =A032% of 4625 kB =A0136 kBps 00m22s<br>
 fetch: dmesg.zip appears to be truncated: 1539799/4736036 bytes<br>
 <br>
 Can you email it to me?<br>
 <br>
 Thanks,<br>
 <div class=3D"HOEnZb"><div class=3D"h5"><br>
 Ken<br>
 --<br>
 Kenneth Merry<br>
 ken@FreeBSD.ORG<br>
 </div></div></blockquote></div><br></div>
 
 --089e01160098a3654804e5cbe288--

From: "Kenneth D. Merry" <ken@freebsd.org>
To: Kurapica <kurapica@gmail.com>
Cc: freebsd-gnats-submit <freebsd-gnats-submit@freebsd.org>
Subject: Re: kern/181784: mps driver broken in 9.2 pre-release
Date: Sat, 7 Sep 2013 16:55:14 -0600

 Try sending the jpegs by themselves as attachments.  If it really is a
 virus filter, it may respond better to images than zip files.
 
 My guess is that the GFW is also preventing me from downloading the file.
 
 Ken
 
 On Sat, Sep 07, 2013 at 22:22:10 +0800, Kurapica wrote:
 > I tried to email the package as a attachment, but the mailer  said "550
 > 5.7.1 Please keep viruses to yourself"
 > 
 > So how can I share the file to you? I'm in China and most of file share
 > site(such as dropbox) is filtered by GFW.
 > 
 > 
 > 2013/9/7 Kenneth D. Merry <ken@freebsd.org>
 > 
 > > On Fri, Sep 06, 2013 at 19:31:15 +0800, Kurapica wrote:
 > > > dmesg of the two boxes is attached here:
 > > >
 > > > http://162.105.164.41/dmesg.zip
 > > >
 > > > I found that the new driver does not probe mps0:0:1:0 where my raid(da0)
 > > is
 > > > attached.
 > > >
 > > > I'll try to test LSI's driver in days.
 > >
 > > I can't download the dmesg.  fetch reports the file is truncated at varying
 > > points:
 > >
 > > dmesg.zip                                      32% of 4625 kB  136 kBps
 > > 00m22s
 > > fetch: dmesg.zip appears to be truncated: 1539799/4736036 bytes
 > >
 > > Can you email it to me?
 > >
 > > Thanks,
 > >
 > > Ken
 > > --
 > > Kenneth Merry
 > > ken@FreeBSD.ORG
 > >
 
 -- 
 Kenneth Merry
 ken@FreeBSD.ORG

From: "Kenneth D. Merry" <ken@freebsd.org>
To: Kurapica <kurapica@gmail.com>
Cc: freebsd-gnats-submit <freebsd-gnats-submit@freebsd.org>
Subject: Re: kern/181784: mps driver broken in 9.2 pre-release
Date: Tue, 10 Sep 2013 14:33:00 -0600

 --AqsLC8rIMeq19msA
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 
 On Sun, Sep 08, 2013 at 20:01:22 +0800, Kurapica wrote:
 > OK, I'll try again.
 
 I got the images this time, thanks!
 
 Please try the attatched patch and see whether it works for you.
 
 The issue was that with the new rescanning changes, we weren't scanning
 RAID volumes that got created.
 
 This doesn't address the extra verbosity that happens when you enable IR
 volumes.  But that's a cosmetic, not a functional issue.
 
 Ken
 -- 
 Kenneth Merry
 ken@FreeBSD.ORG
 
 --AqsLC8rIMeq19msA
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: attachment; filename="mps_sas_lsi.c.20130910.1.txt"
 
 ==== //depot/users/kenm/FreeBSD-stable2/9/sys/dev/mps/mps_sas_lsi.c#10 - /usr/home/kenm/perforce4/kenm/FreeBSD-stable2/9/sys/dev/mps/mps_sas_lsi.c ====
 *** /tmp/tmp.91039.2	Tue Sep 10 14:13:13 2013
 --- /usr/home/kenm/perforce4/kenm/FreeBSD-stable2/9/sys/dev/mps/mps_sas_lsi.c	Tue Sep 10 14:07:08 2013
 ***************
 *** 892,898 ****
 --- 892,901 ----
   		free(lun, M_MPT2);
   	}
   	SLIST_INIT(&targ->luns);
 + #if ((__FreeBSD_version >= 1000000) && (__FreeBSD_version < 1000039)) || \
 +     (__FreeBSD_version < 902502)
   	if ((sassc->flags & MPSSAS_IN_STARTUP) == 0)
 + #endif
   		mpssas_rescan_target(sc, targ);
   	mps_dprint(sc, MPS_MAPPING, "RAID target id %d added (WWID = 0x%jx)\n",
   	    targ->tid, wwid);
 
 --AqsLC8rIMeq19msA--

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/181784: commit references a PR
Date: Thu, 12 Sep 2013 22:06:25 +0000 (UTC)

 Author: ken
 Date: Thu Sep 12 22:06:12 2013
 New Revision: 255501
 URL: http://svnweb.freebsd.org/changeset/base/255501
 
 Log:
   Fix an issue that caused Integrated RAID volumes on LSI mps(4) controllers
   to not get scanned on boot.
   
   The problem originated in change 253549.  With the change to the mps(4)
   driver to scan only targets that it knows it has (as opposed to scanning
   the entire bus), scanning RAID volumes on boot was omitted.
   
   So, for versions of FreeBSD that have the scanning changes
   (__FreeBSD_version 1000039 and higher), scan RAID volumes that are added
   whether or not we're booting.
   
   PR:		kern/181784
   Reported by:	Xiguang Wang <kurapica@gmail.com>
   Tested by:	Dennis Glatting <dg@pki2.com>
   Sponsored by:	Spectra Logic
   Approved by:	re (delphij)
   MFC After:	3 days
 
 Modified:
   head/sys/dev/mps/mps_sas_lsi.c
 
 Modified: head/sys/dev/mps/mps_sas_lsi.c
 ==============================================================================
 --- head/sys/dev/mps/mps_sas_lsi.c	Thu Sep 12 21:24:59 2013	(r255500)
 +++ head/sys/dev/mps/mps_sas_lsi.c	Thu Sep 12 22:06:12 2013	(r255501)
 @@ -898,7 +898,9 @@ mpssas_volume_add(struct mps_softc *sc, 
  		free(lun, M_MPT2);
  	}
  	SLIST_INIT(&targ->luns);
 +#if __FreeBSD_version < 1000039
  	if ((sassc->flags & MPSSAS_IN_STARTUP) == 0)
 +#endif
  		mpssas_rescan_target(sc, targ);
  	mps_dprint(sc, MPS_MAPPING, "RAID target id %d added (WWID = 0x%jx)\n",
  	    targ->tid, wwid);
 _______________________________________________
 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: Kurapica <kurapica@gmail.com>
To: "Kenneth D. Merry" <ken@freebsd.org>
Cc: freebsd-gnats-submit <freebsd-gnats-submit@freebsd.org>
Subject: Re: kern/181784: mps driver broken in 9.2 pre-release
Date: Mon, 16 Sep 2013 18:21:22 +0800

 --089e0122f4800828b004e67d92e8
 Content-Type: text/plain; charset=ISO-8859-1
 
 Sorry for the delay of testing, but it works fine.
 
 Thank you Ken~
 
 
 
 2013/9/11 Kenneth D. Merry <ken@freebsd.org>
 
 > On Sun, Sep 08, 2013 at 20:01:22 +0800, Kurapica wrote:
 > > OK, I'll try again.
 >
 > I got the images this time, thanks!
 >
 > Please try the attatched patch and see whether it works for you.
 >
 > The issue was that with the new rescanning changes, we weren't scanning
 > RAID volumes that got created.
 >
 > This doesn't address the extra verbosity that happens when you enable IR
 > volumes.  But that's a cosmetic, not a functional issue.
 >
 > Ken
 > --
 > Kenneth Merry
 > ken@FreeBSD.ORG
 >
 
 --089e0122f4800828b004e67d92e8
 Content-Type: text/html; charset=ISO-8859-1
 Content-Transfer-Encoding: quoted-printable
 
 <div dir=3D"ltr">Sorry for the delay of testing, but it works fine.<div><br=
 ></div><div>Thank you Ken~</div><div><br></div></div><div class=3D"gmail_ex=
 tra"><br><br><div class=3D"gmail_quote">2013/9/11 Kenneth D. Merry <span di=
 r=3D"ltr">&lt;<a href=3D"mailto:ken@freebsd.org" target=3D"_blank">ken@free=
 bsd.org</a>&gt;</span><br>
 
 <blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1p=
 x #ccc solid;padding-left:1ex"><div class=3D"im">On Sun, Sep 08, 2013 at 20=
 :01:22 +0800, Kurapica wrote:<br>
 &gt; OK, I&#39;ll try again.<br>
 <br>
 </div>I got the images this time, thanks!<br>
 <br>
 Please try the attatched patch and see whether it works for you.<br>
 <br>
 The issue was that with the new rescanning changes, we weren&#39;t scanning=
 <br>
 RAID volumes that got created.<br>
 <br>
 This doesn&#39;t address the extra verbosity that happens when you enable I=
 R<br>
 volumes. =A0But that&#39;s a cosmetic, not a functional issue.<br>
 <div class=3D"HOEnZb"><div class=3D"h5"><br>
 Ken<br>
 --<br>
 Kenneth Merry<br>
 ken@FreeBSD.ORG<br>
 </div></div></blockquote></div><br></div>
 
 --089e0122f4800828b004e67d92e8--

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/181784: commit references a PR
Date: Mon, 23 Sep 2013 21:52:15 +0000 (UTC)

 Author: ken
 Date: Mon Sep 23 21:52:07 2013
 New Revision: 255830
 URL: http://svnweb.freebsd.org/changeset/base/255830
 
 Log:
   MFC r255501
   
   This is slightly modified from the FreeBSD/head version, to include
   version checks for the scanning changes for not only FreeBSD/head
   (1000039 and higher) but also stable/9 (902502 and higher).
   
     ------------------------------------------------------------------------
     r255501 | ken | 2013-09-12 16:06:12 -0600 (Thu, 12 Sep 2013) | 18 lines
   
     Fix an issue that caused Integrated RAID volumes on LSI mps(4) controllers
     to not get scanned on boot.
   
     The problem originated in change 253549.  With the change to the mps(4)
     driver to scan only targets that it knows it has (as opposed to scanning
     the entire bus), scanning RAID volumes on boot was omitted.
   
     So, for versions of FreeBSD that have the scanning changes
     (__FreeBSD_version 1000039 and higher), scan RAID volumes that are added
     whether or not we're booting.
   
     PR:             kern/181784
     Reported by:    Xiguang Wang <kurapica@gmail.com>
     Tested by:      Dennis Glatting <dg@pki2.com>
     Sponsored by:   Spectra Logic
   
   PR:		kern/181784
 
 Modified:
   stable/9/sys/dev/mps/mps_sas_lsi.c
 
 Modified: stable/9/sys/dev/mps/mps_sas_lsi.c
 ==============================================================================
 --- stable/9/sys/dev/mps/mps_sas_lsi.c	Mon Sep 23 20:35:54 2013	(r255829)
 +++ stable/9/sys/dev/mps/mps_sas_lsi.c	Mon Sep 23 21:52:07 2013	(r255830)
 @@ -892,7 +892,10 @@ mpssas_volume_add(struct mps_softc *sc, 
  		free(lun, M_MPT2);
  	}
  	SLIST_INIT(&targ->luns);
 +#if ((__FreeBSD_version >= 1000000) && (__FreeBSD_version < 1000039)) || \
 +    (__FreeBSD_version < 902502)
  	if ((sassc->flags & MPSSAS_IN_STARTUP) == 0)
 +#endif
  		mpssas_rescan_target(sc, targ);
  	mps_dprint(sc, MPS_MAPPING, "RAID target id %d added (WWID = 0x%jx)\n",
  	    targ->tid, wwid);
 _______________________________________________
 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/181784: commit references a PR
Date: Mon, 23 Sep 2013 21:52:15 +0000 (UTC)

 Author: ken
 Date: Mon Sep 23 21:52:07 2013
 New Revision: 255830
 URL: http://svnweb.freebsd.org/changeset/base/255830
 
 Log:
   MFC r255501
   
   This is slightly modified from the FreeBSD/head version, to include
   version checks for the scanning changes for not only FreeBSD/head
   (1000039 and higher) but also stable/9 (902502 and higher).
   
     ------------------------------------------------------------------------
     r255501 | ken | 2013-09-12 16:06:12 -0600 (Thu, 12 Sep 2013) | 18 lines
   
     Fix an issue that caused Integrated RAID volumes on LSI mps(4) controllers
     to not get scanned on boot.
   
     The problem originated in change 253549.  With the change to the mps(4)
     driver to scan only targets that it knows it has (as opposed to scanning
     the entire bus), scanning RAID volumes on boot was omitted.
   
     So, for versions of FreeBSD that have the scanning changes
     (__FreeBSD_version 1000039 and higher), scan RAID volumes that are added
     whether or not we're booting.
   
     PR:             kern/181784
     Reported by:    Xiguang Wang <kurapica@gmail.com>
     Tested by:      Dennis Glatting <dg@pki2.com>
     Sponsored by:   Spectra Logic
   
   PR:		kern/181784
 
 Modified:
   stable/9/sys/dev/mps/mps_sas_lsi.c
 
 Modified: stable/9/sys/dev/mps/mps_sas_lsi.c
 ==============================================================================
 --- stable/9/sys/dev/mps/mps_sas_lsi.c	Mon Sep 23 20:35:54 2013	(r255829)
 +++ stable/9/sys/dev/mps/mps_sas_lsi.c	Mon Sep 23 21:52:07 2013	(r255830)
 @@ -892,7 +892,10 @@ mpssas_volume_add(struct mps_softc *sc, 
  		free(lun, M_MPT2);
  	}
  	SLIST_INIT(&targ->luns);
 +#if ((__FreeBSD_version >= 1000000) && (__FreeBSD_version < 1000039)) || \
 +    (__FreeBSD_version < 902502)
  	if ((sassc->flags & MPSSAS_IN_STARTUP) == 0)
 +#endif
  		mpssas_rescan_target(sc, targ);
  	mps_dprint(sc, MPS_MAPPING, "RAID target id %d added (WWID = 0x%jx)\n",
  	    targ->tid, wwid);
 _______________________________________________
 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: open->closed 
State-Changed-By: ken 
State-Changed-When: Mon Sep 23 22:08:37 UTC 2013 
State-Changed-Why:  
Fixed in FreeBSD/head in revision 255501 on September 12, 2013.  Fixed in 
FreeBSD stable/9 in revision 255830 on September 23, 2013. 

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