From: Jamey Sharp <jamey-sH+B+fTmh7PR7s880joybQ@public.gmane.org>
To: Zhigang Gong <zhigang.gong-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Cc: intel-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
xorg-devel-go0+a7rfsptAfugRpC6u6w@public.gmane.org
Subject: Re: Glamor update
Date: Fri, 11 Nov 2011 13:47:00 -0800 [thread overview]
Message-ID: <20111111214700.GC3224@oh.minilop.net> (raw)
In-Reply-To: <055401cca04d$db738a10$925a9e30$@linux.intel.com>
[-- 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
next prev parent reply other threads:[~2011-11-11 21:47 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-11 8:42 Glamor update Zhigang Gong
2011-11-11 21:47 ` Jamey Sharp [this message]
2011-11-14 5:03 ` Zhigang Gong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20111111214700.GC3224@oh.minilop.net \
--to=jamey-sh+b+ftmh7pr7s880joybq@public.gmane.org \
--cc=intel-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=xorg-devel-go0+a7rfsptAfugRpC6u6w@public.gmane.org \
--cc=zhigang.gong-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox