Glamor update

Glamor update

[Zhigang Gong]

[-- Attachment #1.1: Type: text/plain, Size: 1157 bytes --]

Hi folks，

 

During the last discussion about glamor’s plan in this list, we got a
conclusion that to extract glamor from xorg and

build a separate glamor library to be used by any possible DDX driver. And
Eric suggested I can incrementally 

merge glamor into Intel video driver. Now here is the update.

 

The separate glamor library is at :
git://people.freedesktop.org/~gongzg/glamor，it provides two interfaces:

1.       glamor : Rendering library. All the rendering functions are
implemented in this package. 

2.       glamor-egl : EGL support library. This package provides functions
to create and initialize OpenGL/EGL context.

 

There are a little bit more details to introduce glamor in the README file.

 

The Intel video driver to merge glamor is at
git://people.freedesktop.org/~gongzg/xf86-video-intel’s “glamor” branch.

I just started the merging stage. Only finished 3 patches to enable glamor
in UXA code path. Currently, only migrate

fillspans and polyfillrect to glamor. Will continue to migrate the rest
functions.

 

I will submit the patches to intel-gfx mail list soon.

 


[-- Attachment #1.2: Type: text/html, Size: 7263 bytes --]

[-- Attachment #2: Type: text/plain, Size: 219 bytes --]

_______________________________________________
xorg-devel-go0+a7rfsptAfugRpC6u6w@public.gmane.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: Glamor update

[Jamey Sharp]

[-- Attachment #1.1: Type: text/plain, Size: 1903 bytes --]

Hello!

On Fri, Nov 11, 2011 at 04:42:32PM +0800, Zhigang Gong wrote:
> During the last discussion about glamor’s plan in this list, we got a
> conclusion that to extract glamor from xorg and build a separate
> glamor library to be used by any possible DDX driver. And Eric
> suggested I can incrementally merge glamor into Intel video driver.
> Now here is the update.
> 
> The separate glamor library is at :
> git://people.freedesktop.org/~gongzg/glamor，it provides two interfaces:
> 
> 1.       glamor : Rendering library. All the rendering functions are
> implemented in this package. 
> 
> 2.       glamor-egl : EGL support library. This package provides functions
> to create and initialize OpenGL/EGL context.
> 
> There are a little bit more details to introduce glamor in the README file.
> 
> The Intel video driver to merge glamor is at
> git://people.freedesktop.org/~gongzg/xf86-video-intel’s “glamor” branch.
> 
> I just started the merging stage. Only finished 3 patches to enable
> glamor in UXA code path. Currently, only migrate fillspans and
> polyfillrect to glamor. Will continue to migrate the rest functions.

I've looked over all the source in your glamor repo, and your patches to
the Intel driver. This work looks very promising! I'm eager to see more
progress.

I don't speak for the Intel graphics team or anything, but I noticed a
couple minor things you might clean up. In the "glamor: turn on glamor."
commit, there's a patch hunk that just adds an empty ifdef:

-
+#ifdef GLAMOR
+#endif

You might want to remove that. Also, the call to
intel_glamor_create_textured_pixmap in intel_uxa_create_pixmap is
inconsistently indented; it should just be tabs, looks like.

Any chance you'll add some of the Xephyr/glamor functionality to
xf86-video-nested, too? I think that should be easier to test and hack
on.

Jamey

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

[-- Attachment #2: Type: text/plain, Size: 219 bytes --]

_______________________________________________
xorg-devel-go0+a7rfsptAfugRpC6u6w@public.gmane.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: Glamor update

[Zhigang Gong]

> -----Original Message-----
> From: Jamey Sharp [mailto:jamey@minilop.net]
> Sent: Saturday, November 12, 2011 5:47 AM
> To: Zhigang Gong
> Cc: xorg-devel@lists.x.org; intel-gfx@lists.freedesktop.org
> Subject: Re: Glamor update
> 
> Hello!
> 
> On Fri, Nov 11, 2011 at 04:42:32PM +0800, Zhigang Gong wrote:
> > During the last discussion about glamor’s plan in this list, we got a
> > conclusion that to extract glamor from xorg and build a separate
> > glamor library to be used by any possible DDX driver. And Eric
> > suggested I can incrementally merge glamor into Intel video driver.
> > Now here is the update.
> >
> > The separate glamor library is at :
> > git://people.freedesktop.org/~gongzg/glamor，it provides two
> interfaces:
> >
> > 1.       glamor : Rendering library. All the rendering functions are
> > implemented in this package.
> >
> > 2.       glamor-egl : EGL support library. This package provides
> functions
> > to create and initialize OpenGL/EGL context.
> >
> > There are a little bit more details to introduce glamor in the README file.
> >
> > The Intel video driver to merge glamor is at
> > git://people.freedesktop.org/~gongzg/xf86-video-intel’s “glamor”
> branch.
> >
> > I just started the merging stage. Only finished 3 patches to enable
> > glamor in UXA code path. Currently, only migrate fillspans and
> > polyfillrect to glamor. Will continue to migrate the rest functions.
> 
> I've looked over all the source in your glamor repo, and your patches to
> the Intel driver. This work looks very promising! I'm eager to see more
> progress.
> 
> I don't speak for the Intel graphics team or anything, but I noticed a couple
> minor things you might clean up. In the "glamor: turn on glamor."
> commit, there's a patch hunk that just adds an empty ifdef:
> 
> -
> +#ifdef GLAMOR
> +#endif
Fixed.
> 
> You might want to remove that. Also, the call to
> intel_glamor_create_textured_pixmap in intel_uxa_create_pixmap is
> inconsistently indented; it should just be tabs, looks like.
Fixed.
I will submit the new version patch again latter. Thanks for your carefully 
reviewing.

> 
> Any chance you'll add some of the Xephyr/glamor functionality to
> xf86-video-nested, too? I think that should be easier to test and hack on.

Currently, we don't have enough bandwidth to do that. If anyone in the 
community want to do that, I'd be happy to provide help. 

> 
> Jamey

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx