From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 0/1] uxa/glamor: Route some drawing function to glamor. Date: Tue, 03 Jan 2012 21:46:31 +0000 Message-ID: References: <1325337495-18375-1-git-send-email-zhigang.gong@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id 8E5469E7A9 for ; Tue, 3 Jan 2012 13:46:37 -0800 (PST) In-Reply-To: <1325337495-18375-1-git-send-email-zhigang.gong@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: zhigang.gong@linux.intel.com Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Sat, 31 Dec 2011 21:18:14 +0800, zhigang.gong@linux.intel.com wrote: > From: Zhigang Gong > > I agree to pending the last patch which is to create glamor pixmap by default. > This patch is based on the other 3 patch. After apply this patch, and use > the latest glamor. I've pushed those 4 patches (all bar to use intel_glamor_create_textured_pixmap by default). I've a new segfault for you in glamor_core.c::glamor_validate_gc(), line 425 as pixmap_priv is NULL, which I only hit just as I thought I had completed testing the code. Afaics, the remaining big topics for the ddx are how to enable glx and integration between glamor/uxa render paths, right? -Chris -- Chris Wilson, Intel Open Source Technology Centre