Re: PR/53059 CVS commit: src/sys/dev/ic

[Jaromír Doleček <jaromir.dolecek%gmail.com@localhost>]

The following reply was made to PR kern/53059; it has been noted by GNATS.

From: =?UTF-8?B?SmFyb23DrXIgRG9sZcSNZWs=?= <jaromir.dolecek%gmail.com@localhost>
To: Frank Kardel <kardel%netbsd.org@localhost>
Cc: gnats-bugs%netbsd.org@localhost
Subject: Re: PR/53059 CVS commit: src/sys/dev/ic
Date: Sat, 17 Mar 2018 10:38:29 +0100

 --94eb2c05f1f6c84ca80567987c43
 Content-Type: text/plain; charset="UTF-8"
 
 Try now with rev 1.36 please, should be fixed.
 
 Jaromir
 
 2018-03-17 9:10 GMT+01:00 Frank Kardel <kardel%netbsd.org@localhost>:
 >
 > Well, we are not quite there yet.
 >
 > I get a page fault trap, code=0.
 >
 > hand copied stack trace (no dump device available at this point, photo
 available on request).
 >
 > nvme_pt_fill() at nvme_pt_fill+0x1e
 >
 > nvme_poll() at nvme_poll+0xa0
 >
 > nvme_attach() atnvme_attch+0x9a4
 >
 > nvme_pci_attch() at name_pci_attach+0x530
 >
 > config_attach_loc()
 >
 > config_found_sm_loc()
 >
 > pci_probe_device()
 >
 > pci_enumerate_bus()
 >
 > pcirescan()
 >
 > pciattach()
 >
 > ....
 >
 > Frank
 >
 >
 >
 > On 03/17/18 01:30, Jaromir Dolecek wrote:
 >>
 >> The following reply was made to PR kern/53059; it has been noted by
 GNATS.
 >>
 >> From: "Jaromir Dolecek" <jdolecek%netbsd.org@localhost>
 >> To: gnats-bugs%gnats.NetBSD.org@localhost
 >> Cc:
 >> Subject: PR/53059 CVS commit: src/sys/dev/ic
 >> Date: Sat, 17 Mar 2018 00:28:03 +0000
 >>
 >>   Module Name:  src
 >>   Committed By: jdolecek
 >>   Date:         Sat Mar 17 00:28:03 UTC 2018
 >>     Modified Files:
 >>         src/sys/dev/ic: nvme.c
 >>     Log Message:
 >>   switch handling of passthrough commands to use queue, instead of
 polling
 >>     should fix PR kern/53059 by Frank Kardel
 >>       To generate a diff of this commit:
 >>   cvs rdiff -u -r1.34 -r1.35 src/sys/dev/ic/nvme.c
 >>     Please note that diffs are not public domain; they are subject to the
 >>   copyright notices on the relevant files.
 >>
 >
 >
 
 --94eb2c05f1f6c84ca80567987c43
 Content-Type: text/html; charset="UTF-8"
 Content-Transfer-Encoding: quoted-printable
 
 <div dir=3D"ltr">Try now with rev 1.36 please, should be fixed.<br><br>Jaro=
 mir<br><br>2018-03-17 9:10 GMT+01:00 Frank Kardel &lt;<a href=3D"mailto:kar=
 del%netbsd.org@localhost">kardel%netbsd.org@localhost</a>&gt;:<br>&gt;<br>&gt; Well, we are not=
  quite there yet.<br>&gt;<br>&gt; I get a page fault trap, code=3D0.<br>&gt=
 ;<br>&gt; hand copied stack trace (no dump device available at this point, =
 photo available on request).<br>&gt;<br>&gt; nvme_pt_fill() at nvme_pt_fill=
 +0x1e<br>&gt;<br>&gt; nvme_poll() at nvme_poll+0xa0<br>&gt;<br>&gt; nvme_at=
 tach() atnvme_attch+0x9a4<br>&gt;<br>&gt; nvme_pci_attch() at name_pci_atta=
 ch+0x530<br>&gt;<br>&gt; config_attach_loc()<br>&gt;<br>&gt; config_found_s=
 m_loc()<br>&gt;<br>&gt; pci_probe_device()<br>&gt;<br>&gt; pci_enumerate_bu=
 s()<br>&gt;<br>&gt; pcirescan()<br>&gt;<br>&gt; pciattach()<br>&gt;<br>&gt;=
  ....<br>&gt;<br>&gt; Frank<br>&gt;<br>&gt;<br>&gt;<br>&gt; On 03/17/18 01:=
 30, Jaromir Dolecek wrote:<br>&gt;&gt;<br>&gt;&gt; The following reply was =
 made to PR kern/53059; it has been noted by GNATS.<br>&gt;&gt;<br>&gt;&gt; =
 From: &quot;Jaromir Dolecek&quot; &lt;<a href=3D"mailto:jdolecek%netbsd.org@localhost=
 ">jdolecek%netbsd.org@localhost</a>&gt;<br>&gt;&gt; To: <a href=3D"mailto:gnats-bugs@=
 gnats.NetBSD.org">gnats-bugs%gnats.NetBSD.org@localhost</a><br>&gt;&gt; Cc:<br>&gt;&g=
 t; Subject: PR/53059 CVS commit: src/sys/dev/ic<br>&gt;&gt; Date: Sat, 17 M=
 ar 2018 00:28:03 +0000<br>&gt;&gt;<br>&gt;&gt; =C2=A0 Module Name: =C2=A0sr=
 c<br>&gt;&gt; =C2=A0 Committed By: jdolecek<br>&gt;&gt; =C2=A0 Date: =C2=A0=
  =C2=A0 =C2=A0 =C2=A0 Sat Mar 17 00:28:03 UTC 2018<br>&gt;&gt; =C2=A0 =C2=
 =A0 Modified Files:<br>&gt;&gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 src/sys/dev/ic:=
  nvme.c<br>&gt;&gt; =C2=A0 =C2=A0 Log Message:<br>&gt;&gt; =C2=A0 switch ha=
 ndling of passthrough commands to use queue, instead of polling<br>&gt;&gt;=
  =C2=A0 =C2=A0 should fix PR kern/53059 by Frank Kardel<br>&gt;&gt; =C2=A0 =
 =C2=A0 =C2=A0 To generate a diff of this commit:<br>&gt;&gt; =C2=A0 cvs rdi=
 ff -u -r1.34 -r1.35 src/sys/dev/ic/nvme.c<br>&gt;&gt; =C2=A0 =C2=A0 Please =
 note that diffs are not public domain; they are subject to the<br>&gt;&gt; =
 =C2=A0 copyright notices on the relevant files.<br>&gt;&gt; =C2=A0<br>&gt;<=
 br>&gt;<br></div>
 
 --94eb2c05f1f6c84ca80567987c43--