Re: [PATCH v3] gdb/csky support .reg2 for kernel 4.x and later

[Tom de Vries via Gdb-patches <gdb-patches@sourceware.org>]

On 8/9/22 04:29, jiangshuai_li wrote:
>  >On 8/1/22 04:40, Jiangshuai Li via Gdb-patches wrote:
>  >> +      warning (_("Unknow size %ld of section .reg2, can not get value"
>  >> +                 " of float registers."), len);
>  >
>  >This broke the build on the gdb-armhf-ubuntu20_04 builder ( see
>  >https://builder.sourceware.org/buildbot/#/builders/170/builds/246 ):
>  >...
>  >../../binutils-gdb/gdb/csky-linux-tdep.c: In function ‘void
>  >csky_supply_fregset(const regset*, regcache*, int, const void*, size_t)’:
>  >../../binutils-gdb/gdb/csky-linux-tdep.c:194:18: error: format ‘%ld’
>  >expects argument of type ‘long int’, but argument 2 has type ‘size_t’
>  >{aka ‘unsigned int’} [-Werror=format=]
>   >  194 |       warning (_("Unknow size %ld of section .reg2, can not get
>  >value"
>  >       |
>  >^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>  >   195 |    " of float registers."), len);
>  >...
>  >
>  >There are uses of %z in gdb, but very few, so I suppose the way to fix
>  >this is to use %s and pulongest.
> 
> Thanks, i have used your suggestion to fix it, a patch has been committed.
> 

Thanks for fixing it, and I see the gdb-armhf-ubuntu20_04 builder is 
back to green status, yay !

Thanks,
- Tom