Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1676395 > unrolled thread

linux-next: build warnings after merge of the scsi-mkp tree

Started byStephen Rothwell <sfr@canb.auug.org.au>
First post2017-06-28 08:00 +0200
Last post2017-06-29 20:40 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

Contents

  linux-next: build warnings after merge of the scsi-mkp tree Stephen Rothwell <sfr@canb.auug.org.au> - 2017-06-28 08:00 +0200
    Re: linux-next: build warnings after merge of the scsi-mkp tree Stephen Rothwell <sfr@canb.auug.org.au> - 2017-06-29 07:10 +0200
      Re: linux-next: build warnings after merge of the scsi-mkp tree "Madhani, Himanshu" <Himanshu.Madhani@cavium.com> - 2017-06-29 20:40 +0200

#1676395 — linux-next: build warnings after merge of the scsi-mkp tree

FromStephen Rothwell <sfr@canb.auug.org.au>
Date2017-06-28 08:00 +0200
Subjectlinux-next: build warnings after merge of the scsi-mkp tree
Message-ID<tXeb7-2PV-1@gated-at.bofh.it>
Hi Martin,

After merging the scsi-mkp tree, today's linux-next build
(powerpc_ppc64_defconfig) produced these warnings:

In file included from include/linux/byteorder/big_endian.h:4:0,
                 from arch/powerpc/include/uapi/asm/byteorder.h:13,
                 from include/asm-generic/bitops/le.h:5,
                 from arch/powerpc/include/asm/bitops.h:246,
                 from include/linux/bitops.h:36,
                 from include/linux/kernel.h:10,
                 from include/asm-generic/bug.h:15,
                 from arch/powerpc/include/asm/bug.h:127,
                 from include/linux/bug.h:4,
                 from arch/powerpc/include/asm/mmu.h:125,
                 from arch/powerpc/include/asm/lppaca.h:36,
                 from arch/powerpc/include/asm/paca.h:21,
                 from arch/powerpc/include/asm/current.h:16,
                 from include/linux/sched.h:11,
                 from include/linux/blkdev.h:4,
                 from include/linux/blk-mq.h:4,
                 from drivers/scsi/qla2xxx/qla_nvme.h:10,
                 from drivers/scsi/qla2xxx/qla_nvme.c:7:
drivers/scsi/qla2xxx/qla_nvme.c: In function 'qla2x00_start_nvme_mq':
include/uapi/linux/byteorder/big_endian.h:32:26: warning: large integer implicitly truncated to unsigned type [-Woverflow]
 #define __cpu_to_le32(x) ((__force __le32)__swab32((x)))
                          ^
include/linux/byteorder/generic.h:87:21: note: in expansion of macro '__cpu_to_le32'
 #define cpu_to_le32 __cpu_to_le32
                     ^
drivers/scsi/qla2xxx/qla_nvme.c:444:27: note: in expansion of macro 'cpu_to_le32'
    cont_pkt->entry_type = cpu_to_le32(CONTINUE_A64_TYPE);
                           ^
drivers/scsi/qla2xxx/qla_nvme.c: At top level:
drivers/scsi/qla2xxx/qla_nvme.c:667:13: warning: 'qla_nvme_unregister_remote_port' defined but not used [-Wunused-function]
 static void qla_nvme_unregister_remote_port(struct work_struct *work)
             ^
drivers/scsi/qla2xxx/qla_nvme.c:604:12: warning: 'qla_nvme_wait_on_rport_del' defined but not used [-Wunused-function]
 static int qla_nvme_wait_on_rport_del(fc_port_t *fcport)
            ^
drivers/scsi/qla2xxx/qla_nvme.c:634:13: warning: 'qla_nvme_abort_all' defined but not used [-Wunused-function]
 static void qla_nvme_abort_all(fc_port_t *fcport)
             ^

Introduced by commit

  e84067d74301 ("scsi: qla2xxx: Add FC-NVMe F/W initialization and transport registration")

-- 
Cheers,
Stephen Rothwell

[toc] | [next] | [standalone]

#1677387

FromStephen Rothwell <sfr@canb.auug.org.au>
Date2017-06-29 07:10 +0200
Message-ID<tXzSi-3ni-9@gated-at.bofh.it>
In reply to#1676395
Hi James,

This has now migrated to the scsi tree.

On Wed, 28 Jun 2017 15:55:10 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> After merging the scsi-mkp tree, today's linux-next build
> (powerpc_ppc64_defconfig) produced these warnings:
> 
> In file included from include/linux/byteorder/big_endian.h:4:0,
>                  from arch/powerpc/include/uapi/asm/byteorder.h:13,
>                  from include/asm-generic/bitops/le.h:5,
>                  from arch/powerpc/include/asm/bitops.h:246,
>                  from include/linux/bitops.h:36,
>                  from include/linux/kernel.h:10,
>                  from include/asm-generic/bug.h:15,
>                  from arch/powerpc/include/asm/bug.h:127,
>                  from include/linux/bug.h:4,
>                  from arch/powerpc/include/asm/mmu.h:125,
>                  from arch/powerpc/include/asm/lppaca.h:36,
>                  from arch/powerpc/include/asm/paca.h:21,
>                  from arch/powerpc/include/asm/current.h:16,
>                  from include/linux/sched.h:11,
>                  from include/linux/blkdev.h:4,
>                  from include/linux/blk-mq.h:4,
>                  from drivers/scsi/qla2xxx/qla_nvme.h:10,
>                  from drivers/scsi/qla2xxx/qla_nvme.c:7:
> drivers/scsi/qla2xxx/qla_nvme.c: In function 'qla2x00_start_nvme_mq':
> include/uapi/linux/byteorder/big_endian.h:32:26: warning: large integer implicitly truncated to unsigned type [-Woverflow]
>  #define __cpu_to_le32(x) ((__force __le32)__swab32((x)))
>                           ^
> include/linux/byteorder/generic.h:87:21: note: in expansion of macro '__cpu_to_le32'
>  #define cpu_to_le32 __cpu_to_le32
>                      ^
> drivers/scsi/qla2xxx/qla_nvme.c:444:27: note: in expansion of macro 'cpu_to_le32'
>     cont_pkt->entry_type = cpu_to_le32(CONTINUE_A64_TYPE);
>                            ^
> drivers/scsi/qla2xxx/qla_nvme.c: At top level:
> drivers/scsi/qla2xxx/qla_nvme.c:667:13: warning: 'qla_nvme_unregister_remote_port' defined but not used [-Wunused-function]
>  static void qla_nvme_unregister_remote_port(struct work_struct *work)
>              ^
> drivers/scsi/qla2xxx/qla_nvme.c:604:12: warning: 'qla_nvme_wait_on_rport_del' defined but not used [-Wunused-function]
>  static int qla_nvme_wait_on_rport_del(fc_port_t *fcport)
>             ^
> drivers/scsi/qla2xxx/qla_nvme.c:634:13: warning: 'qla_nvme_abort_all' defined but not used [-Wunused-function]
>  static void qla_nvme_abort_all(fc_port_t *fcport)
>              ^
> 
> Introduced by commit
> 
>   e84067d74301 ("scsi: qla2xxx: Add FC-NVMe F/W initialization and transport registration")

-- 
Cheers,
Stephen Rothwell

[toc] | [prev] | [next] | [standalone]

#1678058

From"Madhani, Himanshu" <Himanshu.Madhani@cavium.com>
Date2017-06-29 20:40 +0200
Message-ID<tXMwb-2FO-33@gated-at.bofh.it>
In reply to#1677387
Hi Stephen, James,

> On Jun 28, 2017, at 10:07 PM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> 
> Hi James,
> 
> This has now migrated to the scsi tree.
> 
> On Wed, 28 Jun 2017 15:55:10 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>> 
>> After merging the scsi-mkp tree, today's linux-next build
>> (powerpc_ppc64_defconfig) produced these warnings:
>> 
>> In file included from include/linux/byteorder/big_endian.h:4:0,
>>                 from arch/powerpc/include/uapi/asm/byteorder.h:13,
>>                 from include/asm-generic/bitops/le.h:5,
>>                 from arch/powerpc/include/asm/bitops.h:246,
>>                 from include/linux/bitops.h:36,
>>                 from include/linux/kernel.h:10,
>>                 from include/asm-generic/bug.h:15,
>>                 from arch/powerpc/include/asm/bug.h:127,
>>                 from include/linux/bug.h:4,
>>                 from arch/powerpc/include/asm/mmu.h:125,
>>                 from arch/powerpc/include/asm/lppaca.h:36,
>>                 from arch/powerpc/include/asm/paca.h:21,
>>                 from arch/powerpc/include/asm/current.h:16,
>>                 from include/linux/sched.h:11,
>>                 from include/linux/blkdev.h:4,
>>                 from include/linux/blk-mq.h:4,
>>                 from drivers/scsi/qla2xxx/qla_nvme.h:10,
>>                 from drivers/scsi/qla2xxx/qla_nvme.c:7:
>> drivers/scsi/qla2xxx/qla_nvme.c: In function 'qla2x00_start_nvme_mq':
>> include/uapi/linux/byteorder/big_endian.h:32:26: warning: large integer implicitly truncated to unsigned type [-Woverflow]
>> #define __cpu_to_le32(x) ((__force __le32)__swab32((x)))
>>                          ^
>> include/linux/byteorder/generic.h:87:21: note: in expansion of macro '__cpu_to_le32'
>> #define cpu_to_le32 __cpu_to_le32
>>                     ^
>> drivers/scsi/qla2xxx/qla_nvme.c:444:27: note: in expansion of macro 'cpu_to_le32'
>>    cont_pkt->entry_type = cpu_to_le32(CONTINUE_A64_TYPE);
>>                           ^
>> drivers/scsi/qla2xxx/qla_nvme.c: At top level:
>> drivers/scsi/qla2xxx/qla_nvme.c:667:13: warning: 'qla_nvme_unregister_remote_port' defined but not used [-Wunused-function]
>> static void qla_nvme_unregister_remote_port(struct work_struct *work)
>>             ^
>> drivers/scsi/qla2xxx/qla_nvme.c:604:12: warning: 'qla_nvme_wait_on_rport_del' defined but not used [-Wunused-function]
>> static int qla_nvme_wait_on_rport_del(fc_port_t *fcport)
>>            ^
>> drivers/scsi/qla2xxx/qla_nvme.c:634:13: warning: 'qla_nvme_abort_all' defined but not used [-Wunused-function]
>> static void qla_nvme_abort_all(fc_port_t *fcport)
>>             ^
>> 
>> Introduced by commit
>> 
>>  e84067d74301 ("scsi: qla2xxx: Add FC-NVMe F/W initialization and transport registration")
> 
> -- 
> Cheers,
> Stephen Rothwell


Would it be okay if we send patch to address this warnings in couple days? 

Thanks,
- Himanshu

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web