On 6/15/19 8:42 AM, Mauro Rossi wrote:
Hi,
there is a typo in the commit title, the library is libmesa_winsys_virgl_common
I will correct it in the final commit

Reviewed-by: Tapani Pälli <tapani.pa...@intel.com>

Mauro

On Sat, Jun 15, 2019 at 7:39 AM Mauro Rossi <issor.or...@gmail.com <mailto:issor.or...@gmail.com>> wrote:

    Fixes the following building errors and resolves Bug 110922
    Fixes gallium_dri target missing symbols at linking.

    external/mesa/src/gallium/winsys/virgl/drm/Android.mk:
    error: libmesa_winsys_virgl (STATIC_LIBRARIES android-x86_64)
    missing libmesa_winsys_virgl_common (STATIC_LIBRARIES android-x86_64)
    ...
    external/mesa/src/gallium/winsys/virgl/vtest/Android.mk:
    error: libmesa_winsys_virgl_vtest (STATIC_LIBRARIES android-x86_64)
    missing libmesa_winsys_virgl_common (STATIC_LIBRARIES android-x86_64)
    ...
    build/core/main.mk:728 <http://main.mk:728>: error: exiting from
    previous errors.

    In file included from
    external/mesa/src/gallium/winsys/virgl/vtest/virgl_vtest_socket.c:34:
    external/mesa/src/gallium/winsys/virgl/vtest/virgl_vtest_winsys.h:35:10:
    fatal error: 'virgl_resource_cache.h' file not found
              ^~~~~~~~~~~~~~~~~~~~~~~~
    1 error generated.

    In file included from
    external/mesa/src/gallium/winsys/virgl/vtest/virgl_vtest_winsys.c:32:
    external/mesa/src/gallium/winsys/virgl/vtest/virgl_vtest_winsys.h:35:10:
    fatal error: 'virgl_resource_cache.h' file not found
    #include "virgl_resource_cache.h"
              ^~~~~~~~~~~~~~~~~~~~~~~~
    1 error generated.

    Fixes: b18f09a ("virgl: Introduce virgl_resource_cache")
    Signed-off-by: Mauro Rossi <issor.or...@gmail.com
    <mailto:issor.or...@gmail.com>>
    ---
      src/gallium/Android.mk                    | 2 +-
      src/gallium/drivers/virgl/Android.mk      | 2 +-
      src/gallium/winsys/virgl/drm/Android.mk   | 2 ++
      src/gallium/winsys/virgl/vtest/Android.mk | 2 ++
      4 files changed, 6 insertions(+), 2 deletions(-)

    diff --git a/src/gallium/Android.mk b/src/gallium/Android.mk
    index 3a3f042c7a..37e923c225 100644
    --- a/src/gallium/Android.mk
    +++ b/src/gallium/Android.mk
    @@ -43,7 +43,7 @@ SUBDIRS += winsys/radeon/drm drivers/r300
      SUBDIRS += winsys/radeon/drm drivers/r600
      SUBDIRS += winsys/radeon/drm winsys/amdgpu/drm drivers/radeonsi
      SUBDIRS += winsys/vc4/drm drivers/vc4
    -SUBDIRS += winsys/virgl/drm winsys/virgl/vtest drivers/virgl
    +SUBDIRS += winsys/virgl/common winsys/virgl/drm winsys/virgl/vtest
    drivers/virgl
      SUBDIRS += winsys/svga/drm drivers/svga
      SUBDIRS += winsys/etnaviv/drm drivers/etnaviv drivers/renderonly
      SUBDIRS += state_trackers/dri
    diff --git a/src/gallium/drivers/virgl/Android.mk
    b/src/gallium/drivers/virgl/Android.mk
    index 0067dfa702..a6fe53fbe9 100644
    --- a/src/gallium/drivers/virgl/Android.mk
    +++ b/src/gallium/drivers/virgl/Android.mk
    @@ -35,5 +35,5 @@ include $(BUILD_STATIC_LIBRARY)

      ifneq ($(HAVE_GALLIUM_VIRGL),)
      GALLIUM_TARGET_DRIVERS += virtio_gpu
    -$(eval GALLIUM_LIBS += $(LOCAL_MODULE) libmesa_winsys_virgl
    libmesa_winsys_virgl_vtest)
    +$(eval GALLIUM_LIBS += $(LOCAL_MODULE) libmesa_winsys_virgl_common
    libmesa_winsys_virgl libmesa_winsys_virgl_vtest)
      endif
    diff --git a/src/gallium/winsys/virgl/drm/Android.mk
    b/src/gallium/winsys/virgl/drm/Android.mk
    index 5e2500774e..398a7645bc 100644
    --- a/src/gallium/winsys/virgl/drm/Android.mk
    +++ b/src/gallium/winsys/virgl/drm/Android.mk
    @@ -27,6 +27,8 @@ include $(CLEAR_VARS)

      LOCAL_SRC_FILES := $(C_SOURCES)

    +LOCAL_C_INCLUDES := $(GALLIUM_TOP)/winsys/virgl/common
    +
      LOCAL_MODULE := libmesa_winsys_virgl

      LOCAL_STATIC_LIBRARIES := libmesa_winsys_virgl_common
    diff --git a/src/gallium/winsys/virgl/vtest/Android.mk
    b/src/gallium/winsys/virgl/vtest/Android.mk
    index 5b33f67711..6d35223c8e 100644
    --- a/src/gallium/winsys/virgl/vtest/Android.mk
    +++ b/src/gallium/winsys/virgl/vtest/Android.mk
    @@ -27,6 +27,8 @@ include $(CLEAR_VARS)

      LOCAL_SRC_FILES := $(C_SOURCES)

    +LOCAL_C_INCLUDES := $(GALLIUM_TOP)/winsys/virgl/common
    +
      LOCAL_MODULE := libmesa_winsys_virgl_vtest

      LOCAL_STATIC_LIBRARIES := libmesa_winsys_virgl_common
-- 2.20.1

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to