[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <259dad79-9bed-0b6b-3723-82bf4a0a9603@roeck-us.net>
Date: Thu, 22 Nov 2018 14:30:58 -0800
From: Guenter Roeck <linux@...ck-us.net>
To: Thomas Voegtle <tv@...96.de>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org,
akpm@...ux-foundation.org, shuah@...nel.org, patches@...nelci.org,
ben.hutchings@...ethink.co.uk, lkft-triage@...ts.linaro.org,
stable@...r.kernel.org, Chris Wilson <chris@...is-wilson.co.uk>
Subject: Re: [PATCH 4.19 00/42] 4.19.4-stable review
On 11/22/18 2:01 PM, Thomas Voegtle wrote:
> On Thu, 22 Nov 2018, Thomas Voegtle wrote:
>
>>
>> Doesn't compile for me on OpenSuSE 15.0 (gcc 7.3.1):
>>
>> CALL scripts/checksyscalls.sh
>> DESCEND objtool
>> CHK include/generated/compile.h
>> CC [M] drivers/gpu/drm/i915/i915_gem_gtt.o
>> drivers/gpu/drm/i915/i915_gem_gtt.c: In function ‘gen6_dump_ppgtt’:
>> drivers/gpu/drm/i915/i915_gem_gtt.c:1771:41: error: format ‘%llx’ expects
>> argument of type ‘long long unsigned int’, but argument 5 has type ‘long
>> unsigned int’ [-Werror=format=]
>> seq_printf(m, "\t\t(%03d, %04d) %08llx: ",
>> ~~~~~ ^
>> %08lx
>> cc1: all warnings being treated as errors
>> make[4]: *** [scripts/Makefile.build:306:
>> drivers/gpu/drm/i915/i915_gem_gtt.o] Error 1
>> make[3]: *** [scripts/Makefile.build:546: drivers/gpu/drm/i915] Error 2
>> make[2]: *** [scripts/Makefile.build:546: drivers/gpu/drm] Error 2
>> make[1]: *** [scripts/Makefile.build:546: drivers/gpu] Error 2
>> make: *** [Makefile:1052: drivers] Error 2
>>
>>
>>
>> 4.19.3 is broken for me, too. 4.19.2 is works.
>> 4.20-rc tree works.
>>
>> Config attached.
>
>
> quick bisect shows problem is:
>
> commit 1a25e1a1be71a49ee7f34fb14b5a26191e6cf501
> Author: Chris Wilson <chris@...is-wilson.co.uk>
> Date: Thu Oct 25 10:18:22 2018 +0100
>
> drm/i915: Mark up GTT sizes as u64
>
> commit c58281056a8b26d5d9dc15c19859a7880835ef44 upstream.
>
>
> reverted on 4.19.4-rc1 it compiles for me. I guess something is missing here?
>
AFAICS someone was not careful with the backport. Upstream doesn't have %llx
at that place.
Guenter
Powered by blists - more mailing lists