Re: [PATCH 7/7] lib: add igt_draw

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 07, 2015 at 10:10:25AM +0200, Daniel Vetter wrote:
> On Thu, Apr 02, 2015 at 12:15:13AM +0100, Chris Wilson wrote:
> > On Wed, Apr 01, 2015 at 07:40:59PM -0300, Paulo Zanoni wrote:
> > > +static void draw_rect_mmap_wc(int fd, struct buf_data *buf, struct rect *rect,
> > > +			      uint32_t color)
> > > +{
> > > +	uint32_t *ptr;
> > > +	uint32_t tiling, swizzle;
> > > +
> > > +	gem_get_tiling(fd, buf->handle, &tiling, &swizzle);
> > > +
> > > +	/* We didn't implement suport for the older tiling methods yet. */
> > > +	if (tiling != I915_TILING_NONE)
> > > +		igt_require(intel_gen(intel_get_drm_devid(fd)) >= 5);
> > 
> > But you now do! You need something like:
> 
> The problem is that the kernel hides bit17 swizzling. I chatted with Paulo
> on irc about this and we decided just ignore them all is the simplest
> approach.

Urm, that was the whole point of GET_TILING v2. That small function is
all you need to determine when bit17 is in effect and then you get to
reuse all the direct CPU methods (as they are also used by userspace)
for earlier gen.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux