Bug 104149

Summary: Crucible build fails on m32 architecture
Product: piglit Reporter: Clayton Craft <clayton.a.craft>
Component: CrucibleAssignee: Piglit Mailing List <piglit>
Status: RESOLVED FIXED QA Contact: Piglit Mailing List <piglit>
Severity: normal    
Priority: medium CC: jason
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:

Description Clayton Craft 2017-12-07 02:46:45 UTC 
Format specifier is for %lu but it is being passed a uint64_t, causing the compiler to complain when building crucible for m32 targets:


16:27:50 src/framework/test/t_phase_setup.c: In function ‘debug_cb’:
16:27:50 src/framework/test/t_phase_setup.c:312:24: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘uint64_t {aka long long unsigned int}’ [-Werror=format=]
16:27:50          logi("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                         ^
16:27:50 src/framework/test/t_phase_setup.c:312:47: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘size_t {aka unsigned int}’ [-Werror=format=]
16:27:50          logi("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                                                ^
16:27:50 src/framework/test/t_phase_setup.c:316:24: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘uint64_t {aka long long unsigned int}’ [-Werror=format=]
16:27:50          logw("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                         ^
16:27:50 src/framework/test/t_phase_setup.c:316:47: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘size_t {aka unsigned int}’ [-Werror=format=]
16:27:50          logw("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                                                ^
16:27:50 src/framework/test/t_phase_setup.c:320:24: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘uint64_t {aka long long unsigned int}’ [-Werror=format=]
16:27:50          logw("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                         ^
16:27:50 src/framework/test/t_phase_setup.c:320:47: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘size_t {aka unsigned int}’ [-Werror=format=]
16:27:50          logw("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                                                ^
16:27:50 src/framework/test/t_phase_setup.c:324:24: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘uint64_t {aka long long unsigned int}’ [-Werror=format=]
16:27:50          loge("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                         ^
16:27:50 src/framework/test/t_phase_setup.c:324:47: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘size_t {aka unsigned int}’ [-Werror=format=]
16:27:50          loge("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                                                ^
16:27:50 src/framework/test/t_phase_setup.c:330:24: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘uint64_t {aka long long unsigned int}’ [-Werror=format=]
16:27:50          logd("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                         ^
16:27:50 src/framework/test/t_phase_setup.c:330:47: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘size_t {aka unsigned int}’ [-Werror=format=]
16:27:50          logd("object %lu type 0x%x location %lu code %u layer \"%s\" msg %s",
16:27:50                                                ^
16:27:50 cc1: some warnings being treated as errors
Comment 1 Tapani Pälli 2018-02-27 11:28:53 UTC 
commit b58596b85007cfba8aff92b5eeb93f2461ecf87c (origin/master, origin/HEAD)
Author: Clayton Craft <clayton.a.craft@intel.com>
Date:   Wed Dec 6 18:49:30 2017 -0800

    crucible: fix m32 build errors due to formatting for uint64 and size_t
    
    This fixes compile errors when building crucible for m32 targets, mainly
    that %lu means something different on m32 than it does on m64.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104149

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.