[Bug 87858] New: [bisected] build errors on xserver master branch

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=87858

            Bug ID: 87858
           Summary: [bisected] build errors on xserver master branch
           Product: xorg
           Version: git
          Hardware: All
                OS: Linux (All)
            Status: NEW
          Severity: major
          Priority: high
         Component: Driver/intel
          Assignee: chris at chris-wilson.co.uk
          Reporter: xunx.fang at intel.com
        QA Contact: intel-gfx-bugs at lists.freedesktop.org
                CC: kenneth at whitecape.org

Created attachment 111508
  --> https://bugs.freedesktop.org/attachment.cgi?id=111508&action=edit
build log

Bug detailed description:
------------------------- 
Build fails on xserver master branch with following error:
present.c: In function 'ms_present_flush':
present.c:204:9: error: implicit declaration of function 'glamor_block_handler'
[-Werror=implicit-function-declaration]
         glamor_block_handler(screen);
         ^present.c:204:9: warning: nested extern declaration of
'glamor_block_handler' [-Wnested-externs]
cc1: some warnings being treated as errors
make[5]: *** [present.lo] Error 1

Bisect shows commit 09230a2 is the first bad commit.
commit 09230a2d435aa7f893ad5022220e3eb39ffb6db5
Author:     Kenneth Graunke <kenneth at whitecape.org>
AuthorDate: Thu Dec 11 13:37:14 2014 -0800
Commit:     Keith Packard <keithp at keithp.com>
CommitDate: Thu Dec 25 13:39:19 2014 -0800

    modesetting: Add vblank synchronization support when using Present.

    modesetting hooked up vblank support for DRI2, but was missing support
    for vblanks in Present.

    This is mostly copy and pasted from Keith's code in the intel driver.

    v2: Use ms_crtc_msc_to_kernel_msc in ms_present_queue_vblank to hook
        up the vblank_offset workaround for bogus MSC values (which the
        DRI2 code already did).

        Also simplify the ms_present_get_crtc function.  vblank.c already
        implements the functionality; we just need to convert types.

    v3: Fix ms_flush_drm_events return code.  I'd copied code where 0 meant
        success into a function that returned a boolean, so the return code
        was always backwards.

Reproduce steps:
----------------
./autogen.sh  --prefix=${INSTALL_ROOT} --enable-config-udev
--disable-config-hal --disable-unit-tests

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20141230/efe0f3b4/attachment.html>