<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 Thu, Jul 25, 2019 at 9:43 PM He Zhe <<a href="mailto:zhe.he@windriver.com">zhe.he@windriver.com</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"><br>
<br>
On 7/25/19 9:04 PM, Bruce Ashfield wrote:<br>
> On Thu, Jul 25, 2019 at 9:03 AM Bruce Ashfield <<a href="mailto:bruce.ashfield@gmail.com" target="_blank">bruce.ashfield@gmail.com</a>> wrote:<br>
>> On Thu, Jul 25, 2019 at 4:54 AM <<a href="mailto:zhe.he@windriver.com" target="_blank">zhe.he@windriver.com</a>> wrote:<br>
>>> From: YueHaibing <<a href="mailto:yuehaibing@huawei.com" target="_blank">yuehaibing@huawei.com</a>><br>
>>><br>
>>> If CONFIG_BTRFS_FS is y and CONFIG_LIBCRC32C is m,<br>
>>> building fails:<br>
>> I've already fixed this by changing the configuration in the kernel-cache. I'll<br>
>> pick this up in linux-yocto-dev once it makes a -rc release.<br>
> i.e. make sure you have this commit from the kernel-cache master:<br>
<br>
You mean you're going to merge it soon, right? I can't find it for the moment.<br>
<a href="http://git.yoctoproject.org/cgit/cgit.cgi/yocto-kernel-cache/log/?h=master" rel="noreferrer" target="_blank">http://git.yoctoproject.org/cgit/cgit.cgi/yocto-kernel-cache/log/?h=master</a></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">It should be there, I pushed things on Wednesday. I'll double check to see what has happened.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Bruce</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>
Zhe<br>
<br>
><br>
> commit 578e0f7fef5a0772460e3c4bcae1200d35a70b21<br>
> Author: Bruce Ashfield <<a href="mailto:bruce.ashfield@gmail.com" target="_blank">bruce.ashfield@gmail.com</a>><br>
> Date:  Â Mon Jul 22 23:41:42 2019 -0400<br>
><br>
>  Â  Â config: set CONFIG_LIBCRC32C=y<br>
><br>
>  Â  Â Since commit d5178578bcd461cc79118c7a139882350fe505aa<br>
><br>
>  Â  Â Author: Johannes Thumshirn <<a href="mailto:jthumshirn@suse.de" target="_blank">jthumshirn@suse.de</a>><br>
>  Â  Â Date:  Â Mon Jun 3 16:58:57 2019 +0200<br>
><br>
>  Â  Â  Â  Â btrfs: directly call into crypto framework for checksumming<br>
><br>
>  Â  Â We now have a dependency on crc32 in crypto, and it must be built<br>
>  Â  Â into the kernel to avoid:<br>
><br>
>  Â  Â  Â  Â | x86_64-poky-linux-ld.bfd: fs/btrfs/super.o: in function<br>
>  Â  Â  Â  Â `btrfs_mount_root':<br>
>  Â  Â  Â  Â | super.c:(.text+0xb9b6): undefined reference to `crc32c_impl'<br>
>  Â  Â  Â  Â | x86_64-poky-linux-ld.bfd: fs/btrfs/super.o: in function<br>
>  Â  Â  Â  Â `init_btrfs_fs':<br>
>  Â  Â  Â  Â | super.c:(.init.text+0x362b): undefined reference to `crc32c_impl'<br>
>  Â  Â  Â  Â | x86_64-poky-linux-ld.bfd: fs/btrfs/extent-tree.o: in function<br>
>  Â  Â  Â  Â `hash_extent_data_ref':<br>
>  Â  Â  Â  Â | extent-tree.c:(.text+0xdfa): undefined reference to `crc32c'<br>
>  Â  Â  Â  Â | x86_64-poky-linux-ld.bfd: extent-tree.c:(.text+0xe13): undefined<br>
>  Â  Â  Â  Â reference to `crc32c'<br>
>  Â  Â  Â  Â | x86_64-poky-linux-ld.bfd: extent-tree.c:(.text+0xe27): undefined<br>
>  Â  Â  Â  Â reference to `crc32c'<br>
>  Â  Â  Â  Â | x86_64-poky-linux-ld.bfd: fs/btrfs/dir-item.o: in function<br>
>  Â  Â  Â  Â `btrfs_insert_xattr_item':<br>
>  Â  Â  Â  Â | dir-item.c:(.text+0x286): undefined reference to `crc32c'<br>
><br>
>  Â  Â So we set our defaults to cover the btrfs build cases without error.<br>
><br>
>  Â  Â Signed-off-by: Bruce Ashfield <<a href="mailto:bruce.ashfield@gmail.com" target="_blank">bruce.ashfield@gmail.com</a>><br>
><br>
><br>
>> Bruce<br>
>><br>
>>>  Â fs/btrfs/super.o: In function `btrfs_mount_root':<br>
>>>  Â super.c:(.text+0xb7f9): undefined reference to `crc32c_impl'<br>
>>>  Â fs/btrfs/super.o: In function `init_btrfs_fs':<br>
>>>  Â super.c:(.init.text+0x3465): undefined reference to `crc32c_impl'<br>
>>>  Â fs/btrfs/extent-tree.o: In function `hash_extent_data_ref':<br>
>>>  Â extent-tree.c:(.text+0xe60): undefined reference to `crc32c'<br>
>>>  Â extent-tree.c:(.text+0xe78): undefined reference to `crc32c'<br>
>>>  Â extent-tree.c:(.text+0xe8b): undefined reference to `crc32c'<br>
>>>  Â fs/btrfs/dir-item.o: In function `btrfs_insert_xattr_item':<br>
>>>  Â dir-item.c:(.text+0x291): undefined reference to `crc32c'<br>
>>>  Â fs/btrfs/dir-item.o: In function `btrfs_insert_dir_item':<br>
>>>  Â dir-item.c:(.text+0x429): undefined reference to `crc32c'<br>
>>><br>
>>> Select LIBCRC32C to fix it.<br>
>>><br>
>>> Reported-by: Hulk Robot <<a href="mailto:hulkci@huawei.com" target="_blank">hulkci@huawei.com</a>><br>
>>> Fixes: d5178578bcd4 ("btrfs: directly call into crypto framework for checksumming")<br>
>>> Reviewed-by: Johannes Thumshirn <<a href="mailto:jthumshirn@suse.de" target="_blank">jthumshirn@suse.de</a>><br>
>>> Signed-off-by: YueHaibing <<a href="mailto:yuehaibing@huawei.com" target="_blank">yuehaibing@huawei.com</a>><br>
>>> Reviewed-by: David Sterba <<a href="mailto:dsterba@suse.com" target="_blank">dsterba@suse.com</a>><br>
>>> Signed-off-by: David Sterba <<a href="mailto:dsterba@suse.com" target="_blank">dsterba@suse.com</a>><br>
>>><br>
>>> commit 314c4cd6d9e60b9412dcd1b1783a66532f91ea2d upstream<br>
>>><br>
>>> Signed-off-by: He Zhe <<a href="mailto:zhe.he@windriver.com" target="_blank">zhe.he@windriver.com</a>><br>
>>> ---<br>
>>>  fs/btrfs/Kconfig | 1 +<br>
>>>  1 file changed, 1 insertion(+)<br>
>>><br>
>>> diff --git a/fs/btrfs/Kconfig b/fs/btrfs/Kconfig<br>
>>> index 212b4a8..38651fa 100644<br>
>>> --- a/fs/btrfs/Kconfig<br>
>>> +++ b/fs/btrfs/Kconfig<br>
>>> @@ -4,6 +4,7 @@ config BTRFS_FS<br>
>>>  Â  Â  Â  Â tristate "Btrfs filesystem support"<br>
>>>  Â  Â  Â  Â select CRYPTO<br>
>>>  Â  Â  Â  Â select CRYPTO_CRC32C<br>
>>> +  Â  Â  Â select LIBCRC32C<br>
>>>  Â  Â  Â  Â select ZLIB_INFLATE<br>
>>>  Â  Â  Â  Â select ZLIB_DEFLATE<br>
>>>  Â  Â  Â  Â select LZO_COMPRESS<br>
>>> --<br>
>>> 2.7.4<br>
>>><br>
>><br>
>> --<br>
>> - Thou shalt not follow the NULL pointer, for chaos and madness await<br>
>> thee at its end<br>
>> - "Use the force Harry" - Gandalf, Star Trek II<br>
><br>
><br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">- Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end<br>- "Use the force Harry" - Gandalf, Star Trek II<br><br></div></div>