linux-kernel - Re: [kbuild-all] Re: [RFC PATCH stable] RDMA/qedr: qedr_iw_load

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <c247b253-708e-be32-5bea-dbf22f9c32e9@intel.com>
Date:   Sun, 28 Jun 2020 09:02:37 +0800
From:   "Chen, Rong A" <rong.a.chen@...el.com>
To:     Jason Gunthorpe <jgg@...lanox.com>,
        kernel test robot <lkp@...el.com>
Cc:     Michal Kalderon <michal.kalderon@...vell.com>,
        kbuild-all@...ts.01.org,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Ariel Elior <ariel.elior@...vell.com>,
        Sasha Levin <alexander.levin@...rosoft.com>,
        Doug Ledford <dledford@...hat.com>, linux-rdma@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [kbuild-all] Re: [RFC PATCH stable] RDMA/qedr: qedr_iw_load_qp()
 can be static



On 6/19/2020 8:26 AM, Jason Gunthorpe wrote:
> On Sat, Jun 13, 2020 at 04:19:03AM +0800, kernel test robot wrote:
>> Fixes: 8a69220b659c ("RDMA/qedr: Fix synchronization methods and memory leaks in qedr")
>> Signed-off-by: kernel test robot <lkp@...el.com>
>> ---
>>   qedr_iw_cm.c |    2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
> What is this report??

Hi Jason,

The report was lost and it should be:

tree:https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git  linux-5.4.y
head:   5e3c511539228fa03c8d00d61b5b5f32333ed0b0
commit: 8a69220b659c31ccd481538193220d732814b324 [5532/5583] RDMA/qedr: Fix synchronization methods and memory leaks in qedr
config: ia64-randconfig-s031-20200612 (attached as .config)
compiler: ia64-linux-gcc (GCC) 9.3.0
reproduce:
         # apt-get install sparse
         # sparse version: v0.6.1-250-g42323db3-dirty
         git checkout 8a69220b659c31ccd481538193220d732814b324
         # save the attached .config to linux build tree
         make W=1 C=1 ARCH=ia64 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot<lkp@...el.com>


sparse warnings: (new ones prefixed by >>)

    ./arch/ia64/include/generated/uapi/asm/unistd_64.h:348:39: sparse: sparse: no newline at end of file

>> drivers/infiniband/hw/qedr/qedr_iw_cm.c:509:16: sparse: sparse: symbol 'qedr_iw_load_qp' was not declared. Should it be static?

We'll fix the problem recently.

Best Regards,
Rong Chen

>
> commit 9a5407d74c22821f7944e2be4209bdfc5faf8143
> Author: Kamal Heib <kamalheib1@...il.com>
> Date:   Sun Nov 10 13:36:45 2019 +0200
>
>      RDMA/qedr: Make qedr_iw_load_qp() static
>      
>      The function qedr_iw_load_qp() is only used in qedr_iw_cm.c
>
>
> Jason
> _______________________________________________
> kbuild-all mailing list -- kbuild-all@...ts.01.org
> To unsubscribe send an email to kbuild-all-leave@...ts.01.org