Module: Mesa
Branch: staging/23.0
Commit: d2dc9fd75c7c8137c17ac2fc90b367db747bfb74
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=d2dc9fd75c7c8137c17ac2fc90b367db747bfb74

Author: Marcin Ĺšlusarz <[email protected]>
Date:   Fri Mar 24 17:54:15 2023 +0100

anv,hasvk: remove stale comments

Added in 060a6434eca9fb05ca2dfd612f8abd4786ee4549.
The field it described was removed in c1f4343807d1040bd7b5440aa2f5fccf5f12842d.

Fixes: c1f4343807d ("anv: Set up memory types and heaps during physical device 
init")
Reviewed-by: Faith Ekstrand <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22113>
(cherry picked from commit 698135fd3e262894b4198f7b3d15ce84879a7106)

---

 .pick_status.json                    | 2 +-
 src/intel/vulkan/anv_private.h       | 8 --------
 src/intel/vulkan_hasvk/anv_private.h | 8 --------
 3 files changed, 1 insertion(+), 17 deletions(-)

diff --git a/.pick_status.json b/.pick_status.json
index 009caa421f4..8dd016dd93b 100644
--- a/.pick_status.json
+++ b/.pick_status.json
@@ -3892,7 +3892,7 @@
         "description": "anv,hasvk: remove stale comments",
         "nominated": true,
         "nomination_type": 1,
-        "resolution": 0,
+        "resolution": 1,
         "main_sha": null,
         "because_sha": "c1f4343807d1040bd7b5440aa2f5fccf5f12842d"
     },
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h
index 3b7db8531fb..2fd659665ca 100644
--- a/src/intel/vulkan/anv_private.h
+++ b/src/intel/vulkan/anv_private.h
@@ -914,14 +914,6 @@ struct anv_physical_device {
     struct anv_instance *                       instance;
     char                                        path[20];
     struct intel_device_info                      info;
-    /** Amount of "GPU memory" we want to advertise
-     *
-     * Clearly, this value is bogus since Intel is a UMA architecture.  On
-     * gfx7 platforms, we are limited by GTT size unless we want to implement
-     * fine-grained tracking and GTT splitting.  On Broadwell and above we are
-     * practically unlimited.  However, we will never report more than 3/4 of
-     * the total system ram to try and avoid running out of RAM.
-     */
     bool                                        supports_48bit_addresses;
     struct brw_compiler *                       compiler;
     struct isl_device                           isl_dev;
diff --git a/src/intel/vulkan_hasvk/anv_private.h 
b/src/intel/vulkan_hasvk/anv_private.h
index 4a5e19d9bdc..5e47f1af5b9 100644
--- a/src/intel/vulkan_hasvk/anv_private.h
+++ b/src/intel/vulkan_hasvk/anv_private.h
@@ -858,14 +858,6 @@ struct anv_physical_device {
     struct anv_instance *                       instance;
     char                                        path[20];
     struct intel_device_info                      info;
-    /** Amount of "GPU memory" we want to advertise
-     *
-     * Clearly, this value is bogus since Intel is a UMA architecture.  On
-     * gfx7 platforms, we are limited by GTT size unless we want to implement
-     * fine-grained tracking and GTT splitting.  On Broadwell and above we are
-     * practically unlimited.  However, we will never report more than 3/4 of
-     * the total system ram to try and avoid running out of RAM.
-     */
     bool                                        supports_48bit_addresses;
     struct brw_compiler *                       compiler;
     struct isl_device                           isl_dev;

Reply via email to