On Tue, Dec 7, 2021 at 11:36 AM Alexander Kanavin <alex.kana...@gmail.com> wrote: > > No more 'Pending' please :) mesa has a gitlab, so it's trivial to submit the > patch.
Good point, I have it already under review, I should have updated the status here, v2 has it updated. > > Alex > > On Tue, 7 Dec 2021 at 20:25, Khem Raj <raj.k...@gmail.com> wrote: >> >> Signed-off-by: Khem Raj <raj.k...@gmail.com> >> --- >> ...unt-for-64bit-time_t-on-32bit-arches.patch | 41 +++++++++++++++++++ >> meta/recipes-graphics/mesa/mesa.inc | 1 + >> 2 files changed, 42 insertions(+) >> create mode 100644 >> meta/recipes-graphics/mesa/files/0001-v3dv-account-for-64bit-time_t-on-32bit-arches.patch >> >> diff --git >> a/meta/recipes-graphics/mesa/files/0001-v3dv-account-for-64bit-time_t-on-32bit-arches.patch >> >> b/meta/recipes-graphics/mesa/files/0001-v3dv-account-for-64bit-time_t-on-32bit-arches.patch >> new file mode 100644 >> index 00000000000..9032eaba0f1 >> --- /dev/null >> +++ >> b/meta/recipes-graphics/mesa/files/0001-v3dv-account-for-64bit-time_t-on-32bit-arches.patch >> @@ -0,0 +1,41 @@ >> +From e2e2fe86dd61b6fae682a1fec31bd0755ced5cd5 Mon Sep 17 00:00:00 2001 >> +From: Khem Raj <raj.k...@gmail.com> >> +Date: Tue, 7 Dec 2021 11:13:27 -0800 >> +Subject: [PATCH] v3dv: account for 64bit time_t on 32bit arches >> + >> +This makes is a bit more portable, especially on 32bit architectures >> +with 64bit time_t defaults. Especially on musl its a must. >> + >> +Fixes >> +../mesa-21.3.0/src/broadcom/vulkan/v3dv_bo.c:71:15: error: format specifies >> type 'long' but the argument has type 'time_t' (aka 'long long') >> [-Werror,-Wformat] >> + time.tv_sec); >> + ^~~~~~~~~~~ >> + >> +Also reported here [1] >> + >> +[1] https://github.com/agherzan/meta-raspberrypi/issues/969 >> + >> +Upstream-Status: Pending >> +Signed-off-by: Khem Raj <raj.k...@gmail.com> >> +--- >> + src/broadcom/vulkan/v3dv_bo.c | 4 ++-- >> + 1 file changed, 2 insertions(+), 2 deletions(-) >> + >> +diff --git a/src/broadcom/vulkan/v3dv_bo.c b/src/broadcom/vulkan/v3dv_bo.c >> +index 71679ce..dc5e7f5 100644 >> +--- a/src/broadcom/vulkan/v3dv_bo.c >> ++++ b/src/broadcom/vulkan/v3dv_bo.c >> +@@ -67,8 +67,8 @@ bo_dump_stats(struct v3dv_device *device) >> + >> + struct timespec time; >> + clock_gettime(CLOCK_MONOTONIC, &time); >> +- fprintf(stderr, " now: %ld\n", >> +- time.tv_sec); >> ++ fprintf(stderr, " now: %lld\n", >> ++ (long long)time.tv_sec); >> + } >> + >> + if (cache->size_list_size) { >> +-- >> +2.34.1 >> + >> diff --git a/meta/recipes-graphics/mesa/mesa.inc >> b/meta/recipes-graphics/mesa/mesa.inc >> index 30b9e93f630..31ff38dfdca 100644 >> --- a/meta/recipes-graphics/mesa/mesa.inc >> +++ b/meta/recipes-graphics/mesa/mesa.inc >> @@ -20,6 +20,7 @@ SRC_URI = >> "https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \ >> file://0001-meson-misdetects-64bit-atomics-on-mips-clang.patch \ >> file://0001-futex.h-Define-__NR_futex-if-it-does-not-exist.patch >> \ >> file://0001-util-format-Check-for-NEON-before-using-it.patch \ >> + file://0001-v3dv-account-for-64bit-time_t-on-32bit-arches.patch \ >> " >> >> SRC_URI[sha256sum] = >> "a2753c09deef0ba14d35ae8a2ceff3fe5cd13698928c7bb62c2ec8736eb09ce1" >> -- >> 2.34.1 >> >> >> >>
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#159336): https://lists.openembedded.org/g/openembedded-core/message/159336 Mute This Topic: https://lists.openembedded.org/mt/87572545/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-