linux-kernel - Re: linux-next: build warning after merge of the libata tree

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <8e0717c2-d3c0-c81c-3a33-afdc045d123c@omp.ru>
Date:   Mon, 20 Jun 2022 13:57:53 +0300
From:   Sergey Shtylyov <s.shtylyov@....ru>
To:     Damien Le Moal <damien.lemoal@...nsource.wdc.com>,
        Stephen Rothwell <sfr@...b.auug.org.au>
CC:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux Next Mailing List <linux-next@...r.kernel.org>
Subject: Re: linux-next: build warning after merge of the libata tree

Hello!

On 6/20/22 11:53 AM, Damien Le Moal wrote:

[...]
>>> After merging the libata tree, today's linux-next build (powercp
>>> ppc64_defconfig) produced this warning:
>>>
>>> In file included from include/linux/device.h:15,
>>>                  from arch/powerpc/include/asm/io.h:27,
>>>                  from include/linux/io.h:13,
>>>                  from include/linux/irq.h:20,
>>>                  from arch/powerpc/include/asm/hardirq.h:6,
>>>                  from include/linux/hardirq.h:11,
>>>                  from include/linux/highmem.h:11,
>>>                  from include/linux/bvec.h:10,
>>>                  from include/linux/blk_types.h:10,
>>>                  from include/linux/blkdev.h:9,
>>>                  from drivers/ata/pata_macio.c:18:
>>> drivers/ata/pata_macio.c: In function 'pmac_macio_calc_timing_masks':
>>> drivers/ata/pata_macio.c:1031:28: warning: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'unsigned int' [-Wformat=]
>>>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>>
>> 	.
>> 	.
>> 	.
>>
>>> Introduced by commit
>>>
>>>   f0a6d77b351c ("ata: make transfer mode masks *unsigned int*")

   Oops, sorry about that!

>> This turned into a build failure in my powerpc allyesconfig build, so I
>> reverted the above commit for today.
> 
> I pushed the fix already. It is in ata for-next branch.

   Thank you!
   Note that your Fixes: tag doesn't look right -- the asterisks are missing
in the summary...

MBR, Sergey