bugzilla-daemon | 1 Jun 2011 14:22

[Bug 37177] Mathematica Plot3D Crash

https://bugs.freedesktop.org/show_bug.cgi?id=37177

Ivan Iakoupov <voxiac <at> gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Mathematica Plot3D: Crash   |Mathematica Plot3D Crash
                   |in dri2InvalidateBuffers    |

--- Comment #1 from Ivan Iakoupov <voxiac <at> gmail.com> 2011-06-01 05:22:51 PDT ---
I did some updates: libdrm, mesa, xf86-videl-intel to recent'ish versions from
master, kernel 2.6.39 (xserver is still 1.9.5 though). Now I cannot reproduce
any crash with that apitrace file. I can still reliably reproduce it using
Mathematica 8 itself though. Just open a new notebook. Type something like

Plot3D[Sin[x*y], {x, 0, 1}, {y, 0, 1}]

Shift+Enter to run and then rotate the resulting plot with the mouse. Upon
releasing the mouse button it crashes. Backtrace is this

Program received signal SIGSEGV, Segmentation fault.
intelDRI2Flush (drawable=<value optimized out>) at intel_screen.c:111
111       if (intel->gen < 4)
(gdb) bt
#0  intelDRI2Flush (drawable=<value optimized out>) at intel_screen.c:111
#1  0x00007f5517f51d23 in dri2SwapBuffers (pdraw=<value optimized out>,
target_msc=0, divisor=0, 
    remainder=0) at dri2_glx.c:542
#2  0x0000000000a9a425 in ?? ()
#3  0x0000000000ad934b in ?? ()
(Continue reading)

bugzilla-daemon | 1 Jun 2011 14:44

[Bug 37177] Mathematica Plot3D Crash

https://bugs.freedesktop.org/show_bug.cgi?id=37177

--- Comment #2 from Ivan Iakoupov <voxiac <at> gmail.com> 2011-06-01 05:44:08 PDT ---
Also running with LIBGL_ALWAYS_SOFTWARE="True" results in errors like 

X Error of failed request:  BadMatch (invalid parameter attributes)
  Major opcode of failed request:  152 (GLX)
  Minor opcode of failed request:  11 (X_GLXSwapBuffers)
  Serial number of failed request:  92490
  Current serial number in output stream:  92493

but no crash

--

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
Brian Paul | 1 Jun 2011 15:20
Picon

Re: [PATCH] mesa: UseShaderProgramEXT and Uniform* shouldn't be allowed inside Begin/End

On Tue, May 31, 2011 at 7:23 AM, Marek Olšák <maraeo <at> gmail.com> wrote:
> I couldn't find this being required by the spec.
> ---
>  src/mesa/main/shaderapi.c |    2 ++
>  src/mesa/main/uniforms.c  |    4 ++++
>  2 files changed, 6 insertions(+), 0 deletions(-)
>

Reviewed-by: Brian Paul <brianp <at> vmware.com>

NVIDIA's driver generates GL_INVALID_OPERATION for glUniform between
glBegin/End, btw.

-Brian
_______________________________________________
mesa-dev mailing list
mesa-dev <at> lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
bugzilla-daemon | 1 Jun 2011 15:30

[Bug 36238] Mesa release files don't contain scons control files

https://bugs.freedesktop.org/show_bug.cgi?id=36238

--- Comment #7 from Brian Paul <brianp <at> vmware.com> 2011-06-01 06:30:01 PDT ---
Anthony, I've put new zip files which include the c99 header files at
ftp://ftp.freedesktop.org/pub/mesa/tmp/ .  Let me know if that helps.

--

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
Brian Paul | 1 Jun 2011 16:30
Favicon

Re: [PATCH] glapi: Make xserver location error more helpful

On 05/31/2011 09:42 AM, Nathan Kidd wrote:
> From: Nathan Kidd <nkidd <at> opentext.com>
>
> glx code hasn't lived under xserver/GL for a long time now.
>
> Signed-off-by: Nathan Kidd <nkidd <at> opentext.com>
> ---
> src/mapi/glapi/gen/Makefile | 5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/src/mapi/glapi/gen/Makefile b/src/mapi/glapi/gen/Makefile
> index 87b928c..c3829dc 100644
> --- a/src/mapi/glapi/gen/Makefile
> +++ b/src/mapi/glapi/gen/Makefile
>  <at>  <at>  -128,7 +128,10  <at>  <at>  xorg: check-xorg-source $(XORG_OUTPUTS)
>
> check-xorg-source:
>  <at> if ! test -d $(XORG_GLX_DIR); then \
> - echo "ERROR: Must specify path to xserver/GL/ checkout; set
> XORG_BASE env var."; \
> + echo "ERROR: Must specify path to xserver/ checkout. Set XORG_BASE
> env var."; \
> + if test x$(XORG_BASE) != x; then \
> + echo "'$(XORG_GLX_DIR)' does not exist."; \
> + fi; \
> exit 1; \
> fi

Thanks.  I'll commit this soon.

(Continue reading)

Adam Jackson | 1 Jun 2011 19:44
Picon
Favicon

[PATCH 1/2] glx: Fix another case of using req outside of the display lock

Signed-off-by: Adam Jackson <ajax <at> redhat.com>
---
 src/glx/glxcmds.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/glx/glxcmds.c b/src/glx/glxcmds.c
index dbfa0ed..8b4151d 100644
--- a/src/glx/glxcmds.c
+++ b/src/glx/glxcmds.c
 <at>  <at>  -644,13 +644,13  <at>  <at>  glXCreateGLXPixmap(Display * dpy, XVisualInfo * vis, Pixmap pixmap)
       if (psc->driScreen == NULL)
          break;
       config = glx_config_find_visual(psc->visuals, vis->visualid);
-      pdraw = psc->driScreen->createDrawable(psc, pixmap, req->glxpixmap, config);
+      pdraw = psc->driScreen->createDrawable(psc, pixmap, xid, config);
       if (pdraw == NULL) {
          fprintf(stderr, "failed to create pixmap\n");
          break;
       }

-      if (__glxHashInsert(priv->drawHash, req->glxpixmap, pdraw)) {
+      if (__glxHashInsert(priv->drawHash, xid, pdraw)) {
          (*pdraw->destroyDrawable) (pdraw);
          return None;           /* FIXME: Check what we're supposed to do here... */
       }
--

-- 
1.7.5.1
Adam Jackson | 1 Jun 2011 19:44
Picon
Favicon

[PATCH 2/2] glx: Remove (unused, broken) fastImageUnpack fast path

Signed-off-by: Adam Jackson <ajax <at> redhat.com>
---
 src/glx/glxclient.h    |    8 ----
 src/glx/indirect_glx.c |    3 -
 src/glx/renderpix.c    |   98 +++++++++++++++++++-----------------------------
 3 files changed, 39 insertions(+), 70 deletions(-)

diff --git a/src/glx/glxclient.h b/src/glx/glxclient.h
index 2b6966f..a443f78 100644
--- a/src/glx/glxclient.h
+++ b/src/glx/glxclient.h
 <at>  <at>  -310,14 +310,6  <at>  <at>  struct glx_context
    /* <at> } */

     /**
-     * This is \c GL_TRUE if the pixel unpack modes are such that an image
-     * can be unpacked from the clients memory by just copying.  It may
-     * still be true that the server will have to do some work.  This
-     * just promises that a straight copy will fetch the correct bytes.
-     */
-   GLboolean fastImageUnpack;
-
-    /**
      * Fill newImage with the unpacked form of \c oldImage getting it
      * ready for transport to the server.
      */
diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c
index 1870ede..b4f16c7 100644
--- a/src/glx/indirect_glx.c
+++ b/src/glx/indirect_glx.c
(Continue reading)

Eric Anholt | 1 Jun 2011 22:17
Gravatar

[PATCH 2/2] meta: Don't do sRGB encode for framebuffer blits on sRGB-enabled framebuffers.

Fixes fbo-srgb-blit.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=35373
---
 src/mesa/drivers/common/meta.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
index a0d4b9e..979926a 100644
--- a/src/mesa/drivers/common/meta.c
+++ b/src/mesa/drivers/common/meta.c
 <at>  <at>  -1187,6 +1187,7  <at>  <at>  blitframebuffer_texture(struct gl_context *ctx,
          const GLenum wrapSSave = texObj->Sampler.WrapS;
          const GLenum wrapTSave = texObj->Sampler.WrapT;
          const GLenum srgbSave = texObj->Sampler.sRGBDecode;
+         const GLenum fbo_srgb_save = ctx->Color.sRGBEnabled;
          const GLenum target = texObj->Target;

          if (drawAtt->Texture == readAtt->Texture) {
 <at>  <at>  -1219,10 +1220,12  <at>  <at>  blitframebuffer_texture(struct gl_context *ctx,
          _mesa_TexParameteri(target, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);
          _mesa_TexParameteri(target, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);

+	 /* Always do our blits with no sRGB decode or encode.*/
 	 if (ctx->Extensions.EXT_texture_sRGB_decode) {
 	    _mesa_TexParameteri(target, GL_TEXTURE_SRGB_DECODE_EXT,
 				GL_SKIP_DECODE_EXT);
 	 }
+	 _mesa_Disable(GL_FRAMEBUFFER_SRGB_EXT);

          _mesa_TexEnvi(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_REPLACE);
(Continue reading)

Eric Anholt | 1 Jun 2011 22:17
Gravatar

[PATCH 1/2] meta: Don't do srgb to linear decode when blitting srgb textures.

Fixes the GL_SRGB8_ALPHA8 -> GL_RGBA8 blits in fbo-srgb-blit.c
---
 src/mesa/drivers/common/meta.c |   10 ++++++++++
 1 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
index 08b6024..a0d4b9e 100644
--- a/src/mesa/drivers/common/meta.c
+++ b/src/mesa/drivers/common/meta.c
 <at>  <at>  -1186,6 +1186,7  <at>  <at>  blitframebuffer_texture(struct gl_context *ctx,
          const GLint maxLevelSave = texObj->MaxLevel;
          const GLenum wrapSSave = texObj->Sampler.WrapS;
          const GLenum wrapTSave = texObj->Sampler.WrapT;
+         const GLenum srgbSave = texObj->Sampler.sRGBDecode;
          const GLenum target = texObj->Target;

          if (drawAtt->Texture == readAtt->Texture) {
 <at>  <at>  -1217,6 +1218,12  <at>  <at>  blitframebuffer_texture(struct gl_context *ctx,
          }
          _mesa_TexParameteri(target, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);
          _mesa_TexParameteri(target, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);
+
+	 if (ctx->Extensions.EXT_texture_sRGB_decode) {
+	    _mesa_TexParameteri(target, GL_TEXTURE_SRGB_DECODE_EXT,
+				GL_SKIP_DECODE_EXT);
+	 }
+
          _mesa_TexEnvi(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_REPLACE);
          _mesa_set_enable(ctx, target, GL_TRUE);

(Continue reading)

Brian Paul | 1 Jun 2011 22:35
Picon

Re: [PATCH 1/2] meta: Don't do srgb to linear decode when blitting srgb textures.

On Wed, Jun 1, 2011 at 2:17 PM, Eric Anholt <eric <at> anholt.net> wrote:
> Fixes the GL_SRGB8_ALPHA8 -> GL_RGBA8 blits in fbo-srgb-blit.c
> ---
>  src/mesa/drivers/common/meta.c |   10 ++++++++++
>  1 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
> index 08b6024..a0d4b9e 100644
> --- a/src/mesa/drivers/common/meta.c
> +++ b/src/mesa/drivers/common/meta.c
>  <at>  <at>  -1186,6 +1186,7  <at>  <at>  blitframebuffer_texture(struct gl_context *ctx,
>          const GLint maxLevelSave = texObj->MaxLevel;
>          const GLenum wrapSSave = texObj->Sampler.WrapS;
>          const GLenum wrapTSave = texObj->Sampler.WrapT;
> +         const GLenum srgbSave = texObj->Sampler.sRGBDecode;
>          const GLenum target = texObj->Target;
>
>          if (drawAtt->Texture == readAtt->Texture) {
>  <at>  <at>  -1217,6 +1218,12  <at>  <at>  blitframebuffer_texture(struct gl_context *ctx,
>          }
>          _mesa_TexParameteri(target, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);
>          _mesa_TexParameteri(target, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);
> +
> +        if (ctx->Extensions.EXT_texture_sRGB_decode) {
> +           _mesa_TexParameteri(target, GL_TEXTURE_SRGB_DECODE_EXT,
> +                               GL_SKIP_DECODE_EXT);
> +        }
> +
>          _mesa_TexEnvi(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_REPLACE);
>          _mesa_set_enable(ctx, target, GL_TRUE);
(Continue reading)


Gmane