Re: [Mesa-dev] [PATCH] i965: Set "Subslice Hashing Mode" to 16x16 on Apollolake.

Thanks Eero! On Tue, 2017-08-29 at 12:26 +0300, Eero Tamminen wrote: > Hi, > > On 28.08.2017 17:33, Andres Gomez wrote: > > Kenneth, would we want this patch in 17.1 or we shouldn't bother ? > > See this for more extensive info on its impact: > https://bugs.freedes

Re: [Mesa-dev] [PATCH] i965: Set "Subslice Hashing Mode" to 16x16 on Apollolake.

Kenneth, would we want this patch in 17.1 or we shouldn't bother ? On Tue, 2017-05-30 at 16:28 -0700, Kenneth Graunke wrote: > As of 4.11, the kernel isn't bothering to set the subslice hashing mode > on Apollolake, leaving it at the default of 8x8. (It initializes it to > 16x4 on most

[Mesa-dev] [ANNOUNCE] mesa 17.1.8

includes a patch to prevent a potential crash. nouveau's has gotten a fix to properly set the sType for TXF ops to U32. For EGL, the Wayland platform of the the DRI2 drivers has received a fix to prevent a possible indefinite block. Andres Gomez (7):   docs: add sha256 checksums for 17.1.7

[Mesa-dev] Mesa 17.1.8 release candidate

  (cherry picked from commit 87556a650ad363b41d86f4e25d5c4696f9af4550) Cheers, Andres Mesa stable queue - Nominated (0) = Queued (15) === Andres Gomez (5):   docs: add sha256 checksums for 17.1.7   cherry-ignore: cherry-ignore: added 17.2 nominations.   cherry-

[Mesa-dev] [PATCH 2/2] docs: add an additional final cycle for 17.1

Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Juan A. Suarez Romero <jasua...@igalia.com> Signed-off-by: Andres Gomez <ago...@igalia.com> --- docs/release-calendar.html | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/docs/release-calendar.h

[Mesa-dev] [PATCH 1/2] docs: remove released and extend the calendar until the end of 2017

Completed the 17.2 cycle and added the beginning of the 17.3 one. Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Juan A. Suarez Romero <jasua...@igalia.com> Signed-off-by: Andres Gomez <ago...@igalia.com> --- docs/release-calendar.html | 86 +++

[Mesa-dev] [PATCH 0/2] docs: update the release calendar until the end of 2017

The first email updates the calendar with a proposal for the future releases until the end of 2017. It also removes versions that have already been released. The second patch is a proposal to add yet another final iteration to the 17.1 cycle. Andres Gomez (2): docs: remove released and extend

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/2] i965: Make a BRW_NEW_FAST_CLEAR_COLOR dirty bit.

Kenneth, for 17.1, this patch depends on f9fd976e8adb and others. If you still want to add it for 17.1, I think we would need a backport. Br. On Thu, 2017-08-17 at 15:16 -0700, Kenneth Graunke wrote: > When changing fast clear colors, we need to emit new SURFACE_STATE > with the updated color

Re: [Mesa-dev] [PATCH] ac/nir: fixup layer/viewport export for GFX9.

Hi Dave, This patch landed tagged for 17.2 only. Was it, then, not nominated for 17.1 intentionally ? Br. On Thu, 2017-08-17 at 14:27 +1000, Dave Airlie wrote: > From: Dave Airlie > > GFX9 moved where the viewport index export goes. > > Signed-off-by: Dave Airlie

Re: [Mesa-dev] [Mesa-stable] [PATCH] i965/tex: Don't pass samples to miptree_create_for_teximage

Jason, 76e2f390f9863a35 didn't land in 17.1 so I understand this is not really needed there and we only need it for 17.2. WDYT? On Sat, 2017-08-19 at 11:07 -0700, Jason Ekstrand wrote: > In 76e2f390f9863a35, when Topi switched num_samples from 0 to 1 for > single-sampled, he accidentally

Re: [Mesa-dev] [PATCH] isl: Validate row pitch of stencil surfaces.

On Thu, 2017-08-10 at 12:51 -0700, Jason Ekstrand wrote: > On August 10, 2017 12:45:43 PM Ilia Mirkin wrote: > > > On Wed, Aug 9, 2017 at 4:09 PM, Kenneth Graunke > > wrote: > > > Also, silence an obnoxious finishme that started occurring for all >

[Mesa-dev] [ANNOUNCE] mesa 17.1.7

led. EGL has gotten a fix to avoid a crash if an invalid display is used. In the DRI2 drivers, the X11 platform received a fix to prevent leaking xfixes_query in the error path. From build and integration point of view, we have added a fix to solve building on old glibc. Andres Gomez

Re: [Mesa-dev] Mesa 17.1.7 release candidate

On Sun, 2017-08-20 at 07:19 -0700, Jason Ekstrand wrote: > On August 19, 2017 8:38:04 AM Andres Gomez <ago...@igalia.com> wrote: > [...] > > > > Mesa stable queue > > - > > > > Nominated (3) > > = > > > &

Re: [Mesa-dev] Mesa 17.1.7 release candidate

On Sat, 2017-08-19 at 17:56 -0400, Ilia Mirkin wrote: > On Sat, Aug 19, 2017 at 11:37 AM, Andres Gomez <ago...@igalia.com> wrote: > > nouveau's nv50 has gotten a fix for a problem involving ConstantFolding > > with saturation that was visible in Hitman Absolution and, possib

Re: [Mesa-dev] Mesa 17.1.7 release candidate

On Sat, 2017-08-19 at 21:40 +0100, Emil Velikov wrote: > [Slimming down the CC list] > > Hi Andres, > > On 19 August 2017 at 16:37, Andres Gomez <ago...@igalia.com> wrote: > > > 4c09f648929fbde25e4be44fa2983c0f3ac2fd11 > > commit 774e77ab64de05d43

[Mesa-dev] Mesa 17.1.7 release candidate

(1): i965: Stop looking at NewDriverState when emitting 3DSTATE_URB Queued (33) === Andres Gomez (7): cherry-ignore: add "swr: use the correct variable for no undefined symbols" cherry-ignore: add "radeon/ac: use ds_swizzle for derivs on si/cik."

Re: [Mesa-dev] [PATCH] radeonsi/gfx9: use the VI codepath for clamping Z

Hi Marek, This patch landed tagged for 17.2 stable and has been collected for 17.2.0-rc4. However, it seems like it could be also interesting for 17.1.x (?) WDYT? Br. On Thu, 2017-08-10 at 22:33 +0200, Marek Olšák wrote: > From: Marek Olšák > > This fixes corrupted

Re: [Mesa-dev] [PATCH] radeon/ac: use ds_swizzle for derivs on si/cik.

Hi Dave, this patch has been collected for 17.2.0-rc3 but, for 17.1.x, the conflicts are not so trivial and the landed commit was tagged for stable without specifying the actual branch. Therefore, unless you intended this also for 17.1.x and would like to provide a backport, I'm more keen on

Re: [Mesa-dev] [Mesa-stable] [PATCH 3/4] ac/nir: fix 64-bit shifts

so we > can catch stuff like this easier. > > On Wed, Aug 16, 2017 at 8:49 PM, Andres Gomez <ago...@igalia.com> wrote: > > Connor, it looks like we could want this patch in 17.1 (?) > > > > On Fri, 2017-06-30 at 19:56 -0700, Connor Abbott wrote: > > >

Re: [Mesa-dev] [PATCH 3/4] ac/nir: fix 64-bit shifts

Connor, it looks like we could want this patch in 17.1 (?) On Fri, 2017-06-30 at 19:56 -0700, Connor Abbott wrote: > From: Connor Abbott > > NIR always makes the shift amount 32 bits, but LLVM asserts if the two > sources aren't the same type. Zero-extend the shift amount

Re: [Mesa-dev] [Mesa-stable] [PATCH 3/3] swr: use the correct variable for no undefined symbols

Emil, this patch breaks SWR build in 17.1, although it states that fixes 9475251145174882b532. Should I assume that this should be dropped for 17.1 or are you planning to send some backport (?). On Fri, 2017-07-21 at 19:05 +0100, Emil Velikov wrote: > From: Emil Velikov

Re: [Mesa-dev] [PATCH 3/4] ac/nir: fix 64-bit shifts

Connor, it looks like we could want this patch in 17.1 (?) On Fri, 2017-06-30 at 19:56 -0700, Connor Abbott wrote: > From: Connor Abbott > > NIR always makes the shift amount 32 bits, but LLVM asserts if the two > sources aren't the same type. Zero-extend the shift amount

Re: [Mesa-dev] [PATCH 2/2] egl: add linux-dmabuf-unstable-v1-protocol.c to "nodist"

I'll be pushing already 1/2 and keep this on hold to see whether we favor instead: https://lists.freedesktop.org/archives/mesa-dev/2017-July/163730.html On Thu, 2017-07-20 at 01:44 +0300, Andres Gomez wrote: > This fixes `make distcheck` > > > make[3]: *** No rule to make target

Re: [Mesa-dev] [PATCH] configure.ac: rework wayland-protocols handling

On Mon, 2017-07-24 at 01:27 +0300, Andres Gomez wrote: > This is: > > Reviewed-by: Andres Gomez <ago...@igalia.com> Mmmm ... I hit the send button too quickly. Just wanted to mention that I share the same concerns than Daniel. Would it be up to me, I would not be adding the

Re: [Mesa-dev] [PATCH] configure.ac: rework wayland-protocols handling

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Fri, 2017-07-21 at 13:02 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > At dist/distcheck time we need to ensure that all the files and their > respective dependencies are han

Re: [Mesa-dev] [PATCH 2/2] egl: add linux-dmabuf-unstable-v1-protocol.c to "nodist"

On Thu, 2017-07-20 at 14:37 +0100, Emil Velikov wrote: > On 20 July 2017 at 13:54, Daniel Stone <dan...@fooishbar.org> wrote: > > Hi Emil, > > > > On 20 July 2017 at 13:51, Emil Velikov <emil.l.veli...@gmail.com> wrote: > > > On 19 July 2017 at 23:

[Mesa-dev] [PATCH 1/2] broadcom: correct header file in BROADCOM_FILES

a/src' > make[2]: *** [distdir] Error 1 > make[1]: *** [distdir] Error 1 Fixes: 42799c ("broadcom: Introduce a header for talking about chip revisions.") Cc: Emil Velikov <emil.veli...@collabora.com> Signed-off-by: Andres Gomez <ago...@igalia.com> --- src/broadcom

[Mesa-dev] [PATCH 2/2] egl: add linux-dmabuf-unstable-v1-protocol.c to "nodist"

[distdir] Error 1 > make[1]: *** [distdir] Error 1 > make: *** [dist] Error 2 Fixes: 02cc359372 ("egl/wayland: Use linux-dmabuf interface for buffers") Cc: Emil Velikov <emil.veli...@collabora.com> Signed-off-by: Andres Gomez <ago...@igalia.com> --- src/egl/Makefile.am

[Mesa-dev] [PATCH 0/2] Fix distcheck

A couple of patches to fix recent problems while running make distcheck. Andres Gomez (2): broadcom: correct header file in BROADCOM_FILES egl: add linux-dmabuf-unstable-v1-protocol.c to "nodist" src/broadcom/Makefile.sources | 2 +- src/egl/Makefile.am | 10 -

[Mesa-dev] [PATCH] travis: add missing wayland-protocols

-config man page for more details. Also, added extra path to PKG_CONFIG_PATH env variable. Signed-off-by: Andres Gomez <ago...@igalia.com> --- .travis.yml | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index fa52bf96f1..246ad30eff

Re: [Mesa-dev] [PATCH 3/4] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

On Wed, 2017-07-19 at 13:19 +0100, Emil Velikov wrote: > On 18 July 2017 at 14:05, Andres Gomez <ago...@igalia.com> wrote: > > On Tue, 2017-07-18 at 10:48 +0100, Emil Velikov wrote: > > > On 15 July 2017 at 17:01, Andres Gomez <ago...@igalia.com> wrote: > > &

Re: [Mesa-dev] [PATCH 3/4] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

On Tue, 2017-07-18 at 10:48 +0100, Emil Velikov wrote: > On 15 July 2017 at 17:01, Andres Gomez <ago...@igalia.com> wrote: > > On Mon, 2017-07-10 at 12:19 +0100, Emil Velikov wrote: > > > On 8 July 2017 at 20:59, Andres Gomez <ago...@igalia.com> wrote: > >

[Mesa-dev] [PATCH 2/3] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

The LD_LIBRARY_PATH environment variable could be already defined so we extend it and restore it rather than just overwriting it. v2: - Unset the __old_ld helper variable when we are done with it. - Corrected test for and escaping of variables (Eric). Signed-off-by: Andres Gomez <

[Mesa-dev] [PATCH 3/3] docs: update master's release notes, news and calendar commit

This reflects closer what we are actually doing. Signed-off-by: Andres Gomez <ago...@igalia.com> Reviewed-by: Eric Engestrom <eric.engest...@imgtec.com> Reviewed-by: Emil Velikov <emil.veli...@collabora.com> --- docs/releasing.html | 15 --- 1 file changed, 4

[Mesa-dev] [PATCH 1/3] docs: add instructions to specify LLVM version for basic testing

d the need to define LLVM_CONFIG also before running the release.sh script. Signed-off-by: Andres Gomez <ago...@igalia.com> Reviewed-by: Eric Engestrom <eric.engest...@imgtec.com> Reviewed-by: Emil Velikov <emil.veli...@collabora.com> --- docs/releasing.html | 8 1 fil

[Mesa-dev] [PATCH 0/3] docs: update basic testing on the releasing instructions (v2)

e actually doing. Andres Gomez (3): docs: add instructions to specify LLVM version for basic testing docs: avoid overwrite of LD_LIBRARY_PATH during basic testing docs: update master's release notes, news and calendar commit docs/releasing.html | 27 +++ 1 file c

Re: [Mesa-dev] [PATCH 2/4] docs: remove redundant parameter for scons build basic testing

On Mon, 2017-07-10 at 12:22 +0100, Emil Velikov wrote: > On 8 July 2017 at 20:59, Andres Gomez <ago...@igalia.com> wrote: > > toolchain=crossmingw is the default option for platform=windows when > > invoking scons. > > > > It may be a bit redundant, but I'

Re: [Mesa-dev] [PATCH 3/4] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

On Mon, 2017-07-10 at 12:19 +0100, Emil Velikov wrote: > On 8 July 2017 at 20:59, Andres Gomez <ago...@igalia.com> wrote: > > The LD_LIBRARY_PATH environment variable could be already defined so > > we extend it and restore it rather than just overwriting it. > > &

Re: [Mesa-dev] [PATCH 3/4] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

On Mon, 2017-07-10 at 12:15 +0100, Eric Engestrom wrote: > On Saturday, 2017-07-08 22:59:40 +0300, Andres Gomez wrote: > > The LD_LIBRARY_PATH environment variable could be already defined so > > we extend it and restore it rather than just overwriting it. > > > > Signe

[Mesa-dev] [ANNOUNCE] mesa 17.1.5

ap intrinsics Andres Gomez (4):   docs: add sha256 checksums for 17.1.4   cherry-ignore: i965: Fix anisotropic filtering for mag filter   Update version to 17.1.5   docs: add release notes for 17.1.5 Anuj Phogat (2):   intel/isl: Use uint64_t to store total surface size   i

Re: [Mesa-dev] Mesa 17.1.5 release candidate

Hi, On Thu, 2017-07-13 at 12:39 +0900, Michel Dänzer wrote: > Hi Andres, > > > On 13/07/17 04:02 AM, Andres Gomez wrote: > > > > Gallivm has seen a fix for properly checking the line width in order to > > use the wide line stage. The VA library, has seen fixed

[Mesa-dev] Mesa 17.1.5 release candidate

ac/nir: Fix ordering of parameters for image atomic cmpswap intrinsics Andres Gomez (2): docs: add sha256 checksums for 17.1.4 cherry-ignore: i965: Fix anisotropic filtering for mag filter Anuj Phogat (2): intel/isl: Use uint64_t to store total surface size intel/isl: Ad

Re: [Mesa-dev] [PATCH v3 1/3] spirv: Fix reaching unreachable for compare exchange on images

James, it doesn't seem like this patch has landed in master. Are you in need of review or is it that this has been superseded? Thanks! On Mon, 2017-06-26 at 10:46 +0100, James Legg wrote: > We were hitting the > unreachable("Invalid image opcode") > near the end of vtn_handle_image when

Re: [Mesa-dev] [PATCH] meta: Fix BlitFramebuffer temp texture setup

Ville, has this patch fallen through the cracks ? On Fri, 2017-06-23 at 14:58 +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Pass the correct src coordinates to CopyTexSubImage() > when creating the temporary texture, and also take care to

Re: [Mesa-dev] [PATCH] i965/blorp: Use the renderbuffer format for clears

Jason, which is the status of this patch? Has it been superseded or discarded? On Mon, 2017-06-26 at 09:01 -0700, Jason Ekstrand wrote: > This fixes the Piglit ARB_texture_views rendering-formats test. > > Cc: "17.1" > --- >

Re: [Mesa-dev] [Mesa-stable] [PATCH] nir: copy front interpolation when creating fake back color input

oblematic condition can't > happen there.) > > On Mon, Jul 10, 2017 at 1:30 PM, Andres Gomez <ago...@igalia.com> wrote: > > Ilia, would we want this patch in -stable ? > > > > On Fri, 2017-07-07 at 20:34 -0400, Ilia Mirkin wrote: > > > Fixes a bunch of

Re: [Mesa-dev] [Mesa-stable] [PATCH] gallium: improve selection of pixel format

algorithm seems to be in line with the behavior of most Windows > drivers (Microsoft, NVIDIA, AMD) and, more important, I can't imagine > a sensible scenario where this change would break an existing > application. > > -Olivier > > -Message d'origine- > De : Andres Go

Re: [Mesa-dev] [PATCH] nir: copy front interpolation when creating fake back color input

Ilia, would we want this patch in -stable ? On Fri, 2017-07-07 at 20:34 -0400, Ilia Mirkin wrote: > Fixes a bunch of gl_BackColor interpolation tests that had explicit > interpolation specified on the fragment shader gl_Color. > > Signed-off-by: Ilia Mirkin > --- >

Re: [Mesa-dev] [Mesa-stable] [Fwd: Re: [PATCH 1/2] intel/isl: Use uint64_t to store total surface size]

the sake of > completion. > Thanks for marking them for mesa-stable. > > > On Fri, Jul 7, 2017 at 7:25 AM, Andres Gomez <ago...@igalia.com> wrote: > > On Thu, 2017-07-06 at 18:21 +0100, Emil Velikov wrote: > > > On 3 July 2017 at 21:14, Andres Gomez <ag

Re: [Mesa-dev] [Mesa-stable] [PATCH 03/14] gallium/radeon: fix a possible crash for buffer exports

On Sun, 2017-07-09 at 14:29 +0200, Marek Olšák wrote: > On Sat, Jul 8, 2017 at 4:07 PM, Andres Gomez <ago...@igalia.com> wrote: > > Marek, do we want this into -stable? > > Yes, this one can go to stable if it applies cleanly. It does. Thanks for the feedbac

Re: [Mesa-dev] [PATCH] drirc: Add glsl_correct_derivatives_after_discard for The Witcher 2

On Sun, 2017-07-09 at 14:23 +0200, Marek Olšák wrote: > On Sat, Jul 8, 2017 at 3:58 PM, Andres Gomez <ago...@igalia.com> wrote: > > Related to my previous mail, do we try to pick this for -stable? > > This one actually fixes a very visible incorrect rendering in Witcher >

Re: [Mesa-dev] [PATCH 0/5] Gallium: Passing drirc options to create_screen() and fixing Rocket League

On Sun, 2017-07-09 at 14:21 +0200, Marek Olšák wrote: > On Sat, Jul 8, 2017 at 3:56 PM, Andres Gomez <ago...@igalia.com> wrote: > > Marek, is it worth the effort of trying to bring this series into > > -stable? > > Probably not. The incorrect rendering in Rocket Leagu

Re: [Mesa-dev] [PATCH 1/2] vbo: fix glPrimitiveRestartNV crash inside a display list

It looks like we could want this series into -stable (?) On Fri, 2017-07-07 at 08:10 -0600, Brian Paul wrote: > From: Olivier Lauffenburger > > glPrimitiveRestartNV crashes when it is called during the compilation > of a display list. > > There are two reasons: >

Re: [Mesa-dev] [PATCH] gallium: improve selection of pixel format

Olivier, Brian, do we want this into -stable? On Thu, 2017-07-06 at 17:08 +0200, Olivier Lauffenburger wrote: > Current selection of pixel format does not enforce the request of > stencil or depth buffer if the color depth is not the same as > requested. > For instance, GLUT requests a 32-bit

[Mesa-dev] [PATCH 3/4] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

The LD_LIBRARY_PATH environment variable could be already defined so we extend it and restore it rather than just overwriting it. Signed-off-by: Andres Gomez <ago...@igalia.com> --- docs/releasing.html | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/docs/releasing.

[Mesa-dev] [PATCH 4/4] docs: update master's release notes, news and calendar commit

This reflects closer what we are actually doing. Signed-off-by: Andres Gomez <ago...@igalia.com> --- docs/releasing.html | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/docs/releasing.html b/docs/releasing.html index 99235d8412..152f5cea73 100644 ---

[Mesa-dev] [PATCH 2/4] docs: remove redundant parameter for scons build basic testing

toolchain=crossmingw is the default option for platform=windows when invoking scons. Signed-off-by: Andres Gomez <ago...@igalia.com> --- docs/releasing.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/releasing.html b/docs/releasing.html index ceb6e365e4..8e6e

[Mesa-dev] [PATCH 1/4] docs: add instructions to specify LLVM version for basic testing

The "Perform basic testing" section provides some instructions to do so. We add now some comments in order to use a recent enough LLVM version to test the distcheck and the automake generated binaries. Signed-off-by: Andres Gomez <ago...@igalia.com> --- docs/releasing.html |

[Mesa-dev] [PATCH 0/4] docs: update basic testing on the releasing instructions

Some small update on the "Making a new release" section of the releasing instructions. Patches 1-3 add some small modifications for performing the basic testing. Patch 4 merges the section for updating the calendar into the previous one to reflect closer what we are actually doing. An

Re: [Mesa-dev] [Mesa-stable] [PATCH] etnaviv: don't dereference etna_resource pointer if allocation fails

On Sat, 2017-07-08 at 18:45 +0200, Christian Gmeiner wrote: > 2017-07-08 14:42 GMT+02:00 Andres Gomez <ago...@igalia.com>: > > It looks like we could want this into -stable (?) > > Forgot to add CC during push but I have nominated it for stable now. Great! Thanks a lo

Re: [Mesa-dev] [Mesa-stable] [PATCH] mesa: Require mipmap completeness for glCopyImageSubData() at times.

On Sat, 2017-07-08 at 07:52 -0700, Kenneth Graunke wrote: > On Saturday, July 8, 2017 7:01:30 AM PDT Andres Gomez wrote: > > Kenneth, worth cherry-picking for -stable? > > I wouldn't - this patch just imposes additional restrictions, making > things we used to support il

Re: [Mesa-dev] [PATCH 03/14] gallium/radeon: fix a possible crash for buffer exports

Marek, do we want this into -stable? On Thu, 2017-06-29 at 21:47 +0200, Marek Olšák wrote: > From: Marek Olšák > > --- > src/gallium/drivers/radeon/r600_texture.c | 24 +++- > 1 file changed, 15 insertions(+), 9 deletions(-) > > diff --git

Re: [Mesa-dev] [PATCH 1/2] st/mesa: check for incomplete texture in st_finalize_texture()

Brian, it looks like we could want this series into -stable (?) On Wed, 2017-06-28 at 16:12 -0600, Brian Paul wrote: > Return early from st_finalize_texture() if we have an incomplete > texture. This avoids trying to create a texture resource with invalid > parameters (too many mipmap levels

Re: [Mesa-dev] [PATCH] mesa: Require mipmap completeness for glCopyImageSubData() at times.

Kenneth, worth cherry-picking for -stable? On Wed, 2017-06-28 at 02:44 -0700, Kenneth Graunke wrote: > This patch makes glCopyImageSubData require mipmap completeness when the > texture object's built-in sampler object has a mipmapping MinFilter. > This is apparently the de facto behavior and

Re: [Mesa-dev] [PATCH] drirc: Add glsl_correct_derivatives_after_discard for The Witcher 2

Related to my previous mail, do we try to pick this for -stable? On Wed, 2017-06-21 at 22:40 +0200, Edmondo Tommasina wrote: > This fixes the long-standing problem with black transitions in The Wicher 2. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98238 > --- > This patch depends

Re: [Mesa-dev] [PATCH 0/5] Gallium: Passing drirc options to create_screen() and fixing Rocket League

Marek, is it worth the effort of trying to bring this series into -stable? On Wed, 2017-06-21 at 00:54 +0200, Marek Olšák wrote: > Hi, > > This series updates pipe loaders so that flags such as drirc options > can be passed to create_screen(). I have compile-tested everything > except clover. >

Re: [Mesa-dev] [PATCH] etnaviv: don't dereference etna_resource pointer if allocation fails

It looks like we could want this into -stable (?) On Thu, 2017-07-06 at 11:35 +0200, Aleksander Morgado wrote: > The check for the pointer being non-NULL was being done too late. > > Signed-off-by: Aleksander Morgado > --- >

[Mesa-dev] [PATCH] nir/spirv: Remove unnecessary comment.

It should have been removed after 00c47e111c. Cc: Jason Ekstrand <ja...@jlekstrand.net> Cc: Connor Abbott <cwabbo...@gmail.com> Signed-off-by: Andres Gomez <ago...@igalia.com> --- src/compiler/spirv/spirv_to_nir.c | 5 - 1 file changed, 5 deletions(-) diff --git a/

Re: [Mesa-dev] [PATCH 1/3] mesa: GL_TEXTURE_BORDER_COLOR exists in OpenGL 1.0, so don't depend on GL_ARB_texture_border_clamp

Ian, it looks like we could want this patch (and the others from the series when they land) in -stable (?) On Tue, 2017-06-27 at 10:09 -0700, Ian Romanick wrote: > From: Ian Romanick > > On NV20 (and probably also on earlier NV GPUs that lack >

Re: [Mesa-dev] [PATCH] swr: invalidate attachment on transition change

George, would we want this patch in -stable or we shouldn't bother ? On Tue, 2017-06-20 at 11:42 -0500, George Kyriazis wrote: > Consider the following RT attachment order: > 1. Attach surfaces attachments 0 & 1, and render with them > 2. Detach 0 & 1 > 3. Re-attach 0 & 1 to different surfaces >

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/8] mesa: check for allocation failures in _mesa_new_texture_object()

On Fri, 2017-07-07 at 09:14 +0200, Samuel Pitoiset wrote: > > On 07/06/2017 08:06 PM, Andres Gomez wrote: > > It looks like we could want patches 1 and 3-8 from this series into > > -stable (?) > > These are not critical fixes, your call. OK. I will leave them o

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/4] radeonsi: fix DCC fast clear for luminance and alpha formats

On Thu, 2017-07-06 at 23:24 +0200, Marek Olšák wrote: > On Thu, Jul 6, 2017 at 8:24 PM, Andres Gomez <ago...@igalia.com> wrote: > > Marek, would we want this series in -stable or we shouldn't bother ? > > Don't bother. Patch 1 isn't that important. Other patches are for &

Re: [Mesa-dev] [Mesa-stable] [Fwd: Re: [PATCH 1/2] intel/isl: Use uint64_t to store total surface size]

On Thu, 2017-07-06 at 18:21 +0100, Emil Velikov wrote: > On 3 July 2017 at 21:14, Andres Gomez <ago...@igalia.com> wrote: > > Actually, forgot to add -stable into CC. > > > > Forwarded Message > > From: Andres Gomez <ago...@igalia.com> >

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/3] mesa/main: Move NULL pointer check.

On Thu, 2017-07-06 at 18:02 +0100, Emil Velikov wrote: > On 6 July 2017 at 15:35, Andres Gomez <ago...@igalia.com> wrote: > > It looks like we could want this into -stable (?) > > > > _mesa_update_draw_buffer_bounds is a no-op if the buffer pointer is NULL. > There

Re: [Mesa-dev] [Mesa-stable] [st/va] Fix leak in VAAPI subpictures

It looks like we could want this into -stable (?) On Thu, 2017-07-06 at 21:10 +0300, Andres Gomez wrote: > On Thu, 2017-06-22 at 09:25 +, Namburu, Chandu-babu wrote: > > From: Chandu Babu N <cha...@amd.com> > > Subject: [PATCH] [st/va] Fix leak in VAAPI subpictures &

Re: [Mesa-dev] [st/va] Fix leak in VAAPI subpictures

It looks like we could want this into -stable (?) On Thu, 2017-07-06 at 21:10 +0300, Andres Gomez wrote: > On Thu, 2017-06-22 at 09:25 +, Namburu, Chandu-babu wrote: > > From: Chandu Babu N <cha...@amd.com> > > Subject: [PATCH] [st/va] Fix leak in VAAPI subpictures &

Re: [Mesa-dev] [PATCH 1/4] radeonsi: fix DCC fast clear for luminance and alpha formats

Marek, would we want this series in -stable or we shouldn't bother ? On Tue, 2017-06-20 at 19:06 +0200, Marek Olšák wrote: > From: Marek Olšák > > I reproduced this bug on Polaris11 and Raven. > > I can't get this bug on Fiji. The reason might be that Fiji doesn't use > 2D

Re: [Mesa-dev] [st/va] Fix leak in VAAPI subpictures

On Thu, 2017-06-22 at 09:25 +, Namburu, Chandu-babu wrote: > From: Chandu Babu N > Subject: [PATCH] [st/va] Fix leak in VAAPI subpictures >   > sampler view allocated in vaAssociateSubpicture is not cleared > in vaiDeassociateSubpicture. >   > Reviewed-by: Christian König

Re: [Mesa-dev] [PATCH 1/8] mesa: check for allocation failures in _mesa_new_texture_object()

It looks like we could want patches 1 and 3-8 from this series into -stable (?) On Wed, 2017-06-21 at 11:04 +0200, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/mesa/main/texobj.c | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) >

Re: [Mesa-dev] [PATCH 1/3] mesa/main: Move NULL pointer check.

It looks like we could want this into -stable (?) On Wed, 2017-06-14 at 19:33 +0300, Plamena Manolova wrote: > In blit_framebuffer we're already doing a NULL > pointer check for readFb and drawFb so it makes > sense to do it before we actually use the pointers. > > CID: 1412569 > Signed-off-by:

Re: [Mesa-dev] [PATCH] glsl: gl_Max{Vertex, Fragment}UniformComponents exist in all desktop GL versions

It looks like we could want this into -stable (?) On Fri, 2017-06-16 at 12:05 +0200, Iago Toral Quiroga wrote: > The current implementation assumed that these were replaced in GLSL >= 4.10 > by gl_Max{Vertex,Fragment}UniformVectors, however this is not true: both > built-ins should be produced

Re: [Mesa-dev] [Mesa-stable] [PATCH 00/16] anv: Fix our 48-bit problems

On Mon, 2017-07-03 at 22:48 -0700, Jason Ekstrand wrote: > On July 3, 2017 12:31:48 PM Andres Gomez <ago...@igalia.com> wrote: > > > Never too late! ☺ > > > > On Fri, 2017-06-30 at 16:45 -0700, Jason Ekstrand wrote: > > > I know this is rather late but...

Re: [Mesa-dev] [PATCH 1/2] i965: Use true AA line distance on G45/Ironlake.

It looks like we could want these 2 into -stable (?) On Wed, 2017-04-26 at 15:15 -0700, Kenneth Graunke wrote: > The original Broadwater and Crestline platforms computed antialiased > line distances using "manhattan" distance, aka a + b = c. Eaglelake > and Cantiga added "true" distance, aka a^2

Re: [Mesa-dev] [PATCH 1/6] i965: Make Gen4-5 SF_STATE use the point size calculations from Gen6+.

It looks like we could want this whole series into -stable (?) On Wed, 2017-05-10 at 11:47 -0700, Kenneth Graunke wrote: > Apparently, Nanhai made the Gen4-5 point size calculations round to the > nearest integer in commit 8d5231a3582e4f2769ac0685cf0174e09750700e, > "according to spec". When

Re: [Mesa-dev] [PATCH 1/2] intel/isl: Use uint64_t to store total surface size

It looks like we could want these 2 into -stable (?) On Tue, 2017-05-23 at 14:37 -0700, Anuj Phogat wrote: > Signed-off-by: Anuj Phogat > --- > src/intel/isl/isl.c | 3 ++- > src/intel/isl/isl.h | 2 +- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git

Re: [Mesa-dev] [Mesa-stable] [PATCH 00/16] anv: Fix our 48-bit problems

Never too late! ☺ On Fri, 2017-06-30 at 16:45 -0700, Jason Ekstrand wrote: > I know this is rather late but > > On June 26, 2017 1:15:16 PM Andres Gomez <ago...@igalia.com> wrote: > > > Jason, you CCed this whole series for stable. However: > > * Patch 1/

[Mesa-dev] [ANNOUNCE] mesa 17.1.4

Andres Gomez (4):   cherry-ignore: 17.1.4 rejected commits   cherry-ignore: bin/get-fixes-pick-list.sh: better identify multiple "fixes:" tags   Update version to 17.1.4   docs: add release notes for 17.1.4 Anuj Phogat (2):   i965: Add and initialize l3_banks fiel

[Mesa-dev] Mesa 17.1.4 release candidate

, Andres Mesa stable queue - Nominated (0) == Queued (54) === Alex Deucher (1):   radeonsi: add new polaris12 pci id Andres Gomez (2):   cherry-ignore: 17.1.4 rejected commits   cherry-ignore: bin/get-fixes-pick-list.sh: better identify multiple &

Re: [Mesa-dev] [PATCH 1/2] svga: clamp device line width to at least 1 to fix HWv8 line stippling

It looks like we could want these 2 into -stable (?) On Thu, 2017-06-15 at 11:41 -0600, Brian Paul wrote: > The line stipple fallback code for virtual HW version 8 didn't work. > > With HW version 8, we were getting zero when querying the max line > widths (AA and non-AA). This means we were

Re: [Mesa-dev] [PATCH mesa] Fix khrplatform.h not installed if EGL is disabled.

Sound like a good inclusion for -stable (?) On Mon, 2017-06-12 at 12:00 +0100, Eric Engestrom wrote: > From: Eric Le Bihan > > KHR/khrplatform.h is required by the EGL, GLES and VG headers, but is > only installed if Mesa3d is compiled with EGL support. > > This

Re: [Mesa-dev] [Mesa-stable] [PATCH] nv50, nvc0: remove IDX from bufctx immediately, to avoid conflicts with clear

Ilia, this patch tries to fix 61d8f3387d which, in turn, tried to fix 330d0607e. None of them did it for 17.1 so I suppose we should leave this out, then. Let me know if you think otherwise. On Sat, 2017-06-24 at 13:24 -0400, Ilia Mirkin wrote: > The idxbuf could linger, and when a clear

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/3] amd/common: fix off-by-one in sid_tables.py

On Mon, 2017-06-26 at 14:23 +0100, Emil Velikov wrote: > On 23 June 2017 at 00:07, Andres Gomez <ago...@igalia.com> wrote: > > On Mon, 2017-06-19 at 12:36 +0100, Emil Velikov wrote: > > > Hi Nicolai, > > > > > > On 12 June 2017 at 20:33, Nicolai Hähnl

Re: [Mesa-dev] [Mesa-stable] [PATCH 00/16] anv: Fix our 48-bit problems

Jason, you CCed this whole series for stable. However: * Patch 1/16 already landed, but without the -stable tag. Should it be cherry-picked? It looks like yes. * Patch 2/16 did not land. Is it in need of review or it has been superseded? * Patches 3-15/16 were already cherry-picked in the

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/3] anv: Stop racing relocation offsets

Jason, it doesn't seem like this patch has landed in master. Are you in need of review or is it that this has been superseded? Thanks! On Wed, 2017-05-10 at 16:08 -0700, Jason Ekstrand wrote: > One of the key invariants of the relocation system is the > presumed_offset field. The assumption is

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/2] i965: Fix broxton 2x6 l3 config

Anuj, this depends on: https://cgit.freedesktop.org/mesa/mesa/commit/src/mesa?id=eb23be1d97da290073d76c2510b8999b250f0139 Which didn't make it for -stable. Should we cherry-pick that too? On Mon, 2017-06-12 at 10:01 -0700, Anuj Phogat wrote: > The new table added in this patch matches with the

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/3] amd/common: fix off-by-one in sid_tables.py

On Mon, 2017-06-19 at 12:36 +0100, Emil Velikov wrote: > Hi Nicolai, > > On 12 June 2017 at 20:33, Nicolai Hähnle wrote: > > From: Nicolai Hähnle > > > > The very last entry in the sid_strings_offsets table ended up missing, > > leading to

Re: [Mesa-dev] [Mesa-stable] [PATCH] st/glsl_to_tgsi: use correct writemask when converting generic intrinsics

Nicolai, this depends on: https://cgit.freedesktop.org/mesa/mesa/commit/src/mesa?id=944455217b67cb524efef9e628baf09416db5524 Which didn't make it for -stable. Should we cherry-pick that too? On Mon, 2017-06-12 at 21:45 +0200, Nicolai Hähnle wrote: > From: Nicolai Hähnle

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/3] amd/common: fix off-by-one in sid_tables.py

On Mon, 2017-06-19 at 12:36 +0100, Emil Velikov wrote: > Hi Nicolai, > > On 12 June 2017 at 20:33, Nicolai Hähnle wrote: > > From: Nicolai Hähnle > > > > The very last entry in the sid_strings_offsets table ended up missing, > > leading to

Re: [Mesa-dev] [PATCH 2/2] bin/get-fixes-pick-list.sh: better identify multiple "fixes:" tags

On Mon, 2017-05-15 at 11:56 +0100, Emil Velikov wrote: > On 13 May 2017 at 01:11, Andres Gomez <ago...@igalia.com> wrote: > > We were not considering as multiple fixes lines with: > > Fixes: $sha_1, Fixes: $sha_2 > > > > Now, we split the lines so w

Re: [Mesa-dev] [PATCH] anv: FORMAT_FEATURE_TRANSFER_SRC/DST_BIT_KHR not used with VkFormatProperties.bufferFeatures

On Thu, 2017-06-15 at 01:13 +0100, Lionel Landwerlin wrote: > Out of curiosity, does this fix a test (maybe upcoming one)? Not really, but realized checking an upcoming test for vk-gl-cts. The test was failing on some conditions because of erroneously checking some FORMAT_FEATURE flags against

<    1   2   3   4   5   6   7   >