[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <9242894f-b831-067a-48d8-2f235963dedb@intel.com>
Date: Mon, 23 Nov 2020 09:13:49 +0800
From: Rong Chen <rong.a.chen@...el.com>
To: Jakub Kicinski <kuba@...nel.org>, kernel test robot <lkp@...el.com>
Cc: Andrea Mayer <andrea.mayer@...roma2.it>,
"David S. Miller" <davem@...emloft.net>,
David Ahern <dsahern@...nel.org>,
Alexey Kuznetsov <kuznet@....inr.ac.ru>,
Hideaki YOSHIFUJI <yoshfuji@...ux-ipv6.org>,
Shuah Khan <skhan@...uxfoundation.org>,
Shrijeet Mukherjee <shrijeet@...il.com>,
Alexei Starovoitov <ast@...nel.org>,
Daniel Borkmann <daniel@...earbox.net>,
kbuild-all@...ts.01.org, clang-built-linux@...glegroups.com,
netdev@...r.kernel.org
Subject: Re: [kbuild-all] Re: [net-next,v2,4/5] seg6: add support for the SRv6
End.DT4 behavior
Hi Jakub,
Sorry for the inconvenience, we have optimized the build bot to avoid
this situation.
Best Regards,
Rong Chen
On 11/14/20 12:57 AM, Jakub Kicinski wrote:
> Good people of build bot,
>
> would you mind shedding some light on this one? It was also reported on
> v1, and Andrea said it's impossible to repro. Strange that build bot
> would make the same mistake twice, tho.
>
> Thanks!
>
> On Fri, 13 Nov 2020 17:23:09 +0800 kernel test robot wrote:
>> Hi Andrea,
>>
>> Thank you for the patch! Yet something to improve:
>>
>> [auto build test ERROR on ipvs/master]
>> [also build test ERROR on linus/master sparc-next/master v5.10-rc3 next-20201112]
>> [If your patch is applied to the wrong git tree, kindly drop us a note.
>> And when submitting patch, we suggest to use '--base' as documented in
>> https://git-scm.com/docs/git-format-patch]
>>
>> url: https://github.com/0day-ci/linux/commits/Andrea-Mayer/seg6-add-support-for-the-SRv6-End-DT4-behavior/20201109-093019
>> base: https://git.kernel.org/pub/scm/linux/kernel/git/horms/ipvs.git master
>> config: x86_64-randconfig-a005-20201111 (attached as .config)
>> compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 874b0a0b9db93f5d3350ffe6b5efda2d908415d0)
>> reproduce (this is a W=1 build):
>> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>> chmod +x ~/bin/make.cross
>> # install x86_64 cross compiling tool for clang build
>> # apt-get install binutils-x86-64-linux-gnu
>> # https://github.com/0day-ci/linux/commit/761138e2f757ac64efe97b03311c976db242dc92
>> git remote add linux-review https://github.com/0day-ci/linux
>> git fetch --no-tags linux-review Andrea-Mayer/seg6-add-support-for-the-SRv6-End-DT4-behavior/20201109-093019
>> git checkout 761138e2f757ac64efe97b03311c976db242dc92
>> # save the attached .config to linux build tree
>> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
>>
>> If you fix the issue, kindly add following tag as appropriate
>> Reported-by: kernel test robot <lkp@...el.com>
>>
>> All errors (new ones prefixed by >>):
>>
>>>> net/ipv6/seg6_local.c:793:4: error: field designator 'slwt_ops' does not refer to any field in type 'struct seg6_action_desc'
>> .slwt_ops = {
>> ^
>>>> net/ipv6/seg6_local.c:826:10: error: invalid application of 'sizeof' to an incomplete type 'struct seg6_action_desc []'
>> count = ARRAY_SIZE(seg6_action_table);
>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/kernel.h:48:32: note: expanded from macro 'ARRAY_SIZE'
>> #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
>> ^~~~~
>> 2 errors generated.
>>
>> vim +793 net/ipv6/seg6_local.c
>>
>> 757
>> 758 static struct seg6_action_desc seg6_action_table[] = {
>> 759 {
>> 760 .action = SEG6_LOCAL_ACTION_END,
>> 761 .attrs = 0,
>> 762 .input = input_action_end,
>> 763 },
>> 764 {
>> 765 .action = SEG6_LOCAL_ACTION_END_X,
>> 766 .attrs = (1 << SEG6_LOCAL_NH6),
>> 767 .input = input_action_end_x,
>> 768 },
>> 769 {
>> 770 .action = SEG6_LOCAL_ACTION_END_T,
>> 771 .attrs = (1 << SEG6_LOCAL_TABLE),
>> 772 .input = input_action_end_t,
>> 773 },
>> 774 {
>> 775 .action = SEG6_LOCAL_ACTION_END_DX2,
>> 776 .attrs = (1 << SEG6_LOCAL_OIF),
>> 777 .input = input_action_end_dx2,
>> 778 },
>> 779 {
>> 780 .action = SEG6_LOCAL_ACTION_END_DX6,
>> 781 .attrs = (1 << SEG6_LOCAL_NH6),
>> 782 .input = input_action_end_dx6,
>> 783 },
>> 784 {
>> 785 .action = SEG6_LOCAL_ACTION_END_DX4,
>> 786 .attrs = (1 << SEG6_LOCAL_NH4),
>> 787 .input = input_action_end_dx4,
>> 788 },
>> 789 {
>> 790 .action = SEG6_LOCAL_ACTION_END_DT4,
>> 791 .attrs = (1 << SEG6_LOCAL_TABLE),
>> 792 .input = input_action_end_dt4,
>> > 793 .slwt_ops = {
>> 794 .build_state = seg6_end_dt4_build,
>> 795 },
>> 796 },
>> 797 {
>> 798 .action = SEG6_LOCAL_ACTION_END_DT6,
>> 799 .attrs = (1 << SEG6_LOCAL_TABLE),
>> 800 .input = input_action_end_dt6,
>> 801 },
>> 802 {
>> 803 .action = SEG6_LOCAL_ACTION_END_B6,
>> 804 .attrs = (1 << SEG6_LOCAL_SRH),
>> 805 .input = input_action_end_b6,
>> 806 },
>> 807 {
>> 808 .action = SEG6_LOCAL_ACTION_END_B6_ENCAP,
>> 809 .attrs = (1 << SEG6_LOCAL_SRH),
>> 810 .input = input_action_end_b6_encap,
>> 811 .static_headroom = sizeof(struct ipv6hdr),
>> 812 },
>> 813 {
>> 814 .action = SEG6_LOCAL_ACTION_END_BPF,
>> 815 .attrs = (1 << SEG6_LOCAL_BPF),
>> 816 .input = input_action_end_bpf,
>> 817 },
>> 818
>> 819 };
>> 820
>> 821 static struct seg6_action_desc *__get_action_desc(int action)
>> 822 {
>> 823 struct seg6_action_desc *desc;
>> 824 int i, count;
>> 825
>> > 826 count = ARRAY_SIZE(seg6_action_table);
>> 827 for (i = 0; i < count; i++) {
>> 828 desc = &seg6_action_table[i];
>> 829 if (desc->action == action)
>> 830 return desc;
>> 831 }
>> 832
>> 833 return NULL;
>> 834 }
>> 835
>>
>> ---
>> 0-DAY CI Kernel Test Service, Intel Corporation
>> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
> _______________________________________________
> kbuild-all mailing list -- kbuild-all@...ts.01.org
> To unsubscribe send an email to kbuild-all-leave@...ts.01.org
Powered by blists - more mailing lists