[Mesa-dev] [PATCH 1/3] util/vulkan: Move Vulkan utilities to src/vulkan/util
Tapani Pälli
tapani.palli at intel.com
Tue Jun 6 11:20:30 UTC 2017
On 06/06/2017 02:13 PM, Alex Smith wrote:
> Aha, sorry, I missed that. Do you want me to fold your patch into this
> one and send a v2?
That would be ideal as working build makes bisecting a lot less painful,
thanks!
> Alex
>
> On 6 June 2017 at 12:11, Tapani Pälli <tapani.palli at intel.com
> <mailto:tapani.palli at intel.com>> wrote:
>
> Looks nice to me, I've sent required changes to Android build system
> for this change as addition.
>
>
> On 06/06/2017 12:42 PM, Alex Smith wrote:
>
> We have Vulkan utilities in both src/util and src/vulkan/util. The
> latter seems a more appropriate place for Vulkan-specific things, so
> move them there.
>
> Signed-off-by: Alex Smith <asmith at feralinteractive.com
> <mailto:asmith at feralinteractive.com>>
> ---
> src/amd/vulkan/radv_device.c | 2 +-
> src/amd/vulkan/radv_formats.c | 3 ++-
> src/amd/vulkan/radv_private.h | 2 +-
> src/amd/vulkan/radv_wsi.c | 2 +-
> src/intel/vulkan/anv_device.c | 2 +-
> src/intel/vulkan/anv_formats.c | 3 +--
> src/intel/vulkan/anv_pass.c | 2 +-
> src/intel/vulkan/anv_private.h | 2 +-
> src/intel/vulkan/anv_queue.c | 2 +-
> src/intel/vulkan/anv_wsi.c | 2 +-
> src/intel/vulkan/genX_cmd_buffer.c | 2 +-
> src/util/Makefile.sources | 4 +---
> src/vulkan/Makefile.am | 7 ++++++-
> src/vulkan/Makefile.sources | 4 ++++
> src/{ => vulkan}/util/vk_alloc.h | 0
> src/{ => vulkan}/util/vk_util.h | 0
> src/vulkan/wsi/wsi_common.h | 2 +-
> src/vulkan/wsi/wsi_common_wayland.c | 2 +-
> src/vulkan/wsi/wsi_common_x11.c | 2 +-
> 19 files changed, 26 insertions(+), 19 deletions(-)
> rename src/{ => vulkan}/util/vk_alloc.h (100%)
> rename src/{ => vulkan}/util/vk_util.h (100%)
>
> diff --git a/src/amd/vulkan/radv_device.c
> b/src/amd/vulkan/radv_device.c
> index 887916f..a812527 100644
> --- a/src/amd/vulkan/radv_device.c
> +++ b/src/amd/vulkan/radv_device.c
> @@ -33,7 +33,7 @@
> #include "radv_cs.h"
> #include "util/disk_cache.h"
> #include "util/strtod.h"
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> #include <xf86drm.h>
> #include <amdgpu.h>
> #include <amdgpu_drm.h>
> diff --git a/src/amd/vulkan/radv_formats.c
> b/src/amd/vulkan/radv_formats.c
> index 6cff0a5..b13adb9 100644
> --- a/src/amd/vulkan/radv_formats.c
> +++ b/src/amd/vulkan/radv_formats.c
> @@ -28,7 +28,8 @@
> #include "sid.h"
> #include "r600d_common.h"
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> +
> #include "util/u_half.h"
> #include "util/format_srgb.h"
> #include "util/format_r11g11b10f.h"
> diff --git a/src/amd/vulkan/radv_private.h
> b/src/amd/vulkan/radv_private.h
> index ed80ba7..a3920a7 100644
> --- a/src/amd/vulkan/radv_private.h
> +++ b/src/amd/vulkan/radv_private.h
> @@ -47,8 +47,8 @@
> #include "compiler/shader_enums.h"
> #include "util/macros.h"
> #include "util/list.h"
> -#include "util/vk_alloc.h"
> #include "main/macros.h"
> +#include "vk_alloc.h"
> #include "radv_radeon_winsys.h"
> #include "ac_binary.h"
> diff --git a/src/amd/vulkan/radv_wsi.c b/src/amd/vulkan/radv_wsi.c
> index 51fe159..cdb04ca 100644
> --- a/src/amd/vulkan/radv_wsi.c
> +++ b/src/amd/vulkan/radv_wsi.c
> @@ -26,7 +26,7 @@
> #include "radv_private.h"
> #include "radv_meta.h"
> #include "wsi_common.h"
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> static const struct wsi_callbacks wsi_cbs = {
> .get_phys_device_format_properties =
> radv_GetPhysicalDeviceFormatProperties,
> diff --git a/src/intel/vulkan/anv_device.c
> b/src/intel/vulkan/anv_device.c
> index aacd07f..6079588 100644
> --- a/src/intel/vulkan/anv_device.c
> +++ b/src/intel/vulkan/anv_device.c
> @@ -35,7 +35,7 @@
> #include "util/debug.h"
> #include "util/build_id.h"
> #include "util/mesa-sha1.h"
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> #include "genxml/gen7_pack.h"
> diff --git a/src/intel/vulkan/anv_formats.c
> b/src/intel/vulkan/anv_formats.c
> index 25801e8..104d4f7 100644
> --- a/src/intel/vulkan/anv_formats.c
> +++ b/src/intel/vulkan/anv_formats.c
> @@ -23,8 +23,7 @@
> #include "anv_private.h"
> #include "vk_format_info.h"
> -
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> /*
> * gcc-4 and earlier don't allow compound literals where a
> constant
> diff --git a/src/intel/vulkan/anv_pass.c
> b/src/intel/vulkan/anv_pass.c
> index 93f1483..1b30c14 100644
> --- a/src/intel/vulkan/anv_pass.c
> +++ b/src/intel/vulkan/anv_pass.c
> @@ -23,7 +23,7 @@
> #include "anv_private.h"
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> static unsigned
> num_subpass_attachments(const VkSubpassDescription *desc)
> diff --git a/src/intel/vulkan/anv_private.h
> b/src/intel/vulkan/anv_private.h
> index c261faa..fe6ac3b 100644
> --- a/src/intel/vulkan/anv_private.h
> +++ b/src/intel/vulkan/anv_private.h
> @@ -47,7 +47,7 @@
> #include "util/macros.h"
> #include "util/list.h"
> #include "util/u_vector.h"
> -#include "util/vk_alloc.h"
> +#include "vk_alloc.h"
> /* Pre-declarations needed for WSI entrypoints */
> struct wl_surface;
> diff --git a/src/intel/vulkan/anv_queue.c
> b/src/intel/vulkan/anv_queue.c
> index be7fd31..fd4d362 100644
> --- a/src/intel/vulkan/anv_queue.c
> +++ b/src/intel/vulkan/anv_queue.c
> @@ -30,7 +30,7 @@
> #include <sys/eventfd.h>
> #include "anv_private.h"
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> #include "genxml/gen7_pack.h"
> diff --git a/src/intel/vulkan/anv_wsi.c
> b/src/intel/vulkan/anv_wsi.c
> index 7575f58..9369f26 100644
> --- a/src/intel/vulkan/anv_wsi.c
> +++ b/src/intel/vulkan/anv_wsi.c
> @@ -24,7 +24,7 @@
> #include "anv_private.h"
> #include "wsi_common.h"
> #include "vk_format_info.h"
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> #ifdef VK_USE_PLATFORM_WAYLAND_KHR
> static const struct wsi_callbacks wsi_cbs = {
> diff --git a/src/intel/vulkan/genX_cmd_buffer.c
> b/src/intel/vulkan/genX_cmd_buffer.c
> index 072d0d5..59430a2 100644
> --- a/src/intel/vulkan/genX_cmd_buffer.c
> +++ b/src/intel/vulkan/genX_cmd_buffer.c
> @@ -26,7 +26,7 @@
> #include "anv_private.h"
> #include "vk_format_info.h"
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> #include "common/gen_l3_config.h"
> #include "genxml/gen_macros.h"
> diff --git a/src/util/Makefile.sources b/src/util/Makefile.sources
> index e905734..e9f820a 100644
> --- a/src/util/Makefile.sources
> +++ b/src/util/Makefile.sources
> @@ -51,9 +51,7 @@ MESA_UTIL_FILES := \
> u_string.h \
> u_thread.h \
> u_vector.c \
> - u_vector.h \
> - vk_alloc.h \
> - vk_util.h
> + u_vector.h
> MESA_UTIL_GENERATED_FILES = \
> format_srgb.c
> diff --git a/src/vulkan/Makefile.am b/src/vulkan/Makefile.am
> index 71fb2d9..c897a07 100644
> --- a/src/vulkan/Makefile.am
> +++ b/src/vulkan/Makefile.am
> @@ -11,6 +11,10 @@ PYTHON_GEN = $(AM_V_GEN)$(PYTHON2)
> $(PYTHON_FLAGS)
> EXTRA_DIST = \
> util/gen_enum_to_str.py
> +VULKAN_UTIL_SOURCES = \
> + $(VULKAN_UTIL_FILES) \
> + $(VULKAN_UTIL_GENERATED_FILES)
> +
> BUILT_SOURCES = \
> $(VULKAN_UTIL_GENERATED_FILES)
> @@ -18,12 +22,13 @@ util/vk_enum_to_str.c
> util/vk_enum_to_str.h: util/gen_enum_to_str.py $(vulkan_ap
> $(MKDIR_GEN)
> $(PYTHON_GEN) $(srcdir)/util/gen_enum_to_str.py --xml
> $(vulkan_api_xml) --outdir $(top_builddir)/src/vulkan/util
> -libvulkan_util_la_SOURCES = $(VULKAN_UTIL_GENERATED_FILES)
> +libvulkan_util_la_SOURCES = $(VULKAN_UTIL_SOURCES)
> AM_CPPFLAGS = \
> $(DEFINES) \
> -I$(top_srcdir)/include \
> -I$(top_srcdir)/src \
> + -I$(top_srcdir)/src/vulkan/util \
> -I$(top_srcdir)/src/gallium/auxiliary \
> -I$(top_srcdir)/src/gallium/include
> diff --git a/src/vulkan/Makefile.sources
> b/src/vulkan/Makefile.sources
> index 63f4ac1..9962c1b 100644
> --- a/src/vulkan/Makefile.sources
> +++ b/src/vulkan/Makefile.sources
> @@ -15,6 +15,10 @@ VULKAN_WSI_X11_FILES := \
> wsi/wsi_common_x11.c \
> wsi/wsi_common_x11.h
> +VULKAN_UTIL_FILES := \
> + util/vk_alloc.h \
> + util/vk_util.h
> +
> VULKAN_UTIL_GENERATED_FILES := \
> util/vk_enum_to_str.c \
> util/vk_enum_to_str.h
> diff --git a/src/util/vk_alloc.h b/src/vulkan/util/vk_alloc.h
> similarity index 100%
> rename from src/util/vk_alloc.h
> rename to src/vulkan/util/vk_alloc.h
> diff --git a/src/util/vk_util.h b/src/vulkan/util/vk_util.h
> similarity index 100%
> rename from src/util/vk_util.h
> rename to src/vulkan/util/vk_util.h
> diff --git a/src/vulkan/wsi/wsi_common.h
> b/src/vulkan/wsi/wsi_common.h
> index 8aee9c7..8166b7d 100644
> --- a/src/vulkan/wsi/wsi_common.h
> +++ b/src/vulkan/wsi/wsi_common.h
> @@ -26,7 +26,7 @@
> #include <stdint.h>
> #include <stdbool.h>
> -#include "util/vk_alloc.h"
> +#include "vk_alloc.h"
> #include <vulkan/vulkan.h>
> #include <vulkan/vk_icd.h>
> diff --git a/src/vulkan/wsi/wsi_common_wayland.c
> b/src/vulkan/wsi/wsi_common_wayland.c
> index 644ed62..dd283a1 100644
> --- a/src/vulkan/wsi/wsi_common_wayland.c
> +++ b/src/vulkan/wsi/wsi_common_wayland.c
> @@ -31,7 +31,7 @@
> #include <string.h>
> #include <pthread.h>
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> #include "wsi_common_wayland.h"
> #include "wayland-drm-client-protocol.h"
> diff --git a/src/vulkan/wsi/wsi_common_x11.c
> b/src/vulkan/wsi/wsi_common_x11.c
> index 5be56f1..ecdaf91 100644
> --- a/src/vulkan/wsi/wsi_common_x11.c
> +++ b/src/vulkan/wsi/wsi_common_x11.c
> @@ -38,7 +38,7 @@
> #include <xf86drm.h>
> #include "util/hash_table.h"
> -#include "util/vk_util.h"
> +#include "vk_util.h"
> #include "wsi_common.h"
> #include "wsi_common_x11.h"
> #include "wsi_common_queue.h"
>
>
More information about the mesa-dev
mailing list