Re: [PATCH bpf-next v2 0/4] btf_dump fixes for s390

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 13, 2021 at 9:09 AM Ilya Leoshkevich <iii@xxxxxxxxxxxxx> wrote:
>
> v1: https://lore.kernel.org/bpf/20211012023218.399568-1-iii@xxxxxxxxxxxxx/
> v1 -> v2:
> - Remove redundant local variables, use t->size directly instead.
> - Add btf__align_of() patch.
> Pending questions:
> - Can things like defined(__i386__) break cross-compilation?
> - Why exactly do we need both cpu_number and cpu_profile_flip? If we do,
>   is there a suitable replacement for cpu_number in common code?
>
> ---
>
> Hi,
>
> This series along with [1] and [2] fixes all the failures in the
> btf_dump testsuite currently present on s390, in particular:
>
> * [1] fixes intermittent build bug causing "failed to encode tag ..."
>   * error messages.
> * [2] fixes missing VAR entries on s390.
> * Patch 1 disables Intel-specific code in a testcase.
> * Patch 2 fixes an endianness-related bug.
> * Patch 3 fixes an alignment-related bug.
> * Patch 4 improves overly pessimistic alignment handling.
>
> [1] https://lore.kernel.org/bpf/20211012022521.399302-1-iii@xxxxxxxxxxxxx/
> [2] https://lore.kernel.org/bpf/20211012022637.399365-1-iii@xxxxxxxxxxxxx/
>
> Best regards,
> Ilya
>
> Ilya Leoshkevich (4):
>   selftests/bpf: Use cpu_number only on arches that have it
>   libbpf: Fix dumping big-endian bitfields
>   libbpf: Fix dumping non-aligned __int128
>   libbpf: Fix ptr_is_aligned() usages
>
>  tools/lib/bpf/btf_dump.c                      | 34 +++++++++----------
>  .../selftests/bpf/prog_tests/btf_dump.c       |  2 ++
>  2 files changed, 19 insertions(+), 17 deletions(-)
>
> --
> 2.31.1
>

I've dropped patch 4 for now, applied the first three to bpf-next, thanks.



[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux