On Wed, Apr 20, 2016 at 04:35:59PM +0300, Imre Deak wrote:
> On ke, 2016-04-20 at 15:16 +0200, Daniel Vetter wrote:
> > On Wed, Apr 20, 2016 at 04:13:25PM +0300, Imre Deak wrote:
> > > On ke, 2016-04-20 at 15:02 +0200, Daniel Vetter wrote:
> > > > On Mon, Apr 18, 2016 at 02:48:21PM +0300, Imre Deak
From: Robert Foss
Fixed ctrc_id comparison failing due to bad initialization of
crtc variable.
Signed-off-by: Robert Foss
---
tests/kms_atomic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c
index 13a5db1..f27ee46 100644
--- a/tests
Chris Wilson writes:
> [ text/plain ]
> Refactor pinning and unpinning of contexts, such that the default
> context for an engine is pinned during initialisation and unpinned
> during teardown (pinning of the context handles the reference counting).
> Thus we can eliminate the special case handli
On 19/04/16 13:59, Chris Wilson wrote:
As the contexts are accessed by the hardware until the switch is completed
to a new context, the hardware may still be writing to the context object
after the breadcrumb is visible. We must not unpin/unbind/prune that
object whilst still active and so we ke
On Wed, 20 Apr 2016, Chris Wilson wrote:
> If the caller, in this case efivarfs_callback(), only provides sufficent
> room for the expanded utf8 and not enough to include the terminating NUL
> byte, that NUL byte is skipped. When the caller then interprets it as a
> string, it may then read from p
From: Robert Foss
Switched to assert helpers to enable better error output.
Signed-off-by: Robert Foss
---
tests/core_getversion.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/core_getversion.c b/tests/core_getversion.c
index 5830068..b336e9f 100644
--- a/t
From: Robert Foss
This change mirrors the change in 8a5c4d567fafffbda57cbe5e4ed8c419193cada5
of drm.
The assert(major < 1) is only needed for the legacy intel driver.
Signed-off-by: Robert Foss
---
tests/core_getversion.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/t
From: Robert Foss
This series enables tests/core_getversion to test vc4.
It's been tested on rpi2 hardware and kernel 4.6.0-rc1.
Robert Foss (2):
core_getversion: Switched from igt_assert to assert helpers.
core_getversion: Only verify major version for i915.
tests/core_getversion.c | 10 +
From: Tim Gore
This patch applies a performance enhancement workaround
based on analysis of DX and OCL S-Curve workloads.
Signed-off-by: Tim Gore
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_pm.c | 4
2 files changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/
On 20 April 2016 at 15:10, Daniel Vetter wrote:
> On Mon, Apr 18, 2016 at 01:42:48PM +0200, Tomeu Vizoso wrote:
>> Hi,
>>
>> these patches allow a few more tests to run on drivers other than i915,
>> mainly by removing the last usage of
>> DRM_IOCTL_I915_GET_PIPE_FROM_CRTC_ID and removing superflu
On 20/04/16 14:30, Dave Gordon wrote:
The recently-added i915_gem_object_pin_map() can be further optimised
for "small" objects. To facilitate this, and simplify the error paths
before adding the new code, this patch pulls out the "mapping" part of
the operation (involving local allocations which
On Wed, Apr 20, 2016 at 04:43:56PM +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Somehow my SNB GT1 (Dell XPS 8300) gets very unhappy around
> GPU hangs if the RPS EI/thresholds aren't suitably aligned.
> It seems like scheduling/timer interupts stop working somehow
> and
On 19/04/16 13:59, Chris Wilson wrote:
If we move the release of the GEM request (i.e. decoupling it from the
various lists used for client and context tracking) after it is complete
(either by the GPU retiring the request, or by the caller cancelling the
request), we can remove the requirement
== Series Details ==
Series: drm/i915: Cache DisplayPort link signal levels
URL : https://patchwork.freedesktop.org/series/5973/
State : failure
== Summary ==
Series 5973v1 drm/i915: Cache DisplayPort link signal levels
http://patchwork.freedesktop.org/api/1.0/series/5973/revisions/1/mbox/
Te
On 19/04/16 18:10, Dave Gordon wrote:
On 19/04/16 17:24, Patchwork wrote:
== Series Details ==
Series: series starting with [1/3] drm/i915/guc: keep GuC doorbell &
process descriptor mapped in kernel
URL : https://patchwork.freedesktop.org/series/5942/
State : failure
== Summary ==
Series
On Wed, Apr 20, 2016 at 03:24:45PM +0200, Daniel Vetter wrote:
> On Tue, Apr 19, 2016 at 09:19:53AM +0100, Chris Wilson wrote:
> > On Tue, Apr 19, 2016 at 09:52:39AM +0200, Maarten Lankhorst wrote:
> >
> > If you think it's ready:
> > dev->mode_config.async_page_flip = true;
>
> Nah, this is
On Wed, Apr 20, 2016 at 03:19:49PM +0200, Daniel Vetter wrote:
> On Tue, Apr 19, 2016 at 03:01:31PM +0300, Ville Syrjälä wrote:
> > On Tue, Apr 19, 2016 at 02:25:40PM +0300, Jani Nikula wrote:
> > > On Mon, 18 Apr 2016, ville.syrj...@linux.intel.com wrote:
> > > > From: Ville Syrjälä
> > > >
> > >
From: Ville Syrjälä
Somehow my SNB GT1 (Dell XPS 8300) gets very unhappy around
GPU hangs if the RPS EI/thresholds aren't suitably aligned.
It seems like scheduling/timer interupts stop working somehow
and things get stuck eg. in usleep_range().
I bisected the problem down to
commit 8a5864377b12
Piglit changed the format of the result json file, replacing time attribute
float with a TimeAttribute object, which stores a start time and an end time.
Fixed this script by calculating the duration when reading the json results.
Signed-off-by: Gabriel Feceoru
---
scripts/quick-testlist.py | 6
On ke, 2016-04-20 at 15:16 +0200, Daniel Vetter wrote:
> On Wed, Apr 20, 2016 at 04:13:25PM +0300, Imre Deak wrote:
> > On ke, 2016-04-20 at 15:02 +0200, Daniel Vetter wrote:
> > > On Mon, Apr 18, 2016 at 02:48:21PM +0300, Imre Deak wrote:
> > > > While we disable runtime PM and with that display p
We're using this function for ringbuffers and other "small" objects, so
it's worth avoiding an extra malloc()/free() cycle if the page array is
small enough to put on the stack. Here we've chosen an arbitrary cutoff
of 32 (4k) pages, which is big enough for a ringbuffer (4 pages) or a
context image
The recently-added i915_gem_object_pin_map() can be further optimised
for "small" objects. To facilitate this, and simplify the error paths
before adding the new code, this patch pulls out the "mapping" part of
the operation (involving local allocations which must be undone before
return) into its
On Mon, 18 Apr 2016, Jani Nikula wrote:
> On Wed, 13 Apr 2016, Daniel Vetter wrote:
>> On Tue, Apr 12, 2016 at 09:03:05PM +0100, Chris Wilson wrote:
>>> Two concurrent writes into the same register cacheline has the chance of
>>> killing the machine on Ivybridge and other gen7. This includes LRI
On Tue, Apr 19, 2016 at 01:48:14PM +0530, Ramalingam C wrote:
> In BXT DSI there is no regs programmed with few horizontal timings
> in Pixels but txbyteclkhs.. So retrieval process adds some
> ROUND_UP ERRORS in the process of PIXELS<==>txbyteclkhs.
>
> Actually here for the given adjusted_mode,
On Tue, Apr 19, 2016 at 09:19:53AM +0100, Chris Wilson wrote:
> On Tue, Apr 19, 2016 at 09:52:39AM +0200, Maarten Lankhorst wrote:
>
> If you think it's ready:
> dev->mode_config.async_page_flip = true;
Nah, this is something entirely different.
"async atomic commit" = doesn't block, but v
On 04/20/16 10:37, Chris Wilson wrote:
> If the caller, in this case efivarfs_callback(), only provides sufficent
> room for the expanded utf8 and not enough to include the terminating NUL
> byte, that NUL byte is skipped. When the caller then interprets it as a
> string, it may then read from past
On Tue, Apr 19, 2016 at 03:01:31PM +0300, Ville Syrjälä wrote:
> On Tue, Apr 19, 2016 at 02:25:40PM +0300, Jani Nikula wrote:
> > On Mon, 18 Apr 2016, ville.syrj...@linux.intel.com wrote:
> > > From: Ville Syrjälä
> > >
> > > intel_pipe_will_have_type() doesn't just look at the passied in
> > > pi
On Wed, Apr 20, 2016 at 04:13:25PM +0300, Imre Deak wrote:
> On ke, 2016-04-20 at 15:02 +0200, Daniel Vetter wrote:
> > On Mon, Apr 18, 2016 at 02:48:21PM +0300, Imre Deak wrote:
> > > While we disable runtime PM and with that display power well support if
> > > the DMC firmware isn't loaded, we st
On ke, 2016-04-20 at 15:02 +0200, Daniel Vetter wrote:
> On Mon, Apr 18, 2016 at 02:48:21PM +0300, Imre Deak wrote:
> > While we disable runtime PM and with that display power well support if
> > the DMC firmware isn't loaded, we still want to disable power wells
> > during system suspend and drive
On Wed, Apr 20, 2016 at 02:47:57PM +0200, Daniel Vetter wrote:
> On Thu, Apr 14, 2016 at 03:28:51PM +0300, Ville Syrjälä wrote:
> > On Thu, Apr 14, 2016 at 11:37:49AM +0200, Daniel Vetter wrote:
> > > On Wed, Apr 13, 2016 at 09:09:30PM +0300, ville.syrj...@linux.intel.com
> > > wrote:
> > > > From
On Mon, Apr 18, 2016 at 01:42:48PM +0200, Tomeu Vizoso wrote:
> Hi,
>
> these patches allow a few more tests to run on drivers other than i915,
> mainly by removing the last usage of
> DRM_IOCTL_I915_GET_PIPE_FROM_CRTC_ID and removing superfluous dependencies
> on bufmgr and tiled BOs.
Ack on all
On Mon, Apr 18, 2016 at 01:42:49PM +0200, Tomeu Vizoso wrote:
> So the test runs on other drivers, drop the usage of the i915-specific
> DRM_IOCTL_I915_GET_PIPE_FROM_CRTC_ID ioctl.
>
> Wait for a vblank event on pipe0 and if we get it, then the test can
> proceed (code copied from kms_vblank).
>
On Mon, Apr 18, 2016 at 12:52:44PM +0300, Gabriel Feceoru wrote:
> Currently, when trying to run i-g-t tests with piglit, it takes up
> to 20s until the first test is executed.
> The main reason is that gem_concurrent_all has ~500k subtests,
> overkilling piglit.
>
> This patch separates gem_concu
Hi,
[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on next-20160420]
[cannot apply to v4.6-rc4]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/ankitprasad-r
On Mon, Apr 18, 2016 at 02:48:21PM +0300, Imre Deak wrote:
> While we disable runtime PM and with that display power well support if
> the DMC firmware isn't loaded, we still want to disable power wells
> during system suspend and driver unload. So drop/reacquire the
> corresponding power refcount
Hi,
[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on next-20160420]
[cannot apply to v4.6-rc4]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/ankitprasad-r
drivers/gpu/drm/i915/i915_gem.c:168:43-44: Unneeded semicolon
Remove unneeded semicolon.
Generated by: scripts/coccinelle/misc/semicolon.cocci
CC: Ankitprasad Sharma
Signed-off-by: Fengguang Wu
---
i915_gem.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/drivers/gpu/drm/i
On Mon, Apr 18, 2016 at 12:18:23PM +0300, Jani Nikula wrote:
> On Fri, 15 Apr 2016, Chris Wilson wrote:
> > Final canvas for opinions for using a magic macro to reduce typing in
> > the common operation of getting our drm_i915_private from the object.
> >
> > 21 files changed, 333 insertions(+
On Fri, Apr 15, 2016 at 02:00:04PM +0200, Tomeu Vizoso wrote:
> The variable used as loop counter in the igt_fixture macro had
> unspecified value from the setjmp(3) man page quoted below. Because of
> that, in certain circumstances and with -O2 and -Os, the initialization
> of that variable would
On Fri, Apr 15, 2016 at 09:47:21AM +0100, tim.g...@intel.com wrote:
> From: Tim Gore
>
> This is a re-send of patch 3/3 from Arun Siluvery's patch
> series "Add Pooled EU support to BXT", originally submitted
> 20 July 2015. Patches 1 and 2 are already merged in preparation.
> This final patch ac
On Thu, Apr 14, 2016 at 03:28:51PM +0300, Ville Syrjälä wrote:
> On Thu, Apr 14, 2016 at 11:37:49AM +0200, Daniel Vetter wrote:
> > On Wed, Apr 13, 2016 at 09:09:30PM +0300, ville.syrj...@linux.intel.com
> > wrote:
> > > From: Ville Syrjälä
> > >
> > > On CHV GTFIFODBG has some read-only bits to
On 04/20/16 11:41, Chris Wilson wrote:
> On Wed, Apr 20, 2016 at 11:36:37AM +0200, Laszlo Ersek wrote:
>> On 04/20/16 10:37, Chris Wilson wrote:
>>> If the caller, in this case efivarfs_callback(), only provides sufficent
>>> room for the expanded utf8 and not enough to include the terminating NUL
On Fri, Apr 15, 2016 at 06:12:32PM +, Vivi, Rodrigo wrote:
> I was talking to PM engineers today on their Summit and they are really
> interrested in a way to toggle features on/off easily.
>
> They also told if debugfs is the only way that powertop should have to
> deal with it.
>
> So I see
On Wed, Apr 20, 2016 at 09:00:09AM +0200, Maarten Lankhorst wrote:
> Op 19-04-16 om 12:13 schreef Lionel Landwerlin:
> > On 19/04/16 07:02, Maarten Lankhorst wrote:
> >> Op 18-04-16 om 18:47 schreef Bob Paauwe:
> >>> The i915 driver is now using atomic properties and atomic commit
> >>> to handle t
It was noticed on bug #94087 that module parameter
i915.edp_vswing=2 that should override the VBT setting
to use default voltage swing (400 mV) was not applied
for Broadwell.
This patch provides a fix for this by checking if default
i.e. higher voltage swing is requested to be used and
applies the
On Fri, Apr 15, 2016 at 05:09:24PM +0100, Chris Wilson wrote:
> On Mon, Apr 04, 2016 at 02:18:10PM +0100, Chris Wilson wrote:
> > From: Akash Goel
> >
> > This provides support for the drivers or shmem file owners to register
> > a set of callbacks, which can be invoked from the address space
> >
On Mon, Apr 18, 2016 at 02:20:43PM +0200, Tomeu Vizoso wrote:
> On 18 April 2016 at 14:09, Daniel Vetter wrote:
> > On Wed, Mar 09, 2016 at 04:57:37PM +0100, Tomeu Vizoso wrote:
> >> From: Micah Fedke
> >>
> >> This function uses an intel-specific ioctl to fetch a mapping between
> >> pipes and
On Wed, Apr 20, 2016 at 04:47:35PM +0530, ankitprasad.r.sha...@intel.com wrote:
> static int
> i915_gem_shmem_pread(struct drm_device *dev,
>struct drm_i915_gem_object *obj,
> @@ -656,6 +795,9 @@ i915_gem_shmem_pread(struct drm_device *dev,
> int needs_clflush = 0;
>
On Wed, Apr 20, 2016 at 04:47:29PM +0530, ankitprasad.r.sha...@intel.com wrote:
> From: Chris Wilson
>
> This utility function is a companion to i915_gem_object_get_page() that
> uses the same cached iterator for the scatterlist to perform fast
> sequential lookup of the dma address associated wi
On ke, 2016-04-20 at 12:04 +0100, Chris Wilson wrote:
> Since we can only swap out shmemfs objects, those are the only ones that
> influence the ability of the shrinker to can free pages. Currently, all
> non-shmemfs objects have a raised pages_pin_count to protect them from
> the shrinker, so this
On Wed, Apr 20, 2016 at 04:47:28PM +0530, ankitprasad.r.sha...@intel.com wrote:
> From: Chris Wilson
>
> Introduced a new vm specfic callback insert_page() to program a single pte in
> ggtt or ppgtt. This allows us to map a single page in to the mappable aperture
> space. This can be iterated ove
This patch restricts usage of live status check for HDMI detection.
While testing certain (monitor + cable) combinations with various
intel platforms, it seems that live status register is not reliable
on some older devices. So limit the live_status check from VLV onwards.
This fixes regression a
On ke, 2016-04-20 at 12:04 +0100, Chris Wilson wrote:
> Inside the shrinker we call can_release_pages() to indicate whether or
> not we can make forward progress in freeing up memory by unbinding that
> object. When adding our report to oom, we should be using the same
> logic.
>
> Whilst here, ch
On Wed, Apr 20, 2016 at 11:24 AM, Daniel Vetter wrote:
> On Wed, Apr 20, 2016 at 10:26 AM, Lu, Han wrote:
>> Hi Jani,
>>
>> Please find the repo and commit id below:
>> Repo: https://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux.git
>> Branch: x86
>> Commit id (begin from earliest):
>>
Chris Wilson writes:
> [ text/plain ]
> Since we can only swap out shmemfs objects, those are the only ones that
> influence the ability of the shrinker to can free pages. Currently, all
> non-shmemfs objects have a raised pages_pin_count to protect them from
> the shrinker, so this just makes th
Chris Wilson writes:
> [ text/plain ]
> Inside the shrinker we call can_release_pages() to indicate whether or
> not we can make forward progress in freeing up memory by unbinding that
> object. When adding our report to oom, we should be using the same
> logic.
>
> Whilst here, change the report
From: Chris Wilson
Ville reminded us that stolen memory is not preserved across
hibernation, and a result of this was that context objects now being
allocated from stolen were being corrupted on S4 and promptly hanging
the GPU on resume.
We want to utilise stolen for as much as possible (nothing
From: Ankitprasad Sharma
When constructing a batchbuffer, it is sometimes crucial to know the
largest hole into which we can fit a fenceable buffer (for example when
handling very large objects on gen2 and gen3). This depends on the
fragmentation of pinned buffers inside the aperture, a question
From: Ankitprasad Sharma
This patch extends the GET_APERTURE ioctl to add support
for getting total size and available size of the stolen region
as well as single largest block available in the stolen region.
Also adds debugfs support to retieve the size information of the
stolen area.
v2: respi
From: Ankitprasad Sharma
The BIOS RapidStartTechnology may corrupt the stolen memory across S3
suspend due to unalarmed hibernation, in which case we will not be able
to preserve the User data stored in the stolen region. Hence this patch
tries to identify presence of the RST device on the ACPI b
From: Chris Wilson
If we run out of stolen memory when trying to allocate an object, see if
we can reap enough purgeable objects to free up enough contiguous free
space for the allocation. This is in principle very much like evicting
objects to free up enough contiguous space in the vma when bind
From: Ankitprasad Sharma
Extend the drm_i915_gem_create structure to add support for
creating Stolen memory backed objects. Added a new flag through
which user can specify the preference to allocate the object from
stolen memory, which if set, an attempt will be made to allocate
the object from s
From: Ankitprasad Sharma
This patch adds support for extending the pread/pwrite functionality
for objects not backed by shmem. The access will be made through
gtt interface. This will cover objects backed by stolen memory as well
as other non-shmem backed objects.
v2: Drop locks around slow_user
From: Ankitprasad Sharma
This patch adds support for clearing buffer objects via CPU/GTT. This
is particularly useful for clearing out the non shmem backed objects.
Currently intend to use this only for buffers allocated from stolen
region.
v2: Added kernel doc for i915_gem_clear_object(), corre
From: Ankitprasad Sharma
In pwrite_fast, map an object page by page if obj_ggtt_pin fails. First,
we try a nonblocking pin for the whole object (since that is fastest if
reused), then failing that we try to grab one page in the mappable
aperture. It also allows us to handle objects larger than th
From: Chris Wilson
This utility function is a companion to i915_gem_object_get_page() that
uses the same cached iterator for the scatterlist to perform fast
sequential lookup of the dma address associated with any page within the
object.
Signed-off-by: Chris Wilson
Signed-off-by: Ankitprasad Sh
From: Chris Wilson
Introduced a new vm specfic callback insert_page() to program a single pte in
ggtt or ppgtt. This allows us to map a single page in to the mappable aperture
space. This can be iterated over to access the whole object by using space as
meagre as page size.
v2: Added low level r
From: Ankitprasad Sharma
This patch series adds support for creating/using Stolen memory backed
objects.
Despite being a unified memory architecture (UMA) some bits of memory
are more equal than others. In particular we have the thorny issue of
stolen memory, memory stolen from the system by the
From: Ankitprasad Sharma
Propagating correct error codes to userspace by using ERR_PTR and
PTR_ERR macros for stolen memory based object allocation. We generally
return -ENOMEM to the user whenever there is a failure in object
allocation. This patch helps user to identify the correct reason for t
On Wed, 20 Apr 2016, Mika Kahola wrote:
> CI hasn't catch this patch for testing?
It has http://mid.gmane.org/20160413170704.11600.78...@emeril.freedesktop.org
>
> On Wed, 2016-04-13 at 12:11 +0300, Mika Kahola wrote:
>> It was noticed on bug #94087 that module parameter
>> i915.edp_vswing=2 tha
On Tue, Apr 19, 2016 at 02:31:13PM -0300, Ezequiel Garcia wrote:
> Currently, our implementation of drm_connector_funcs.detect is
> based on getting a valid EDID.
>
> This requirement makes the driver fail to detect connected
> connectors in case of EDID corruption, which in turn prevents
> from f
CI hasn't catch this patch for testing?
On Wed, 2016-04-13 at 12:11 +0300, Mika Kahola wrote:
> It was noticed on bug #94087 that module parameter
> i915.edp_vswing=2 that should override the VBT setting
> to use default voltage swing (400 mV) was not applied
> for Broadwell.
>
> This patch provi
On Mon, Apr 18, 2016 at 07:03:23PM +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> There doesn't seem to be any need to for the force trigger trick every
> single time on VLV CRT output. Let's do it the same way as on ILK+, and
> do the force trigger only on the first detect
On Wed, Apr 20, 2016 at 11:10:54AM +0200, Luis R. Rodriguez wrote:
> Reason I ask is since I noticed a while ago a lot of drivers
> were using info->fix.smem_start and info->fix.smem_len consistently
> for their ioremap'd areas it might make sense instead to let the
> internal framebuffer (register
Since we can only swap out shmemfs objects, those are the only ones that
influence the ability of the shrinker to can free pages. Currently, all
non-shmemfs objects have a raised pages_pin_count to protect them from
the shrinker, so this just makes the logic for can_release_pages()
clearer (and saf
When iterating over the bound list, we expect all objects there to have
their pages pinned (by the bound VMA). So only report those objects with
additional pin count on their pages as "pinned". These should be those
objects used for display and hardware access.
Reported-by: Akash Goel
Signed-off-
Inside the shrinker we call can_release_pages() to indicate whether or
not we can make forward progress in freeing up memory by unbinding that
object. When adding our report to oom, we should be using the same
logic.
Whilst here, change the reporting from bytes to pages so that it looks
smaller to
Since we can only swap out shmemfs objects, those are the only ones that
influence the ability of the shrinker to can free pages. Currently, all
non-shmemfs objects have a raised pages_pin_count to protect them from
the shrinker, so this just makes the logic for can_release_pages()
clearer (and saf
When iterating over the bound list, we expect all objects there to have
their pages pinned (by the bound VMA). So only report those objects with
additional pin count on their pages as "pinned". These should be those
objects used for display and hardware access.
Reported-by: Akash Goel
Signed-off-
Inside the shrinker we call can_release_pages() to indicate whether or
not we can make forward progress in freeing up memory by unbinding that
object. When adding our report to oom, we should be using the same
logic.
Whilst here, change the reporting from bytes to pages so that it looks
smaller to
== Series Details ==
Series: lib: Always NUL terminate ucs2_as_utf8
URL : https://patchwork.freedesktop.org/series/5969/
State : success
== Summary ==
Series 5969v1 lib: Always NUL terminate ucs2_as_utf8
http://patchwork.freedesktop.org/api/1.0/series/5969/revisions/1/mbox/
bdw-nuci7
When writing a patch that adds a igt_warn() when lockdep is unavailable
I noticed that the warn error doesn't work any more. Fix this by
monitoring
stderr, and only setting 'pass' when stderr is empty.
Signed-off-by: Maarten Lankhorst
Cc: Dylan Baker
[mlankhorst: Changed to not use len over a po
On Mon, Apr 18, 2016 at 08:39:00PM +0100, Emil Velikov wrote:
> On 18 April 2016 at 16:53, Daniel Vetter wrote:
> > On Mon, Apr 18, 2016 at 03:40:11PM +0100, Emil Velikov wrote:
> >> On 18 April 2016 at 13:36, Daniel Vetter wrote:
> >> > On Mon, Apr 18, 2016 at 12:09:51PM +0100, Lionel Landwerlin
== Series Details ==
Series: drm/i915/shrinker: Only report objects with extra pinned pages as pinned
URL : https://patchwork.freedesktop.org/series/5968/
State : success
== Summary ==
Series 5968v1 drm/i915/shrinker: Only report objects with extra pinned pages as
pinned
http://patchwork.free
Cache DisplayPort signal levels including voltage swing and
pre-emphasis. After resume, the DP link is re-trained by trying
previusly computed voltage swing and pre-emphasis. In case, we
are not able to train the link by using these settings, the link
training is restarted. Now, the link is trained
On Wed, 20 Apr 2016, Colin Kincaid Williams wrote:
> bump
I'm not sure you properly appreciate the amount of email people here
handle.
> On Tue, Apr 19, 2016 at 4:05 AM, Colin Kincaid Williams
> wrote:
>> Hello,
>>
>> I tried to find the appropriate place to file a bug report. On my dell
>> la
On Wed, Apr 20, 2016 at 11:36:37AM +0200, Laszlo Ersek wrote:
> On 04/20/16 10:37, Chris Wilson wrote:
> > If the caller, in this case efivarfs_callback(), only provides sufficent
> > room for the expanded utf8 and not enough to include the terminating NUL
> > byte, that NUL byte is skipped.
>
> H
On 19/04/16 20:50, Chris Wilson wrote:
On Tue, Apr 19, 2016 at 06:40:07PM +0100, Dave Gordon wrote:
From: Alex Dai
The recently-added i915_gem_object_pin_map() can be further optimised
for "small" objects. To facilitate this, and simplify the error paths
before adding the new code, this patch
On Wed, Apr 20, 2016 at 11:10:54AM +0200, Luis R. Rodriguez wrote:
> On Tue, Apr 19, 2016 at 01:33:58PM +0100, Chris Wilson wrote:
> > diff --git a/drivers/gpu/drm/i915/i915_gem.c
> > b/drivers/gpu/drm/i915/i915_gem.c
> > index 6ce2c31b9a81..9ef47329e8ae 100644
> > --- a/drivers/gpu/drm/i915/i915_
On 04/20/16 10:37, Chris Wilson wrote:
> If the caller, in this case efivarfs_callback(), only provides sufficent
> room for the expanded utf8 and not enough to include the terminating NUL
> byte, that NUL byte is skipped.
How does that occur? In efivarfs_callback() [fs/efivarfs/super.c], we have
On Tue, Apr 19, 2016 at 01:33:58PM +0100, Chris Wilson wrote:
> diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
> index 6ce2c31b9a81..9ef47329e8ae 100644
> --- a/drivers/gpu/drm/i915/i915_gem.c
> +++ b/drivers/gpu/drm/i915/i915_gem.c
> @@ -3346,6 +3346,15 @@ static vo
On 19/04/16 18:24, Patchwork wrote:
== Series Details ==
Series: series starting with [resend-for-CI,1/3] drm/i915/gen8+: Do not enable
DPF interrupt since the handler does not exist
URL : https://patchwork.freedesktop.org/series/5944/
State : failure
== Summary ==
Series 5944v1 Series wit
On 19/04/16 16:54, Patchwork wrote:
== Series Details ==
Series: drm/i915/gen9: implement WaEnableSamplerGPGPUPreemptionSupport (rev4)
URL : https://patchwork.freedesktop.org/series/5367/
State : success
== Summary ==
Series 5367v4 drm/i915/gen9: implement WaEnableSamplerGPGPUPreemptionSupp
== Series Details ==
Series: drm/i915/shrinker: Only report objects with extra pinned pages as pinned
URL : https://patchwork.freedesktop.org/series/5968/
State : success
== Summary ==
Series 5968v1 drm/i915/shrinker: Only report objects with extra pinned pages as
pinned
http://patchwork.free
On ke, 2016-04-20 at 08:43 +0100, Chris Wilson wrote:
> When iterating over the bound list, we expect all objects there to have
> their pages pinned (by the bound VMA). So only add those objects with
> additional pin count on their pages as "pinned". These should be those
> objects used for display
On Tue, 19 Apr 2016, cac...@quantum-sci.com wrote:
> Confirmed: Xen is completely busted for Skylake. Kernel can't boot due
> to i915 core dump.
Please file a bug at [1] with the dmesg attached.
BR,
Jani.
[1] https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel
PS. I'd u
On 19/04/16 17:38, Dave Gordon wrote:
On 19/04/16 16:08, Tvrtko Ursulin wrote:
On 19/04/16 12:45, Dave Gordon wrote:
Tidying up guc_init_proc_desc() and adding commentary to the client
structure after the recent change in GuC page mapping strategy.
Signed-off-by: Dave Gordon
---
drivers/g
If the caller, in this case efivarfs_callback(), only provides sufficent
room for the expanded utf8 and not enough to include the terminating NUL
byte, that NUL byte is skipped. When the caller then interprets it as a
string, it may then read from past its allocated memory:
[ 170.605647] WARNING:
== Series Details ==
Series: drm/i915/shrinker: Only report objects with extra pinned pages as pinned
URL : https://patchwork.freedesktop.org/series/5968/
State : failure
== Summary ==
Series 5968v1 drm/i915/shrinker: Only report objects with extra pinned pages as
pinned
http://patchwork.free
When iterating over the bound list, we expect all objects there to have
their pages pinned (by the bound VMA). So only add those objects with
additional pin count on their pages as "pinned". These should be those
objects used for display and hardware access.
Reported-by: Akash Goel
Signed-off-by:
201 - 300 of 303 matches
Mail list logo