On Sat, 2017-06-03 at 11:10 -0700, Jason Ekstrand wrote:
> On Sat, Jun 3, 2017 at 10:05 AM, Juan A. Suarez Romero <jasuarez@igal
> ia.com> wrote:
> > On Thu, 2017-06-01 at 16:35 -0700, Jason Ekstrand wrote:
> > > On Thu, Jun 1, 2017 at 4:20 PM, Juan A. Suarez Romero <jasuarez@i
> > > galia.com> wrote:
> > > > Hello list,
> > > >
> > > >
> > > >
> > > > The candidate for the Mesa 17.1.2 is now available. Currently
> > > > we have:
> > > >
> > > > - 51 queued
> > > >
> > > > - 3 nominated (outstanding)
> > > >
> > > > - and 9 rejected patch(es)
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > The current queue contains fixes in several places.
> > > >
> > > >
> > > >
> > > > The current queue consists of:
> > > >
> > > >
> > > >
> > > > Several fixes in the autotools' configure that improves
> > > > handling of target platforms.
> > > >
> > > >
> > > >
> > > > Mesa receives a fix for a leaking in a surface.
> > > >
> > > >
> > > >
> > > > OMX has a couple of fixes when building in a X11-less setup.
> > > >
> > > >
> > > >
> > > > ANV driver receives several fixes, like adding some missing
> > > > transitions between layouts, or improvements in setting memory
> > > > types and heaps.
> > > >
> > > >
> > > >
> > > > There are a few fixes in i965, R100, R200, Radeonsi/GFX9, Radv
> > > > and Freedreno drivers.
> > > >
> > > >
> > > >
> > > > Also, EGL/Android receives a fix for a segfault that replaces a
> > > > previous one that was not correct.
> > > >
> > > >
> > > >
> > > > Vulkan's Wayland gets some fixes, including the use of proxy
> > > > wrappers for swapchains, or per-display event queue usage.
> > > >
> > > >
> > > >
> > > > Likewise EGL's Wayland now use per-surface event queues, as
> > > > well as other fixes like ensuring back buffer is obtained.
> > > >
> > > >
> > > >
> > > > Take a look at section "Mesa stable queue" for more
> > > > information.
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Testing reports/general approval
> > > >
> > > > --------------------------------
> > > >
> > > >
> > > >
> > > > Any testing reports (or general approval of the state of the
> > > > branch)
> > > >
> > > > will be greatly appreciated.
> > > >
> > > >
> > > >
> > > > The plan is to have 17.1.2 this Friday (3rd of June), around or
> > > >
> > > > shortly after 23:00 GMT.
> > > >
> > > >
> > > >
> > > > If you have any questions or suggestions - be that about the
> > > > current
> > > >
> > > > patch queue or otherwise, please go ahead.
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Trivial merge conflicts
> > > >
> > > > -----------------------
> > > >
> > > >
> > > >
> > > > commit e064f7d826812598749ce27868d9ff865c137ef4
> > > >
> > > > Author: Jason Ekstrand <jason.ekstr...@intel.com>
> > > >
> > > >
> > > >
> > > > anv: Set up memory types and heaps during physical device
> > > > init
> > > >
> > > >
> > > >
> > > > (cherry picked from commit
> > > > c1f4343807d1040bd7b5440aa2f5fccf5f12842d)
> > > >
> > > >
> > > >
> > > > commit ce2b96dd8b5a6c4ea0c80c6c0f46652a0b591515
> > > >
> > > > Author: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl>
> > > >
> > > >
> > > >
> > > > radv: Reserve space for descriptor and push constant user
> > > > SGPR setting.
> > > >
> > > >
> > > >
> > > > (cherry picked from commit
> > > > 18efb404cfb38f722a16df7539390cf9a4a71929)
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Cheers,
> > > >
> > > > J.A.
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Mesa stable queue
> > > >
> > > > -----------------
> > > >
> > > >
> > > >
> > > > Nominated (3)
> > > >
> > > > =============
> > > >
> > > > Chad Versace(1):
> > > >
> > > > i965/dri: Fix bad GL error in
> > > > intel_create_winsys_renderbuffer()
> > > >
> > > >
> > > >
> > > > Jason Ekstrand(1):
> > > >
> > > > i965: Rework Sandy Bridge HiZ and stencil layouts
> > >
> > > This just landed in master.
> > >
> >
> >
> > Ok. I've included it in the Queued.
> >
> >
> > > > Nicolas Dechesne(1):
> > > >
> > > > util/rand_xor: add missing include statements
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Queued (51)
> > > >
> > > > ===========
> > > >
> > > > Bartosz Tomczyk (1):
> > > >
> > > > mesa: Avoid leaking surface in st_renderbuffer_delete
> > > >
> > > >
> > > >
> > > > Bas Nieuwenhuizen (1):
> > > >
> > > > radv: Reserve space for descriptor and push constant user
> > > > SGPR setting.
> > > >
> > > >
> > > >
> > > > Daniel Stone (7):
> > > >
> > > > vulkan: Fix Wayland uninitialised registry
> > > >
> > > > vulkan/wsi/wayland: Remove roundtrip when creating image
> > > >
> > > > vulkan/wsi/wayland: Use per-display event queue
> > > >
> > > > vulkan/wsi/wayland: Use proxy wrappers for swapchain
> > > >
> > > > Squashed with
> > > >
> > > > vulkan/wsi/wayland: Fix proxy wrappers for swapchain
> > > > recreation
> > > >
> > > > egl/wayland: Don't open-code roundtrip
> > > >
> > > > egl/wayland: Use per-surface event queues
> > > >
> > > > Squashed with
> > > >
> > > > egl/wayland: verify event queue was allocated
> > > >
> > > > egl/wayland: Ensure we get a back buffer
> > > >
> > > >
> > > >
> > > > Emil Velikov (24):
> > > >
> > > > docs: add sha256 checksums for 17.1.1
> > > >
> > > > configure: move platform handling further up
> > > >
> > > > configure: rename remaining HAVE_EGL_PLATFORM_* guards
> > > >
> > > > configure: update remaining --with-egl-platforms
> > > > references
> > > >
> > > > configure: loosen --with-platforms heuristics
> > > >
> > > > configure: enable the surfaceless platform by default
> > > >
> > > > configure: set HAVE_foo_PLATFORM as applicable
> > > >
> > > > configure: error out when building GLX w/o the X11
> > > > platform
> > > >
> > > > configure: check once for DRI3 dependencies
> > > >
> > > > Squashed with
> > > >
> > > > configure.ac: add xcb-fixes to the XCB DRI3 list
> > > >
> > > > Squashed with
> > > >
> > > > configure.ac: s/xcb-fixes/xcb-xfixes/
> > > >
> > > > loader: build libloader_dri3_helper.la only with
> > > > HAVE_PLATFORM_X11
> > > >
> > > > configure: error out when building X11 Vulkan without
> > > > DRI3
> > > >
> > > > auxiliary/vl: use vl_*_screen_create stubs when building
> > > > w/o platform
> > > >
> > > > st/va: fix misplaced closing bracket
> > > >
> > > > st/omx: remove unneeded X11 include
> > > >
> > > > st/omx: fix building against X11-less setups
> > > >
> > > > gallium/targets: link against XCB only as needed
> > > >
> > > > configure: error out if building VA w/o supported
> > > > platform
> > > >
> > > > configure: error out if building OMX w/o supported
> > > > platform
> > > >
> > > > configure: error out if building VDPAU w/o supported
> > > > platform
> > > >
> > > > configure: error out if building XVMC w/o supported
> > > > platform
> > > >
> > > > travis: remove workarounds for the Vulkan target
> > > >
> > > > anv: automake: list shared libraries after the static
> > > > ones
> > > >
> > > > radv: automake: list shared libraries after the static
> > > > ones
> > > >
> > > > egl/wayland: select the format based on the interface
> > > > used
> > > >
> > > >
> > > >
> > > > Ian Romanick (3):
> > > >
> > > > r100: Don't assume that the base mipmap of a texture
> > > > exists
> > > >
> > > > r100,r200: Don't assume glVisual is non-NULL during
> > > > context creation
> > > >
> > > > r100: Use _mesa_get_format_base_format in
> > > > radeon_update_wrapper
> > > >
> > > >
> > > >
> > > > Jason Ekstrand (8):
> > > >
> > > > anv: Handle color layout transitions from the
> > > > UNINITIALIZED layout
> > > >
> > > > anv: Handle transitioning depth from UNDEFINED to other
> > > > layouts
> > > >
> > > > anv/image: Get rid of the memset(aux, 0, sizeof(aux))
> > > > hack
> > > >
> > > > anv: Predicate 48bit support on gen >= 8
> > > >
> > > > anv: Set up memory types and heaps during physical device
> > > > init
> > > >
> > > > anv: Set image memory types based on the type count
> > > >
> > > > i965/blorp: Do and end-of-pipe sync on both sides of
> > > > fast-clear ops
> > > >
> > > > i965: Round copy size to the nearest block in
> > > > intel_miptree_copy
> > > >
> > > >
> > > >
> > > > Juan A. Suarez Romero (1):
> > > >
> > > > Revert "android: fix segfault within swap_buffers"
> > > >
> > > >
> > > >
> > > > Marek Olšák (1):
> > > >
> > > > radeonsi/gfx9: compile shaders with +xnack
> > > >
> > > >
> > > >
> > > > Nicolai Hähnle (1):
> > > >
> > > > st/mesa: remove redundant stfb->iface checks
> > > >
> > > >
> > > >
> > > > Nicolas Boichat (1):
> > > >
> > > > configure.ac: Also match -androideabi tuple
> > > >
> > > >
> > > >
> > > > Rob Clark (1):
> > > >
> > > > freedreno: fix fence creation fail if no rendering
> > > >
> > > >
> > > >
> > > > Tapani Pälli (1):
> > > >
> > > > egl/android: fix segfault within swap_buffers
> > > >
> > > >
> > > >
> > > > Timothy Arceri (1):
> > > >
> > > > st/mesa: don't mark the program as in cache_fallback when
> > > > there is cache miss
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Rejected (9)
> > > >
> > > > ============
> > > >
> > > > Dave Airlie(1):
> > > >
> > > > radv: fix regression in descriptor set freeing.
> > > >
> > > >
> > > >
> > > > This commit addressed an earlier commit 126d5ad which did not
> > > > land in branch.
> > > >
> > > >
> > > >
> > > > Jason Ekstrand(7):
> > > >
> > > > anv: Determine the type of mapping based on type
> > > > metadata
> > > >
> > > > anv: Stop setting BO flags in bo_init_new
> > > >
> > > > anv: Make supports_48bit_addresses a heap property
> > > >
> > > > anv: Advertise both 32-bit and 48-bit heaps when we have
> > > > enough memory
> > > >
> > > > anv: Require vertex buffers to come from a 32-bit heap
> > > >
> > > > anv: Add valid_bufer_usage to the memory type metadata
> > > >
> > > > anv: Refactor memory type setup
> > > >
> > > >
> > > >
> > > > All these patches have dependencies among them, and also about
> > > > commits which did not
> > > >
> > > > land in branch, like c3c61d210f or 35e626bd0e.
> > >
> > > c3c61d210f should be back-ported to stable as well. I'm aware of
> > > the dependency on 35e626bd0e but it should be fairly easy to
> > > rebase on top of it by just dropping all of the EXEC_ASYNC bits
> > > from "anv: Stop setting BO flags in bo_init_new". I'm happy to
> > > do the rebase for you if you'd like.
> > >
> >
> >
> > Thanks for info, Jason.
> >
> > I've been working hard to try to include them in Queued list.
> > Finally, I added all of the ANV commits, except the one entitled
> > "anv: Require vertex buffers to come from a 32-bit heap". I keep
> > this one in the Rejected list because it was causing a regression
> > in the Crucible test suite. The other 6 are included in queued.
>
> Uh... That's the important one. All of the others lead up to it.
> What is the crucible regression? I was pretty sure crucible master
> (you may have to pull) was working fine.
>
Mark reported around 50 crashes in the crucible testsuite.
Some of them are:
crucible.func.depthstencil.basic-depth.clear-0.0.op-greater
crucible.func.miptree.d32-sfloat.aspect-depth.view-
2d.levels01.array01.extent-1024x512.upload-copy-from-buffer.download-
copy-with-draw
crucible.func.push-constants.basic
crucible.func.shader.fragcoord
Now, you're right: using Crucible master indeed it is working fine (at
least on my local host). I guess Mark is using instead commit
4f60bf1,
which indeed has those regressions. But couldn't confirm with Mark.
In this case, I can include the patch in the queue, ask Mark to update
the used crucible test, and re-try again. If everything goes fine, I'll
do the release. As we are in weekend, I guess I won't get the results
until Monday.
Does this sound fine for you?
J.A.
> > Additionally, I've enqueued the one entitled "anv: Set
> > EXEC_OBJECT_ASYNC when available" squashed with "anv/tests: Create
> > a dummy instance as well as device".
> >
> > All those new commits caused the following new trivial merge
> > conflicts:
> >
> > commit 2562b3252b63f2e938d6e6614e1f9a2fa2064140
> > Author: Jason Ekstrand <jason.ekstr...@intel.com>
> >
> > anv: Make supports_48bit_addresses a heap property
> >
> > (cherry picked from commit
> > b83b1af6f6936f36db42a8f8b8e0854d0f9491fd)
> >
> > commit 86a8854b118fd42d157a5b640fa7a3a1cad92301
> > Author: Jason Ekstrand <jason.ekstr...@intel.com>
> >
> >
> > anv: Stop setting BO flags in bo_init_new
> >
> >
> > (cherry picked from commit
> > 00df1cd9d6234cdfc9fb2bf3615196ff83a3c956)
> >
> > commit 0f042901e3cd451f9a7630376083a805328aebe3
> > Author: Jason Ekstrand <jason.ekstr...@intel.com>
> >
> > anv: Add valid_bufer_usage to the memory type metadata
> >
> > (cherry picked from commit
> > f7736ccf53eaeb66c4270afe0916e2cb29ab8667)
> >
> > commit 15bc6d4d210eee051407a816811012eba0a3be3b
> > Author: Jason Ekstrand <jason.ekstr...@intel.com>
> >
> > anv: Determine the type of mapping based on type metadata
> >
> > (cherry picked from commit
> > 92325a7efc769c32e03031323e21700dc55171e4)
> >
> >
> > All these news changes implies that I need to wait for Intel CI
> > results to ensure nothing is broken, which means I need to delay
> > the release
> > probably until Monday (or sooner if I get the results before).
> >
> > Sorry for the inconveniences!
> >
> >
> > > >
> > > > Nicolai Hähnle(1):
> > > >
> > > > radeonsi: load patch_id for TES-as-ES when exporting for
> > > > PS
> > > >
> > > >
> > > >
> > > > This commit depends on 9fd9a7d0ba3 and 678d568c7b2, neither of
> > > > which is in branch.
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > _______________________________________________
> > > >
> > > > mesa-dev mailing list
> > > >
> > > > mesa-dev@lists.freedesktop.org
> > > >
> > > > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> > > >
> > > >
> > >
> > >
> > > _______________________________________________
> > > mesa-dev mailing list
> > > mesa-dev@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
>
>
>
> _______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
