Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 5 Jun 2025 19:14:19 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        Charlie Li <vishwin@freebsd.org>
Cc:        John Baldwin <jhb@freebsd.org>, src-committers <src-committers@freebsd.org>,  "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>,  "<dev-commits-src-main@freebsd.org>" <dev-commits-src-main@freebsd.org>
Subject:   Re: git: 60159a98a837 - main - nvme: Move opcode and status code tables from base CAM to nvme_util.c
Message-ID:  <CANCZdfoyYathT2Q=TmVjr7ETvB9-hN1hVXgKgaUY5xxsrpDQLw@mail.gmail.com>
In-Reply-To: <371a7bf2-61ee-4528-9aec-846601973b78@freebsd.org>
References:  <202506052106.555L6XBV089396@gitrepo.freebsd.org> <53d9ce10-191c-4edf-85d1-41b3162aefa4@freebsd.org> <d2f95ea5-aa29-4e7a-804c-4980d1a29f9e@FreeBSD.org> <371a7bf2-61ee-4528-9aec-846601973b78@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000001520760636dcf401
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, Jun 5, 2025, 6:46=E2=80=AFPM Charlie Li <vishwin@freebsd.org> wrote=
:

> John Baldwin wrote:
> > On 6/5/25 19:29, Charlie Li wrote:
> >> <sys/dev/nvme.h> doesn't appear to be making it in nvme_all.h as the
> >> build complains about missing nvme_opcode_sbuf() in nvme_all.c.
> >
> > Are you getting a link error or a compile error?  I built this locally
> > several times
> > today to test various combinations (kernels with and without either CAM
> > or nvme and
> > loading nvme.ko in kernels without to make sure it worked).  I also
> > built GENERIC
> > prior to pushing and it built fine.
> >
> Compile error in buildworld.
>

In libcam?

Warner

> Sigh, so from ci.freebsd.org, it appears to be userland that is choking
> > (libcam)
> > rather than the kernel.  I'll fix shortly (mostly it just means removin=
g
> > the
> > #ifdef _KERNEL).
> >
>
> --
> Charlie Li
> ...nope, still don't have an exit line.
>

--0000000000001520760636dcf401
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"auto"><div><br><br><div class=3D"gmail_quote gmail_quote_contai=
ner"><div dir=3D"ltr" class=3D"gmail_attr">On Thu, Jun 5, 2025, 6:46=E2=80=
=AFPM Charlie Li &lt;<a href=3D"mailto:vishwin@freebsd.org">vishwin@freebsd=
.org</a>&gt; wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"mar=
gin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">John Baldwin wr=
ote:<br>
&gt; On 6/5/25 19:29, Charlie Li wrote:<br>
&gt;&gt; &lt;sys/dev/nvme.h&gt; doesn&#39;t appear to be making it in nvme_=
all.h as the<br>
&gt;&gt; build complains about missing nvme_opcode_sbuf() in nvme_all.c.<br=
>
&gt; <br>
&gt; Are you getting a link error or a compile error?=C2=A0 I built this lo=
cally <br>
&gt; several times<br>
&gt; today to test various combinations (kernels with and without either CA=
M <br>
&gt; or nvme and<br>
&gt; loading nvme.ko in kernels without to make sure it worked).=C2=A0 I al=
so <br>
&gt; built GENERIC<br>
&gt; prior to pushing and it built fine.<br>
&gt; <br>
Compile error in buildworld.<br></blockquote></div></div><div dir=3D"auto">=
<br></div><div dir=3D"auto">In libcam?</div><div dir=3D"auto"><br></div><di=
v dir=3D"auto">Warner</div><div dir=3D"auto"><br></div><div dir=3D"auto"><d=
iv class=3D"gmail_quote gmail_quote_container"><blockquote class=3D"gmail_q=
uote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1e=
x">
&gt; Sigh, so from <a href=3D"http://ci.freebsd.org" rel=3D"noreferrer nore=
ferrer" target=3D"_blank">ci.freebsd.org</a>, it appears to be userland tha=
t is choking <br>
&gt; (libcam)<br>
&gt; rather than the kernel.=C2=A0 I&#39;ll fix shortly (mostly it just mea=
ns removing <br>
&gt; the<br>
&gt; #ifdef _KERNEL).<br>
&gt; <br>
<br>
-- <br>
Charlie Li<br>
...nope, still don&#39;t have an exit line.<br>
</blockquote></div></div></div>

--0000000000001520760636dcf401--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfoyYathT2Q=TmVjr7ETvB9-hN1hVXgKgaUY5xxsrpDQLw>