Re: [linux-next:master 13060/13846] arch/powerpc/boot/decompress.c:132:8: error: call to undeclared function '__decompress'; ISO C99 and later do not support implicit function declarations
Michael Ellerman <[email protected]>
| Newsgroups | dev.linux.lists.oe-kbuild-all |
|---|---|
| Message-ID | <[email protected]> |
On 12/8/26 7:15 am, Bjorn Helgaas wrote:
> [+to Madhavan, Michael; seems unlikely that 79752653c136 ("PCI:
> cadence: Fix host/endpoint dependencies for cadence-plat driver")
> would cause this]
Yeah I agree, it's an existing bug that this randconfig build has just tripped over.
cheers
>
> On Tue, Aug 11, 2026 at 07:40:11PM +0800, kernel test robot wrote:
>> Hi Aksh,
>>
>> First bad commit (maybe != root cause):
>>
>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
>> head: 6b8c8af514d739d0335f5579b585e02babe8a727
>> commit: 79752653c136493ae8c3b4260f9f80de0b91d0bf [13060/13846] PCI: cadence: Fix host/endpoint dependencies for cadence-plat driver
>> config: powerpc-randconfig-r133-20260811 (https://download.01.org/0day-ci/archive/20260811/[email protected]/config)
>> compiler: clang version 19.1.7 (https://github.com/llvm/llvm-project cd708029e0b2869e80abe31ddb175f7c35361f90)
>> sparse: v0.6.5-rc1
>> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260811/[email protected]/reproduce)
>>
>> If you fix the issue in a separate patch/commit (i.e. not just a new version of
>> the same patch/commit), kindly add following tags
>> | Reported-by: kernel test robot <[email protected]>
>> | Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
>>
>> All errors (new ones prefixed by >>):
>>
>>>> arch/powerpc/boot/decompress.c:132:8: error: call to undeclared function '__decompress'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
>> 132 | ret = __decompress(inbuf, input_size, NULL, flush, outbuf,
>> | ^
>> 1 error generated.
>>
>>
>> vim +/__decompress +132 arch/powerpc/boot/decompress.c
>>
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 97
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 98 /**
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 99 * partial_decompress - decompresses part or all of a compressed buffer
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 100 * @inbuf: input buffer
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 101 * @input_size: length of the input buffer
>> 930a77c3ad79c3 Zhang Jianhua 2021-05-10 102 * @outbuf: output buffer
>> 930a77c3ad79c3 Zhang Jianhua 2021-05-10 103 * @output_size: length of the output buffer
>> 6efc2f1a64ef62 Yang Li 2024-04-08 104 * @_skip: number of output bytes to ignore
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 105 *
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 106 * This function takes compressed data from inbuf, decompresses and write it to
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 107 * outbuf. Once output_size bytes are written to the output buffer, or the
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 108 * stream is exhausted the function will return the number of bytes that were
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 109 * decompressed. Otherwise it will return whatever error code the decompressor
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 110 * reported (NB: This is specific to each decompressor type).
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 111 *
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 112 * The skip functionality is mainly there so the program and discover
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 113 * the size of the compressed image so that it can ask firmware (if present)
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 114 * for an appropriately sized buffer.
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 115 */
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 116 long partial_decompress(void *inbuf, unsigned long input_size,
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 117 void *outbuf, unsigned long output_size, unsigned long _skip)
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 118 {
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 119 int ret;
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 120
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 121 /*
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 122 * The skipped bytes needs to be included in the size of data we want
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 123 * to decompress.
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 124 */
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 125 output_size += _skip;
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 126
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 127 decompressed_bytes = 0;
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 128 output_buffer = outbuf;
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 129 limit = output_size;
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 130 skip = _skip;
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 131
>> 1b7898ee276b39 Oliver O'Halloran 2016-09-22 @132 ret = __decompress(inbuf, input_size, NULL, flush, outbuf,
>>
>> :::::: The code at line 132 was first introduced by commit
>> :::::: 1b7898ee276b39e54d870dc4ef3374f663d0b426 powerpc/boot: Use the pre-boot decompression API
>>
>> :::::: TO: Oliver O'Halloran <[email protected]>
>> :::::: CC: Michael Ellerman <[email protected]>
>>
>> --
>> 0-DAY CI Kernel Test Service
>> https://github.com/intel/lkp-tests/wiki