Re: [Intel-gfx] [PATCH] drm/i915: Fix context size calculation on SNB/IVB/VLV

2013-08-23 Thread Ville Syrjälä
On Thu, Aug 22, 2013 at 11:30:55AM -0700, Ben Widawsky wrote: > On Thu, Aug 22, 2013 at 07:23:13PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > All the different context sizes reported in the CXT_SIZE register > > aren't meant to be simply added together. > > > > W

Re: [Intel-gfx] [PATCH] drm/i915: Fix context size calculation on SNB/IVB/VLV

2013-08-22 Thread Daniel Vetter
On Thu, Aug 22, 2013 at 12:05:31PM -0700, Ben Widawsky wrote: > On Thu, Aug 22, 2013 at 07:23:13PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > All the different context sizes reported in the CXT_SIZE register > > aren't meant to be simply added together. > > > > W

Re: [Intel-gfx] [PATCH] drm/i915: Fix context size calculation on SNB/IVB/VLV

2013-08-22 Thread Ben Widawsky
On Thu, Aug 22, 2013 at 07:23:13PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > All the different context sizes reported in the CXT_SIZE register > aren't meant to be simply added together. > > While BSpec is somewhat unclear on the topic of the actual context > size, em

Re: [Intel-gfx] [PATCH] drm/i915: Fix context size calculation on SNB/IVB/VLV

2013-08-22 Thread Ben Widawsky
On Thu, Aug 22, 2013 at 09:52:53PM +0300, Ville Syrjälä wrote: > On Thu, Aug 22, 2013 at 11:30:55AM -0700, Ben Widawsky wrote: > > On Thu, Aug 22, 2013 at 07:23:13PM +0300, ville.syrj...@linux.intel.com > > wrote: > > > From: Ville Syrjälä > > > > > > All the different context sizes reported in

Re: [Intel-gfx] [PATCH] drm/i915: Fix context size calculation on SNB/IVB/VLV

2013-08-22 Thread Ville Syrjälä
On Thu, Aug 22, 2013 at 11:30:55AM -0700, Ben Widawsky wrote: > On Thu, Aug 22, 2013 at 07:23:13PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > All the different context sizes reported in the CXT_SIZE register > > aren't meant to be simply added together. > > > > W

Re: [Intel-gfx] [PATCH] drm/i915: Fix context size calculation on SNB/IVB/VLV

2013-08-22 Thread Ville Syrjälä
On Thu, Aug 22, 2013 at 11:30:55AM -0700, Ben Widawsky wrote: > On Thu, Aug 22, 2013 at 07:23:13PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > All the different context sizes reported in the CXT_SIZE register > > aren't meant to be simply added together. > > > > W

Re: [Intel-gfx] [PATCH] drm/i915: Fix context size calculation on SNB/IVB/VLV

2013-08-22 Thread Ben Widawsky
On Thu, Aug 22, 2013 at 07:23:13PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > All the different context sizes reported in the CXT_SIZE register > aren't meant to be simply added together. > > While BSpec is somewhat unclear on the topic of the actual context > size, em

[Intel-gfx] [PATCH] drm/i915: Fix context size calculation on SNB/IVB/VLV

2013-08-22 Thread ville . syrjala
From: Ville Syrjälä All the different context sizes reported in the CXT_SIZE register aren't meant to be simply added together. While BSpec is somewhat unclear on the topic of the actual context size, empirical tests have now revealed the truth. So let's add a big fat comment to remind people ho