[PATCH libdrm v2] amdgpu: Use PRI?64 to format uint64_t

[Christian König]

Am 21.08.23 um 11:14 schrieb Geert Uytterhoeven:
> Hi Christian,
>
> On Fri, Jul 7, 2023 at 9:36 PM Geert Uytterhoeven <geert at linux-m68k.org> wrote:
>> On Fri, Jul 7, 2023 at 2:06 PM Christian König
>> <ckoenig.leichtzumerken at gmail.com> wrote:
>>> Am 06.07.23 um 10:36 schrieb Geert Uytterhoeven:
>>>> On 32-bit:
>>>>
>>>>       ../tests/amdgpu/amdgpu_stress.c: In function ‘alloc_bo’:
>>>>       ../tests/amdgpu/amdgpu_stress.c:178:49: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘uint64_t’ {aka ‘long long unsigned int’} [-Wformat=]
>>>>         fprintf(stdout, "Allocated BO number %u at 0x%lx, domain 0x%x, size %lu\n",
>>>>                                                      ~~^
>>>>                                                      %llx
>>>>          num_buffers++, addr, domain, size);
>>>>                         ~~~~
>> [...]
>>
>>>> Fix this by using the proper "PRI?64" format specifiers.
>>>>
>>>> Fixes: d77ccdf3ba6f5a39 ("amdgpu: add amdgpu_stress utility v2")
>>>> Signed-off-by: Geert Uytterhoeven <geert+renesas at glider.be>
>>> Well generally good patch, but libdrm changes are now reviewed by merge
>>> request and not on the mailing list any more.
>> I heard such a rumor, too ;-)
>>
>> Unfortunately one year later, that process is still not documented in
>> https://gitlab.freedesktop.org/mesa/drm/-/blob/main/CONTRIBUTING.rst
>> which still instructs me (a casual drive-by developer) to just submit
>> my patches to the mailing list...
> So a few weeks ago I created gitlab PRs for all my pending libdrm
> patch series, and I rebased them regularly when needed.
> What needs to be done to get them merged?

You need to ping the userspace maintainers for this. Like Marek, 
Pierre-Eric etc..

Regards,
Christian.

>
> Thanks!
>
> Gr{oetje,eeting}s,
>
>                          Geert
>