Mesa (main): 22 new commits

Thu Oct 7 21:19:17 UTC 2021

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=305b1702293b406c83b0db422e3f87b1e588352d
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Thu Sep 30 11:41:15 2021 -0500

    vulkan/device: Use vk_errorf to report missing features
    
    With a tiny bit more code-gen, we can now not only throw the error but
    also log back to the client exactly which feature was missing.
    
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5b42f1a37425a2c8a8683073837d28db53e044df
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 16:04:51 2021 -0500

    vulkan/device: Use vk_error
    
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=330f4c9bc918977aae65bbb92504fa4922ced793
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 15:52:04 2021 -0500

    vulkan/instance: Use vk_error in vk_instance_init
    
    We have to be a bit careful here.  Calling log functions during instance
    initialization can be a bit sketchy.  However, we know that __vk_log_impl
    only ever touches the callbacks lists if instance->base.client_visible
    so it's safe to call vk_error as soon as we've set up instance_callbacks.
    
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ad27db2ae99a1d61152699490dbec6e2f2c19aa0
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 15:38:40 2021 -0500

    vulkan/log: Drop _impl from the log helper names
    
    Now that we no longer have every driver in the tree defining their own
    __vk_errorf and __vk_errorv, we don't need to worry about the symbol
    collision anymore and can use the "real" names for the common ones.
    
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a1ac8234ec0b2580436ec5f3b4b932664549a776
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 15:35:20 2021 -0500

    turnip: Plumb non-startup errors through the new vk_error helpers
    
    Also, change every vk_error to use the closest object instead of
    fetching all the way back to the instance.
    
    Reviewed-by: Danylo Piliaiev <dpiliaiev at igalia.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=bab0530f07cd9889db1d445d70ab18f7b7985bdf
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 15:31:03 2021 -0500

    v3dv: Switch to the new vk_error helpers
    
    Also, change every vk_error to use the closest object instead of
    fetching all the way back to the instance.
    
    Reviewed-by: Alejandro Piñeiro <apinheiro at igalia.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=708b65f7043bd15bd143fd1a093faa473e8354e3
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 15:23:14 2021 -0500

    panvk: Switch to the new vk_error helpers
    
    Also switch all our calls to use the closest object to the error and let
    vk_error sort out which object to actually use.
    
    Reviewed-by: Boris Brezillon <boris.brezillon at collabora.com>
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=47adb11143767b6e66945197a70dc91383406665
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 15:20:49 2021 -0500

    lavapipe: Switch to the new vk_error helpers
    
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7a2516568d59b113bdcecd801a0fc571b5fb3eec
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 15:19:16 2021 -0500

    radv: Switch to the new common vk_error helpers
    
    Also, change every vk_error to use the closest object instead of
    fetching all the way back to the instance.
    
    Reviewed-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9e299b50ab55f153ad80f4f06c9ee2f74dbb6216
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Wed Oct 6 09:18:42 2021 -0500

    radv: Stop printing descriptor pool allocation failures
    
    The VK_ERROR_FRAGMENTED_POOL and VK_ERROR_OUT_OF_POOL_MEMORY errors are
    not as exceptional cases as most.  These are expected to be hit by
    applications in the normal course of doing their thing.  Probably best
    not to spam stderr and the debug logs with them.
    
    Reviewed-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=88a8b937b5f3c9620a3edc90897ab267e00785b4
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 12:06:32 2021 -0500

    anv: Use the common vk_error and vk_errorf helpers
    
    Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f6d52768d636beffb7a9e8e96d127ce2bbecd696
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 11:22:10 2021 -0500

    anv/queue: Plumb the queue through all the queue_submit calls
    
    Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ab36efcb4c7b43f799d9c2c6b6fc6e34753ccb5b
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 15:15:24 2021 -0500

    anv: Drop unused logging helpers
    
    Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0cad3beb2a0dda7a8f6c6a1e8c9d253b9c988a9f
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 13:38:17 2021 -0500

    vulkan/log: Add common vk_error and vk_errorf helpers
    
    These helpers have quite a bit of smarts in them to log errors to chase
    the object chain as needed and log errors to roughly the appropriate
    object.  For instance, VK_ERROR_OUT_OF_DEVICE_MEMORY always goes to a
    device while VK_ERROR_OUT_OF_HOST_MEMORY always goes to the instance.
    
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ec34ec388b3f88b352886287a0d505f8ea692bf1
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 14:53:13 2021 -0500

    vulkan/log: Handle logging to a physical device
    
    Instance-level objects won't have a device pointer so we can't rely on
    that.  Instead, we should look at the object type and try to chase it
    back to an instance.  Sadly, we can't do that for certain display and
    WSI objects.  However, we never use the vk_log* helpers for those.
    
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ad27b27389dddb2395df0af20cc19f617215f022
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 13:44:41 2021 -0500

    anv: s/vk_error/anv_error/g
    
    Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=aa94220d7d952a67545ab73c9e6909f311fb4ddc
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Wed Oct 6 09:12:43 2021 -0500

    anv: Stop printing descriptor pool allocation failures
    
    The VK_ERROR_FRAGMENTED_POOL and VK_ERROR_OUT_OF_POOL_MEMORY errors are
    not as exceptional cases as most.  These are expected to be hit by
    applications in the normal course of doing their thing.  Probably best
    not to spam stderr and the debug logs with them.
    
    Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=31148ee88e6c8d0ec502d4dcc36360ed42fead29
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 13:43:53 2021 -0500

    anv: drop a misplaced and wrong comment
    
    We do actually use vk_error in the one place we check that limit.
    
    Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9067c12d2a00d4ceb588482225fd9f2254dde807
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 14:26:35 2021 -0500

    vulkan/log: Log to instance messages during instance construction
    
    If the instance isn't client-visible yet (i.e. foo_instance_to_handle
    hasn't been called), then the instance is still under construction and
    we should log using vk_debug_message_instance.  This makes the vk_log*
    macros work regardless of whether the instance is fully constructed or
    not.
    
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e884e35077db94892d17b430ab946127d891b7e1
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 14:27:49 2021 -0500

    vulkan/log: Assert if the driver logs a client-invisible object
    
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=571b5f5000b6ebc9a769d3cbea3c8d3c8bad4bce
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 10:58:39 2021 -0500

    vulkan: Track which objects are client-visible
    
    When dealing with debug logging, it's useful to track when an object's
    construction is finished and it's now visible to the client.  We can
    detect this pretty easily by setting a flag the first time foo_to_handle
    is called.  As long as drivers only ever call that function at the end
    of object construction (they all do to my knowledge), this should be a
    reliable mechanism for detecting when a client knows about a handle.
    
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a815b2b3c1208419a45d46469f230333369751b4
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Fri Sep 24 14:05:16 2021 -0500

    vulkan: Drop vk_object_base_reset
    
    It's no longer used and just makes the init/finish path more
    complicated.
    
    Tested-by: Boris Brezillon <boris.brezillon at collabora.com>
    Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13045>