<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 18, 2022 at 5:50 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Li,<br>
<br>
> On Wed, Mar 16, 2022 at 10:11 PM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> wrote:<br>
<br>
> > > Hi Petr<br>
> > > I don't understand why we must serach foo-x86-64 module, so what problem<br>
> > > do you meet?<br>
<br>
> > > I used 5.17-rc8,  it still use foo-x86_64 named rule for<br>
> > > kernel/arch/x86/crypto/libblake2s-x86_64.ko.<br>
<br>
> > > If kernel has libblake2s-x86_64 module, then tst_check_driver will use<br>
> > > libblake2s_x86_64 to find, it should succeed.<br>
<br>
> > > If kernel doesn't have libblake2s-x86_64 module, then tst_ckeck_driver<br>
> > > will search twice ,the first time use libblake2s-x86_64  and the second<br>
> > > time use libblake2s_x86_64, then search failed.<br>
<br>
> > tst_check_driver.sh is failing on intel based systems. Well, we could make<br>
> > sure<br>
> > it does not try to test libblake2s-x86-64, IMHO it'd be better to make sure<br>
> > tst_search_driver() works with it, because modinfo/modprobe works with it:<br>
<br>
> > $ modinfo libblake2s-x86-64<br>
> > name:           libblake2s_x86_64<br>
> > filename:       (builtin)<br>
> > license:        GPL v2<br>
> > file:           arch/x86/crypto/libblake2s-x86_64<br>
<br>
> > Sure, it's a corner case, but I'd still fix it.<br>
> > Let's see what other think.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Ah, I see. Sure, I'm fine to go with your fix :).</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
> Which kernel (and kmod) version did you use?<br>
<br>
> I tried locally on my rhel8 and fedora 34 but that doesn't work for me.<br>
<br>
> $ rpm -q kmod<br>
> kmod-28-2.fc34.x86_64<br>
<br>
> $ lsmod  |grep  libblake2s<br>
> libblake2s             16384  0<br>
> blake2s_x86_64         20480  1 libblake2s<br>
> libblake2s_generic     20480  1 blake2s_x86_64<br>
<br>
> $ modinfo libblake2s-x86-64<br>
> modinfo: ERROR: Module libblake2s-x86-64 not found.<br>
=> 5.16 had blake2s_x86_64, not libblake2s_x86_64<br>
<br>
openSUSE<br>
* 5.17.0-rc7-26.g04b7727-default, kmod-29-8.4.x86_64<br>
works (testing libblake2s-x86-64)<br>
* 5.16, kmod-29-7.1.x86_64<br>
works (testing blake2s_x86-64)<br>
<br>
Fedora 34<br>
* 5.13.16, kmod-29-2.fc34.x86_64<br>
works (testing blake2s_x86-64)<br>
<br>
<br>
> Then I checked the Linux source code and get:<br>
> ------------------<br>
<br>
> $ cat linux/arch/x86/crypto/Makefile |grep libblake2s<br>
> obj-$(if $(CONFIG_CRYPTO_BLAKE2S_X86),y) += libblake2s-x86_64.o<br>
> libblake2s-x86_64-y := blake2s-core.o blake2s-glue.o<br>
6048fdcc5f26 ("lib/crypto: blake2s: include as built-in")<br>
Follows: v5.16-rc8<br>
Precedes: v5.17-rc1<br>
...<br>
+++ b/arch/x86/crypto/Makefile<br>
...<br>
-blake2s-x86_64-y := blake2s-core.o blake2s-glue.o<br>
+blake2s-x86_64-y := blake2s-shash.o<br>
+obj-$(if $(CONFIG_CRYPTO_BLAKE2S_X86),y) += libblake2s-x86_64.o<br>
+libblake2s-x86_64-y := blake2s-core.o blake2s-glue.o<br>
<br>
Kind regards,<br>
Petr<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>