提交 4cf677f0 编写于 作者: A Alan Maguire 提交者: Zheng Zengkai

libbpf: Avoid use of __int128 in typed dump display

mainline inclusion
from mainline-5.15-rc1
commit a1d3cc3c
category: feature
bugzilla: https://gitee.com/openeuler/kernel/issues/I5EUVD
CVE: NA

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a1d3cc3c5eca598cfabee3a35f30f34fbe2f709b

-------------------------------------------------

__int128 is not supported for some 32-bit platforms (arm and i386).
__int128 was used in carrying out computations on bitfields which
aid display, but the same calculations could be done with __u64
with the small effect of not supporting 128-bit bitfields.

With these changes, a big-endian issue with casting 128-bit integers
to 64-bit for enum bitfields is solved also, as we now use 64-bit
integers for bitfield calculations.
Reported-by: NNaresh Kamboju <naresh.kamboju@linaro.org>
Reported-by: NLinux Kernel Functional Testing <lkft@linaro.org>
Signed-off-by: NAlan Maguire <alan.maguire@oracle.com>
Signed-off-by: NAndrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/bpf/1626770993-11073-2-git-send-email-alan.maguire@oracle.com
(cherry picked from commit a1d3cc3c)
Signed-off-by: NWang Yufen <wangyufen@huawei.com>
上级 ff8961a5
...@@ -1557,31 +1557,26 @@ static int btf_dump_unsupported_data(struct btf_dump *d, ...@@ -1557,31 +1557,26 @@ static int btf_dump_unsupported_data(struct btf_dump *d,
return -ENOTSUP; return -ENOTSUP;
} }
static void btf_dump_int128(struct btf_dump *d, static int btf_dump_get_bitfield_value(struct btf_dump *d,
const struct btf_type *t,
const void *data)
{
__int128 num = *(__int128 *)data;
if ((num >> 64) == 0)
btf_dump_type_values(d, "0x%llx", (long long)num);
else
btf_dump_type_values(d, "0x%llx%016llx", (long long)num >> 32,
(long long)num);
}
static unsigned __int128 btf_dump_bitfield_get_data(struct btf_dump *d,
const struct btf_type *t, const struct btf_type *t,
const void *data, const void *data,
__u8 bits_offset, __u8 bits_offset,
__u8 bit_sz) __u8 bit_sz,
__u64 *value)
{ {
__u16 left_shift_bits, right_shift_bits; __u16 left_shift_bits, right_shift_bits;
__u8 nr_copy_bits, nr_copy_bytes; __u8 nr_copy_bits, nr_copy_bytes;
unsigned __int128 num = 0, ret;
const __u8 *bytes = data; const __u8 *bytes = data;
int sz = t->size;
__u64 num = 0;
int i; int i;
/* Maximum supported bitfield size is 64 bits */
if (sz > 8) {
pr_warn("unexpected bitfield size %d\n", sz);
return -EINVAL;
}
/* Bitfield value retrieval is done in two steps; first relevant bytes are /* Bitfield value retrieval is done in two steps; first relevant bytes are
* stored in num, then we left/right shift num to eliminate irrelevant bits. * stored in num, then we left/right shift num to eliminate irrelevant bits.
*/ */
...@@ -1596,12 +1591,12 @@ static unsigned __int128 btf_dump_bitfield_get_data(struct btf_dump *d, ...@@ -1596,12 +1591,12 @@ static unsigned __int128 btf_dump_bitfield_get_data(struct btf_dump *d,
#else #else
# error "Unrecognized __BYTE_ORDER__" # error "Unrecognized __BYTE_ORDER__"
#endif #endif
left_shift_bits = 128 - nr_copy_bits; left_shift_bits = 64 - nr_copy_bits;
right_shift_bits = 128 - bit_sz; right_shift_bits = 64 - bit_sz;
ret = (num << left_shift_bits) >> right_shift_bits; *value = (num << left_shift_bits) >> right_shift_bits;
return ret; return 0;
} }
static int btf_dump_bitfield_check_zero(struct btf_dump *d, static int btf_dump_bitfield_check_zero(struct btf_dump *d,
...@@ -1610,9 +1605,12 @@ static int btf_dump_bitfield_check_zero(struct btf_dump *d, ...@@ -1610,9 +1605,12 @@ static int btf_dump_bitfield_check_zero(struct btf_dump *d,
__u8 bits_offset, __u8 bits_offset,
__u8 bit_sz) __u8 bit_sz)
{ {
__int128 check_num; __u64 check_num;
int err;
check_num = btf_dump_bitfield_get_data(d, t, data, bits_offset, bit_sz); err = btf_dump_get_bitfield_value(d, t, data, bits_offset, bit_sz, &check_num);
if (err)
return err;
if (check_num == 0) if (check_num == 0)
return -ENODATA; return -ENODATA;
return 0; return 0;
...@@ -1624,10 +1622,14 @@ static int btf_dump_bitfield_data(struct btf_dump *d, ...@@ -1624,10 +1622,14 @@ static int btf_dump_bitfield_data(struct btf_dump *d,
__u8 bits_offset, __u8 bits_offset,
__u8 bit_sz) __u8 bit_sz)
{ {
unsigned __int128 print_num; __u64 print_num;
int err;
print_num = btf_dump_bitfield_get_data(d, t, data, bits_offset, bit_sz); err = btf_dump_get_bitfield_value(d, t, data, bits_offset, bit_sz, &print_num);
btf_dump_int128(d, t, &print_num); if (err)
return err;
btf_dump_type_values(d, "0x%llx", (unsigned long long)print_num);
return 0; return 0;
} }
...@@ -1686,9 +1688,29 @@ static int btf_dump_int_data(struct btf_dump *d, ...@@ -1686,9 +1688,29 @@ static int btf_dump_int_data(struct btf_dump *d,
return btf_dump_bitfield_data(d, t, data, 0, 0); return btf_dump_bitfield_data(d, t, data, 0, 0);
switch (sz) { switch (sz) {
case 16: case 16: {
btf_dump_int128(d, t, data); const __u64 *ints = data;
__u64 lsi, msi;
/* avoid use of __int128 as some 32-bit platforms do not
* support it.
*/
#if __BYTE_ORDER == __LITTLE_ENDIAN
lsi = ints[0];
msi = ints[1];
#elif __BYTE_ORDER == __BIG_ENDIAN
lsi = ints[1];
msi = ints[0];
#else
# error "Unrecognized __BYTE_ORDER__"
#endif
if (msi == 0)
btf_dump_type_values(d, "0x%llx", (unsigned long long)lsi);
else
btf_dump_type_values(d, "0x%llx%016llx", (unsigned long long)msi,
(unsigned long long)lsi);
break; break;
}
case 8: case 8:
if (sign) if (sign)
btf_dump_type_values(d, "%lld", *(long long *)data); btf_dump_type_values(d, "%lld", *(long long *)data);
...@@ -1936,9 +1958,16 @@ static int btf_dump_get_enum_value(struct btf_dump *d, ...@@ -1936,9 +1958,16 @@ static int btf_dump_get_enum_value(struct btf_dump *d,
/* handle unaligned enum value */ /* handle unaligned enum value */
if (!ptr_is_aligned(data, sz)) { if (!ptr_is_aligned(data, sz)) {
*value = (__s64)btf_dump_bitfield_get_data(d, t, data, 0, 0); __u64 val;
int err;
err = btf_dump_get_bitfield_value(d, t, data, 0, 0, &val);
if (err)
return err;
*value = (__s64)val;
return 0; return 0;
} }
switch (t->size) { switch (t->size) {
case 8: case 8:
*value = *(__s64 *)data; *value = *(__s64 *)data;
...@@ -2214,10 +2243,13 @@ static int btf_dump_dump_type_data(struct btf_dump *d, ...@@ -2214,10 +2243,13 @@ static int btf_dump_dump_type_data(struct btf_dump *d,
case BTF_KIND_ENUM: case BTF_KIND_ENUM:
/* handle bitfield and int enum values */ /* handle bitfield and int enum values */
if (bit_sz) { if (bit_sz) {
unsigned __int128 print_num; __u64 print_num;
__s64 enum_val; __s64 enum_val;
print_num = btf_dump_bitfield_get_data(d, t, data, bits_offset, bit_sz); err = btf_dump_get_bitfield_value(d, t, data, bits_offset, bit_sz,
&print_num);
if (err)
break;
enum_val = (__s64)print_num; enum_val = (__s64)print_num;
err = btf_dump_enum_data(d, t, id, &enum_val); err = btf_dump_enum_data(d, t, id, &enum_val);
} else } else
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册