[Mesa-dev] [PATCH 1/2] i965: init image's planar_format in intel_create_image_from_texture

James Xiong james.xiong at intel.com
Thu Apr 5 16:02:57 UTC 2018


On Thu, 5 Apr 2018 08:56:54 -0700
Jason Ekstrand <jason at jlekstrand.net> wrote:

> On Thu, Apr 5, 2018 at 8:45 AM, James Xiong <james.xiong at intel.com>
> wrote:
> 
> > On Thu, 5 Apr 2018 08:24:27 -0700
> > Jason Ekstrand <jason at jlekstrand.net> wrote:
> >  
> > > Does this fix a bug?  If so, what?  
> > Jason, yes. I am sorry for the confusion, please see my earlier
> > reply to Tapani.
> >  
> 
> If it fixes a bug then the commit message should include a
> description of the bug fixed.  If the bug is a regression, it should
> have a "Fixes: SHA1" tag indicating where the regression originated
> and if it is a bug that has been around for a very long time, it
> should have a "Cc: mesa-stable at lists.freedesktop.org" tag.  If the
> bug is filed in bugzilla, it should have a "Bugzilla: <link>" tag.
> 
> --Jason
Thanks for the tip, Jason. I will make the corresponding changes in the
next version.
> 
> 
> 
> > >
> > > On April 4, 2018 16:57:21 James Xiong <james.xiong at intel.com>
> > > wrote: 
> > > > From: "Xiong, James" <james.xiong at intel.com>
> > > >
> > > > When creating a image from a texture, initialize the image's
> > > > planar_format with the texture's.
> > > >
> > > > Signed-off-by: Xiong, James <james.xiong at intel.com>
> > > > ---
> > > > src/mesa/drivers/dri/i965/intel_screen.c | 1 +
> > > > 1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/src/mesa/drivers/dri/i965/intel_screen.c
> > > > b/src/mesa/drivers/dri/i965/intel_screen.c
> > > > index dcb98da..7df8bc4 100644
> > > > --- a/src/mesa/drivers/dri/i965/intel_screen.c
> > > > +++ b/src/mesa/drivers/dri/i965/intel_screen.c
> > > > @@ -578,6 +578,7 @@ intel_create_image_from_texture(__DRIcontext
> > > > *context, int target,
> > > >    intel_setup_image_from_mipmap_tree(brw, image, iobj->mt,
> > > > level, zoffset); image->dri_format =
> > > > driGLFormatToImageFormat(image->format);
> > > > image->has_depthstencil = iobj->mt->stencil_mt? true : false;
> > > > +   image->planar_format = iobj->planar_format;
> > > >    if (image->dri_format == MESA_FORMAT_NONE) {
> > > >       *error = __DRI_IMAGE_ERROR_BAD_PARAMETER;
> > > >       free(image);
> > > > --
> > > > 2.7.4
> > > >
> > > > _______________________________________________
> > > > mesa-dev mailing list
> > > > mesa-dev at lists.freedesktop.org
> > > > https://lists.freedesktop.org/mailman/listinfo/mesa-dev  
> > >
> > >
> > >  
> >
> >  



More information about the mesa-dev mailing list