Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 24 Jun 2024 16:07:56 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        Gleb Smirnoff <glebius@freebsd.org>
Cc:        Justin Hibbits <jhibbits@freebsd.org>, Warner Losh <imp@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: c2e9c5bbf022 - main - tpm: Refactor TIS and add a SPI attachment
Message-ID:  <CANCZdfqrX9FtdDmZScWn6MNwwD0WRxjzJo9Z93Wcn5qM_rLfug@mail.gmail.com>
In-Reply-To: <ZnniHBI7bg--g8pV@cell.glebi.us>
References:  <202405032032.443KWhf0045034@gitrepo.freebsd.org> <ZnnMBXi2sGdtc_67@cell.glebi.us> <20240624161128.33c0f518@ralga.knownspace> <ZnniHBI7bg--g8pV@cell.glebi.us>

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

On Mon, Jun 24, 2024, 3:16=E2=80=AFPM Gleb Smirnoff <glebius@freebsd.org> w=
rote:

> On Mon, Jun 24, 2024 at 04:11:28PM -0400, Justin Hibbits wrote:
> J> > sorry for late notice, but we just discovered that after this commit
> J> > tpm(4) on ACPI no longer attaches.  From what I see ther probe metho=
d
> J> > isn't even called.
> J>
> J> Hi Gleb,
> J>
> J> That's very odd.  The probe method hasn't changed, nor has the tpm_acp=
i
> J> attachment (still acpi), so I'm really confused why it wouldn't be
> J> called now.  Is there any debugging you can do to further narrow it
> J> down?  I haven't seen any issues reported internally at Juniper on x86
> J> hardware.  I just verified everything should get built properly.
>
> I started adding printfs to the probe method and as said - it is not
> even called. So I puzzled on where to start the debugging. I can test
> patches/ideas if you have any.
>
> Warner is also looking into this problem.
>

I'm looking at name mismatched and/or moved parents to see if that's in
play. I thought I'd found something like this earlier, but it was a false
alarm I think.

Warner

>

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

<div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" =
class=3D"gmail_attr">On Mon, Jun 24, 2024, 3:16=E2=80=AFPM Gleb Smirnoff &l=
t;<a href=3D"mailto:glebius@freebsd.org">glebius@freebsd.org</a>&gt; wrote:=
<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;bord=
er-left:1px #ccc solid;padding-left:1ex">On Mon, Jun 24, 2024 at 04:11:28PM=
 -0400, Justin Hibbits wrote:<br>
J&gt; &gt; sorry for late notice, but we just discovered that after this co=
mmit<br>
J&gt; &gt; tpm(4) on ACPI no longer attaches.=C2=A0 From what I see ther pr=
obe method<br>
J&gt; &gt; isn&#39;t even called.<br>
J&gt; <br>
J&gt; Hi Gleb,<br>
J&gt; <br>
J&gt; That&#39;s very odd.=C2=A0 The probe method hasn&#39;t changed, nor h=
as the tpm_acpi<br>
J&gt; attachment (still acpi), so I&#39;m really confused why it wouldn&#39=
;t be<br>
J&gt; called now.=C2=A0 Is there any debugging you can do to further narrow=
 it<br>
J&gt; down?=C2=A0 I haven&#39;t seen any issues reported internally at Juni=
per on x86<br>
J&gt; hardware.=C2=A0 I just verified everything should get built properly.=
<br>
<br>
I started adding printfs to the probe method and as said - it is not<br>
even called. So I puzzled on where to start the debugging. I can test<br>
patches/ideas if you have any.<br>
<br>
Warner is also looking into this problem.<br></blockquote></div></div><div =
dir=3D"auto"><br></div><div dir=3D"auto">I&#39;m looking at name mismatched=
 and/or moved parents to see if that&#39;s in play. I thought I&#39;d found=
 something like this earlier, but it was a false alarm I think.</div><div d=
ir=3D"auto"><br></div><div dir=3D"auto">Warner</div><div dir=3D"auto"><div =
class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=3D"margin:0 0=
 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></d=
iv><div dir=3D"auto"><div class=3D"gmail_quote"><blockquote class=3D"gmail_=
quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1=
ex">
</blockquote></div></div></div>

--0000000000004ae26a061baa041a--



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