frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

modesetting: Fix build with DebugPresent() enabled 

By default, the macro DebugPresent() is a no-op but it can be enabled
at build time for debugging purpose.

However, doing so prevents the code to build because one debug statement
tries to make use of a non-existent variable:

  present.c: In function ‘ms_present_queue_vblank’:
  present.c:147:18: error: ‘vbl’ undeclared (first use in this function)
    147 |                  vbl.request.sequence));
        |                  ^~~
  present.c:49:32: note: in definition of macro ‘DebugPresent’
    49 | #define DebugPresent(x) ErrorF x
       |                                ^

Fix the build with DebugPresent() by removing the vbl variable from the
debug message.

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
This commit is contained in:
Olivier Fourdan 2020-11-26 14:39:32 +01:00 committed by Peter Hutterer
parent 95b79aa907
commit 3ce05a44f3
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
hw/xfree86/drivers/modesetting/present.c
View File

@ -142,9 +142,8 @@ ms_present_queue_vblank(RRCrtcPtr crtc,
if (!ms_queue_vblank(xf86_crtc, MS_QUEUE_ABSOLUTE, msc, NULL, seq)) if (!ms_queue_vblank(xf86_crtc, MS_QUEUE_ABSOLUTE, msc, NULL, seq))
return BadAlloc; return BadAlloc;
DebugPresent(("\t\tmq %lld seq %u msc %llu (hw msc %u)\n", DebugPresent(("\t\tmq %lld seq %u msc %llu\n",
(long long) event_id, seq, (long long) msc, (long long) event_id, seq, (long long) msc));
vbl.request.sequence));
return Success; return Success;
} }