On 04/28/2014 09:41 PM, Daniel Vetter wrote:
> 64bit divisions won't compile on 32bit. You need one of the DO_DIV macros,
> or whatever they're called again. I pain, I know ;-)
Thanks for the correction, here is an updated patch :-)
From: Aaron Lu
Date: Mon, 28 Apr 2014 11:02:52 +0800
Subject: [
On Sun, May 04, 2014 at 03:16:05PM +0800, Aaron Lu wrote:
> On 04/28/2014 09:41 PM, Daniel Vetter wrote:
> > 64bit divisions won't compile on 32bit. You need one of the DO_DIV macros,
> > or whatever they're called again. I pain, I know ;-)
>
> Thanks for the correction, here is an updated patch :
On 05/04/2014 03:22 PM, Chris Wilson wrote:
> On Sun, May 04, 2014 at 03:16:05PM +0800, Aaron Lu wrote:
>> On 04/28/2014 09:41 PM, Daniel Vetter wrote:
>>> 64bit divisions won't compile on 32bit. You need one of the DO_DIV macros,
>>> or whatever they're called again. I pain, I know ;-)
>>
>> Thank
On Tue, 2014-04-29 at 16:33 +0300, Ville Syrjälä wrote:
> On Sun, Apr 20, 2014 at 04:22:48PM +0530, akash.g...@intel.com wrote:
> > From: Akash Goel
> >
> > This patch changes the return type of panel fitter configuration
> > functions from 'void', so that an error could be returned back to
> > U
On Tue, 2014-04-29 at 17:06 +0300, Ville Syrjälä wrote:
> On Sun, Apr 20, 2014 at 04:14:18PM +0530, akash.g...@intel.com wrote:
> > From: Akash Goel
> >
> > This patch adds a new drm crtc property for varying the Pipe Src size
> > or the Panel fitter input size. Pipe Src controls the size that is
On Sun, May 04, 2014 at 03:31:01PM +0800, Aaron Lu wrote:
> On 05/04/2014 03:22 PM, Chris Wilson wrote:
> > Also this still has the same rounding error as before.
>
> I didn't get this, care to explain?
The calculation you use, truncates, rather than say round to nearest,
would is the same discre
From: Akash Goel
This patch could help to reduce the time, 'struct_mutex' is kept
locked during either the exec-buffer path or Page fault
handling path as now the backing pages are requested from shmem layer
without holding the 'struct_mutex'.
Signed-off-by: Akash Goel
---
drivers/gpu/drm/i915
From: Akash Goel
Moved the cache flush of the preallocated shmem pages outside
the span of 'struct_mutex' lock. This shall not lead to any
redundancy as the cache flush of the newly allocated pages
will be done anyways when same buffer is submitted to GPU or
when the domain of the object is chang
From: Akash Goel
We are trying to reduce the time for which the global 'struct_mutex'
is locked. Execbuffer ioctl is one place where it is generally held
for the longest time. And sometimes because of this occasional
glitches/flickers are being observed in 60 fps playback (due to miss of
V-blank
On Sun, May 04, 2014 at 04:48:23PM +0530, akash.g...@intel.com wrote:
> From: Akash Goel
>
> We are trying to reduce the time for which the global 'struct_mutex'
> is locked. Execbuffer ioctl is one place where it is generally held
> for the longest time. And sometimes because of this occasional
There could be something amiss on the User space side, but this is a very
sporadic issue.
It mainly comes to the fore, when some huge buffers (30 MB) are being processed.
Best Regards
Akash
-Original Message-
From: Chris Wilson [mailto:ch...@chris-wilson.co.uk]
Sent: Sunday, May 04, 201
On 05/04/2014 06:41 PM, Chris Wilson wrote:
> On Sun, May 04, 2014 at 03:31:01PM +0800, Aaron Lu wrote:
>> On 05/04/2014 03:22 PM, Chris Wilson wrote:
>>> Also this still has the same rounding error as before.
>>
>> I didn't get this, care to explain?
>
> The calculation you use, truncates, rather
From: Akash Goel
This patch could help to reduce the time, 'struct_mutex' is kept
locked during either the exec-buffer path or Page fault
handling path as now the backing pages are requested from shmem layer
without holding the 'struct_mutex'.
v2: Fixed the merge issue, due to which 'exec_lock'
Thanks Ben. Apologies for delayed response.
I am incorporating the review comment changes next set of patch review.
On Saturday 26 April 2014 03:24 AM, Ben Widawsky wrote:
On Mon, Apr 21, 2014 at 01:34:08PM +0530, deepa...@linux.intel.com wrote:
From: Deepak S
Support to individually contro
On Thu, May 1, 2014 at 1:26 AM, Dave Airlie wrote:
>
> Merged, but 32-bit still a thing,
>
> CC [M] drivers/gpu/drm/i915/i915_cmd_parser.o
> /ssd/git/drm-next/drivers/gpu/drm/i915/i915_cmd_parser.c: In function
> ‘i915_parse_cmds’:
> /ssd/git/drm-next/drivers/gpu/drm/i915/i915_cmd_parser.c:919:
15 matches
Mail list logo
