Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic_helper.c | 302 +++-
1 file changed, 157 insertions(+), 145 deletions(-)
diff --git a/drivers/gpu/drm/drm_atomic_helper.c
b/drivers/gpu/drm/drm_atomic_helper.c
index fcb6e5b55217..c8aba493fc17 100644
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.c | 4 ++--
drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c | 6 +++---
drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.h | 3 ++-
drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c | 4 ++--
drivers/gpu/drm/msm/msm_atomic.c | 16
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/imx/imx-drm-core.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/imx/imx-drm-core.c
b/drivers/gpu/drm/imx/imx-drm-core.c
index 98df09c2b388..d484af773460 100644
--- a/drivers/gpu/drm/imx/imx-drm
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
index 9b17a66cf0e1..b5af77097033 100644
--- a/dr
With the old users of for_each_obj_in_state gone, we can rename
for_each_oldnew_obj_in_state back to that name. It's slightly less
ugly.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic_helper.c | 34 ++---
drivers/gpu/drm/drm_blend.c
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
index c7eba305c488..5c0625e5dca1 100644
--- a/drive
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/rcar-du/rcar_du_kms.c | 2 +-
drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_kms.c
b/drivers/gpu/drm/rcar-du/rcar_du_kms.c
index c76ed9ee6a01..
Add for_each_(old)(new)_(plane,connector,crtc)_in_state iterators to
replace the old for_each_xxx_in_state ones. This is useful for >1 flip
depth and getting rid of all xxx->state dereferences.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic.c | 6 +++
drivers/gpu/drm/dr
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_blend.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/drm_blend.c b/drivers/gpu/drm/drm_blend.c
index 85172a977bf3..70c03eb032fc 100644
--- a/drivers/gpu/drm/drm_blend.c
+++ b/d
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/vc4/vc4_kms.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/vc4/vc4_kms.c b/drivers/gpu/drm/vc4/vc4_kms.c
index f31f72af8551..76cc9b374215 100644
--- a/drivers/gpu/drm/vc4/vc4_kms.c
+++ b/drivers/gpu/
This kills another dereference of connector->state. connector_mask
holds all unchanged connectors at least and any changed connectors
are already in state anyway.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
index 1915ec44f7eb..6672ea93ee73 100644
--- a/drivers/gpu/drm/drm_atomic.c
+++ b/drivers/gpu
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/omapdrm/omap_drv.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/omapdrm/omap_drv.c
b/drivers/gpu/drm/omapdrm/omap_drv.c
index 1735c7accf72..74f9519878a2 100644
--- a/drivers/gpu/drm/omapdrm/omap_d
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/i915/intel_pm.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 2df06b703e3d..163b73b493bf 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++
Add for_each_(old,new)_(crtc,connector,plane)_in_state iterators, and
re-use for_each_*_in_state to be a iterator with the old and the new
state.
This will be less confusing than the previous case, in which
for_each_*_in_state could refer to the old or the new state.
The oldnew macro names are te
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/i915/intel_fbc.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
index faa67624e1ed..0028335fc1bb 100644
--- a/drivers/gpu/drm/i915/intel_fbc.c
+++ b/
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/mediatek/mtk_drm_drv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_drm_drv.c
b/drivers/gpu/drm/mediatek/mtk_drm_drv.c
index db61aa5f32ef..414e848d8cbf 100644
--- a/drivers/gpu/drm/mediat
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.c
b/drivers/gpu/drm/exynos/exynos_drm_drv.c
index 4a21a745c373..fc54f3f0a42c 100644
--- a/drivers/gpu/drm/exynos/
On Mon, Oct 17, 2016 at 05:34:39PM +0530, Shashank Sharma wrote:
> HDMI 2.0/CEA-861-F introduces two new aspect ratios:
> - 64:27
> - 256:135
>
> This patch adds enumeration for the new aspect ratios
> in the existing aspect ratio list.
>
> Signed-off-by: Shashank Sharma
> Reviewed-by: Sean Paul
Add gem_exec_suspend/basic-s3-devices and basic-s4-devices subtests to
BAT. At the same time remove basic-s4 from the list, which is atm
implicitly disabled via HIBERNATION=n in kconfig. We would need at least
basic S4 coverage provided by basic-s4-devices, which requires
HIBERNATION=y.
Signed-off
== Series Details ==
Series: Picture aspect ratio support in DRM layer
URL : https://patchwork.freedesktop.org/series/13868/
State : failure
== Summary ==
Series 13868v1 Picture aspect ratio support in DRM layer
https://patchwork.freedesktop.org/api/1.0/series/13868/revisions/1/mbox/
Test gem
tree: git://anongit.freedesktop.org/drm-intel drm-intel-next-queued
head: 06a75ace46e2fdd1d93b06228df0e2dfe526cc27
commit: 06a75ace46e2fdd1d93b06228df0e2dfe526cc27 [1/1] Merge tag
'gvt-next-2016-10-14' of https://github.com/01org/gvt-linux into
drm-intel-next-queued
config: x86_64-randconfig-
Regards
Shashank
On 10/17/2016 5:55 PM, Daniel Vetter wrote:
On Mon, Oct 17, 2016 at 05:34:36PM +0530, Shashank Sharma wrote:
This patch series adds 4 patches.
- The first two patches add aspect ratio support in DRM layes
- Next two patches add new aspect ratios defined in CEA-861-F
suppor
> == Series Details ==
>
> Series: Picture aspect ratio support in DRM layer
> URL : https://patchwork.freedesktop.org/series/13868/
> State : failure
>
> == Summary ==
>
> Series 13868v1 Picture aspect ratio support in DRM layer
> https://patchwork.freedesktop.org/api/1.0/series/13868/revisio
== Series Details ==
Series: drm/atomic: Use less confusing iterator names.
URL : https://patchwork.freedesktop.org/series/13869/
State : warning
== Summary ==
Series 13869v1 drm/atomic: Use less confusing iterator names.
https://patchwork.freedesktop.org/api/1.0/series/13869/revisions/1/mbox/
Regards
Shashank
On 10/17/2016 6:01 PM, Ville Syrjälä wrote:
On Mon, Oct 17, 2016 at 05:34:38PM +0530, Shashank Sharma wrote:
Current DRM layer functions don't parse aspect ratio information
while converting a user mode->kernel mode or vice versa. This
causes modeset to pick mode with wrong a
On 17/10/2016 12:31, Chris Wilson wrote:
On Mon, Oct 17, 2016 at 11:55:54AM +0100, Tvrtko Ursulin wrote:
On 14/10/2016 13:18, Chris Wilson wrote:
static void
-i915_gem_object_put_pages_gtt(struct drm_i915_gem_object *obj)
+i915_gem_object_put_pages_gtt(struct drm_i915_gem_object *obj,
+
Regards
Shashank
On 10/17/2016 6:04 PM, Jani Nikula wrote:
On Fri, 14 Oct 2016, "Sharma, Shashank" wrote:
Regards
Shashank
On 10/14/2016 8:02 PM, Jani Nikula wrote:
On Fri, 14 Oct 2016, Shashank Sharma wrote:
Many GEN9 boards come with on-board lspcon cards.
Fot these boards, VBT confi
On Mon, Oct 17, 2016 at 07:10:42PM +0530, Sharma, Shashank wrote:
> Regards
>
> Shashank
>
>
> On 10/17/2016 6:01 PM, Ville Syrjälä wrote:
> > On Mon, Oct 17, 2016 at 05:34:38PM +0530, Shashank Sharma wrote:
> >> Current DRM layer functions don't parse aspect ratio information
> >> while convert
On Mon, 17 Oct 2016, Daniel Vetter wrote:
> Ok applied, but a few things to keep in mind before your next pull
> request:
>
> - Dont rebase everything 5 seconds before sending out the pull request.
> That just invalidates all the testing you've done, so not a good idea.
> In general try to avo
On Mon, Oct 17, 2016 at 03:45:47PM +0800, Zhenyu Wang wrote:
> On 2016.10.17 09:33:19 +0200, Daniel Vetter wrote:
> > On Mon, Oct 17, 2016 at 09:30:45AM +0200, Daniel Vetter wrote:
> > > On Fri, Oct 14, 2016 at 06:30:30PM +0800, Zhenyu Wang wrote:
> > > >
> > > > Hi,
> > > >
> > > > This is first
On Mon, Oct 17, 2016 at 02:51:01PM +0100, Tvrtko Ursulin wrote:
>
> On 17/10/2016 12:31, Chris Wilson wrote:
> >On Mon, Oct 17, 2016 at 11:55:54AM +0100, Tvrtko Ursulin wrote:
> >>On 14/10/2016 13:18, Chris Wilson wrote:
> >>> static void
> >>>-i915_gem_object_put_pages_gtt(struct drm_i915_gem_ob
On Fri, Oct 14, 2016 at 08:02:52PM +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> This series should reduce the underrn spam in CI. I expect it won't eliminate
> it entirely
> as we seem to have two classes of dmesg warns: from link retraining, and from
> normal
> modeset
On Mon, Oct 17, 2016 at 03:33:43PM +0300, Joonas Lahtinen wrote:
> On pe, 2016-10-14 at 14:44 +0100, Chris Wilson wrote:
> > If the kernel is old, more than a few releases old, chances are that the
> > user is using an old kernel for a good reason, despite there being GPU
> > hangs. After 180days s
When handling execbuf relocations, we play a delicate dance with
pagefault. We first try to access the user pages underneath our
struct_mutex. However, if those pages were inside a GEM object, we may
trigger a pagefault and deadlock as i915_gem_fault() tries to
recursively acquire struct_mutex. Ins
On Wed, Oct 12, 2016 at 01:49:20PM -, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915: GMBUS don't need no forcewake
> URL : https://patchwork.freedesktop.org/series/13641/
> State : warning
>
> == Summary ==
>
> Series 13641v1 drm/i915: GMBUS don't need no forcewake
> https:/
On Fri, Oct 14, 2016 at 05:51:14PM +0300, Imre Deak wrote:
> While at it fix the order of states for consistency.
>
> Suggested by Daniel.
>
> Cc: Daniel Vetter
> Signed-off-by: Imre Deak
Going ocd with docs makes me really happy \o/ And I think it does help
for new folks to get started in our
On ma, 2016-10-17 at 16:10 +0200, Daniel Vetter wrote:
> On Mon, Oct 17, 2016 at 03:33:43PM +0300, Joonas Lahtinen wrote:
> > Maybe we could even explicitly state that bugs should be reported to
> > the distro bugzilla because of running an old kernel?
>
> Distro's already shut down our warnings "
On Mon, Oct 17, 2016 at 02:28:37PM +0300, Mika Kahola wrote:
> + for (int i = 0; i < iterations; i++) {
> + igt_info("%d/%d: Testing connector %s using pipe %s with %d
> planes\n",
> + i+1, iterations, igt_output_name(output),
> + kmstest_p
On Mon, Oct 17, 2016 at 03:39:04PM +0300, Imre Deak wrote:
> Add gem_exec_suspend/basic-s3-devices and basic-s4-devices subtests to
> BAT. At the same time remove basic-s4 from the list, which is atm
> implicitly disabled via HIBERNATION=n in kconfig. We would need at least
> basic S4 coverage prov
On ma, 2016-10-17 at 16:32 +0200, Daniel Vetter wrote:
> On Mon, Oct 17, 2016 at 03:39:04PM +0300, Imre Deak wrote:
> > Add gem_exec_suspend/basic-s3-devices and basic-s4-devices subtests
> > to
> > BAT. At the same time remove basic-s4 from the list, which is atm
> > implicitly disabled via HIBERN
Regards
Shashank
On 10/17/2016 7:36 PM, Ville Syrjälä wrote:
On Mon, Oct 17, 2016 at 07:10:42PM +0530, Sharma, Shashank wrote:
Regards
Shashank
On 10/17/2016 6:01 PM, Ville Syrjälä wrote:
On Mon, Oct 17, 2016 at 05:34:38PM +0530, Shashank Sharma wrote:
Current DRM layer functions don't p
On Mon, 17 Oct 2016, Imre Deak wrote:
> On ma, 2016-10-17 at 16:32 +0200, Daniel Vetter wrote:
>> On Mon, Oct 17, 2016 at 03:39:04PM +0300, Imre Deak wrote:
>> > Add gem_exec_suspend/basic-s3-devices and basic-s4-devices subtests
>> > to
>> > BAT. At the same time remove basic-s4 from the list, wh
On Mon, Oct 17, 2016 at 08:21:21PM +0530, Sharma, Shashank wrote:
> Regards
>
> Shashank
>
>
> On 10/17/2016 7:36 PM, Ville Syrjälä wrote:
> > On Mon, Oct 17, 2016 at 07:10:42PM +0530, Sharma, Shashank wrote:
> >> Regards
> >>
> >> Shashank
> >>
> >>
> >> On 10/17/2016 6:01 PM, Ville Syrjälä wro
HI,
> -Original Message-
> From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Imre Deak
> Sent: Monday, October 17, 2016 5:47 PM
> To: Daniel Vetter
> Cc: intel-gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH] intel-ci: Add gem_exec_suspend/basic-
On ma, 2016-10-17 at 18:01 +0300, Saarinen, Jani wrote:
> HI,
> > -Original Message-
> > From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On
> > Behalf
> > Of Imre Deak
> > Sent: Monday, October 17, 2016 5:47 PM
> > To: Daniel Vetter
> > Cc: intel-gfx@lists.freedesktop.org
Regards
Shashank
On 10/17/2016 8:30 PM, Ville Syrjälä wrote:
On Mon, Oct 17, 2016 at 08:21:21PM +0530, Sharma, Shashank wrote:
Regards
Shashank
On 10/17/2016 7:36 PM, Ville Syrjälä wrote:
On Mon, Oct 17, 2016 at 07:10:42PM +0530, Sharma, Shashank wrote:
Regards
Shashank
On 10/17/2016
Reviewed-by: Rodrigo Vivi
On Fri, 2016-10-14 at 16:56 -0700, Anusha Srivatsa wrote:
> Make no_stepping_info an array of structs so that
> on plaforms that have only one binary of DMC, we can
> iterate through this array by having the same logic
> for firmware loads
>
> Cc: Rodrigo Vivi
> Signed
On Fri, 2016-10-14 at 16:56 -0700, Anusha Srivatsa wrote:
> Currently, for display there is only one DMC image for KBL.
> Remove the stepping_info table for KBL and use the no_stepping_info
> for loading the firmware image.
>
> Cc: Rodrigo Vivi
> Signed-off-by: Anusha Srivatsa
> ---
> drivers/g
== Series Details ==
Series: drm/i915: Restrict pagefault disabling to just around copy_from_user()
URL : https://patchwork.freedesktop.org/series/13880/
State : failure
== Summary ==
Series 13880v1 drm/i915: Restrict pagefault disabling to just around
copy_from_user()
https://patchwork.freed
On Tue, Oct 11, 2016 at 08:52:47PM +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Now that we use the AUX and GMBUS assignment from VBT for all ports,
> let's clean up the sanitization of the port information a bit.
> Previosuly we only did this for port E, and only complai
From: Ville Syrjälä
Now that we use the AUX and GMBUS assignment from VBT for all ports,
let's clean up the sanitization of the port information a bit.
Previosuly we only did this for port E, and only complained about a
non-standard assignment for the other ports. But as we know that
non-standard
== Series Details ==
Series: drm/i915: Trust VBT aux/ddc information (rev2)
URL : https://patchwork.freedesktop.org/series/13600/
State : failure
== Summary ==
Series 13600v2 drm/i915: Trust VBT aux/ddc information
https://patchwork.freedesktop.org/api/1.0/series/13600/revisions/2/mbox/
Test
On Fri, Oct 07, 2016 at 05:28:57PM -0300, Paulo Zanoni wrote:
> Luckily, the necessary adjustments for when we're using the scaler are
> exactly the same as the ones needed on ILK+, so just reuse the
> function we already have.
>
> v2: Invert the patch order so stable backports get easier.
>
> Cc
Output_check_status() should return a false if it detects
that the connector mode list has changed so that sna_mode_discover
can reprobe.
Fixes: eb01cc549d4d (sna: Refresh mode list if the kernel updates)
Signed-off-by: Manasi Navare
Cc: Chris Wilson
Cc: Ville Syrjala
---
src/sna/sna_display.
Hi Ville,
On Friday 22 Jul 2016 16:43:13 ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Now that all drivers have been converted over to the per-plane rotation
> property, we can just nuke the global rotation property.
>
> Signed-off-by: Ville Syrjälä
Stupid question, but how
tree: git://anongit.freedesktop.org/drm-intel for-linux-next
head: bfd02b3c557caa083be0d55a3164ede706a446e1
commit: 06a75ace46e2fdd1d93b06228df0e2dfe526cc27 [0/3] Merge tag
'gvt-next-2016-10-14' of https://github.com/01org/gvt-linux into
drm-intel-next-queued
config: x86_64-randconfig-s0-1018
mmio_base' in something not a structure or union
(gvt->dev_priv->engine[ring_id].mmio_base + (offset))
^
drivers/gpu/drm/i915/gvt/execlist.c:798:23: note: in expansion of macro
'execlist_ring_mmio'
ctx_
On Thu, Oct 06, 2016 at 10:54:01AM -0700, Rodrigo Vivi wrote:
> According to spec: "KBL re-uses SKL values, except where
> specific KBL values are listed."
>
> And recently spec has changed adding different table for Display Port only.
> But for all SKUs (H,S,U,Y) we have slightly different values
Change GVT-g code reference for intel_engine_cs from static array to
allocated pointer after commit 3b3f1650b1ca ("drm/i915: Allocate
intel_engine_cs structure only for the enabled engines").
Signed-off-by: Zhenyu Wang
---
drivers/gpu/drm/i915/gvt/execlist.c | 2 +-
drivers/gpu/drm/i915/gvt/han
On 2016.10.17 16:07:50 +0200, Daniel Vetter wrote:
> Yeah might be best to have a new branch with upstreaming stuff (now you
> need to at least split out bugfixes for the already merged stuff) and
> treat that like a mostly stable branch. And the still unmerged features
> (vfio and all that) would
== Series Details ==
Series: drm/i915/gvt: Fix build failure after intel_engine_cs change
URL : https://patchwork.freedesktop.org/series/13910/
State : failure
== Summary ==
Series 13910v1 drm/i915/gvt: Fix build failure after intel_engine_cs change
https://patchwork.freedesktop.org/api/1.0/se
Added support to print SKL watermark and DDB registers.
Signed-off-by: Dhinakaran Pandiyan
---
tools/intel_watermark.c | 104 +++-
1 file changed, 103 insertions(+), 1 deletion(-)
diff --git a/tools/intel_watermark.c b/tools/intel_watermark.c
index e9
> == Series Details ==
>
> Series: drm/i915/gvt: Fix build failure after intel_engine_cs change
> URL : https://patchwork.freedesktop.org/series/13910/
> State : failure
>
> == Summary ==
>
> Series 13910v1 drm/i915/gvt: Fix build failure after intel_engine_cs change
> https://patchwork.freede
On 2016.10.18 05:39:11 +, Saarinen, Jani wrote:
> > == Series Details ==
> >
> > Series: drm/i915/gvt: Fix build failure after intel_engine_cs change
> > URL : https://patchwork.freedesktop.org/series/13910/
> > State : failure
> >
> > == Summary ==
> >
> > Series 13910v1 drm/i915/gvt: Fix
&(wa_ctx->indirect_ctx.obj->base));
> ^
> In file included from drivers/gpu/drm/i915/gvt/execlist.c:35:0:
> drivers/gpu/drm/i915/i915_drv.h:2344:13: note: declared here
> extern void drm_gem_object_unreference(struct drm_gem_object *);
> ^
> drivers/gpu/drm/i915/gvt
On ti, 2016-10-18 at 09:40 +0800, Zhenyu Wang wrote:
> @@ -134,7 +134,7 @@ static int render_mmio_to_ring_id(struct intel_gvt *gvt,
> unsigned int reg)
>
> reg &= ~GENMASK(11, 0);
> for (i = 0; i < I915_NUM_ENGINES; i++) {
> - if (gvt->dev_priv->engine[i].mmio_base == reg
On 2016.10.18 09:47:56 +0300, Joonas Lahtinen wrote:
> On ti, 2016-10-18 at 09:40 +0800, Zhenyu Wang wrote:
> > @@ -134,7 +134,7 @@ static int render_mmio_to_ring_id(struct intel_gvt
> > *gvt, unsigned int reg)
> >
> > reg &= ~GENMASK(11, 0);
> > for (i = 0; i < I915_NUM_ENGINES; i++) {
On Sun, 16 Oct 2016, zieg...@uni-freiburg.de wrote:
> If I hibernate from the console, thaw, and switch between the console
> and X, the gpu hangs itself. The machine becomes unresponsive except
> for the power button.
Please file a bug at [1], and attach the error state there.
Thanks,
Jani.
[1
On Fri, Oct 14, 2016 at 06:30:30PM +0800, Zhenyu Wang wrote:
>
> Hi,
>
> This is first pull request to merge GVT-g device model in i915
> which contains core GVT-g device model work to virtualize GPU
> resources. This tries to add feature of Intel GVT-g technology
> for full GPU virtualization. T
On pe, 2016-10-14 at 14:55 +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> I have re-ordered some struct members in patch:
>
> commit 44a655cae3043453f9dd8076538712d52e2e0ce4
> > Author: Tvrtko Ursulin
> Date: Thu Oct 13 11:09:23 2016 +0100
>
> drm/i915: Shrink cxsr_laten
On Mon, Oct 17, 2016 at 09:30:45AM +0200, Daniel Vetter wrote:
> On Fri, Oct 14, 2016 at 06:30:30PM +0800, Zhenyu Wang wrote:
> >
> > Hi,
> >
> > This is first pull request to merge GVT-g device model in i915
> > which contains core GVT-g device model work to virtualize GPU
> > resources. This tr
On Sun, Oct 16, 2016 at 10:51:30PM -0700, Manasi Navare wrote:
> Hi,
>
> I am working on adding a link rate fallback in case of link training failure.
> If the link training fails at a specific link rate/lane count in atomic
> commit phase, I validate the modes again based on the link rate lower t
On pe, 2016-10-14 at 12:05 +0100, Chris Wilson wrote:
> Now that we have reduced the access to the list to either (a) under the
> struct_mutex whilst holding the RPM wakeref (so that concurrent writers to
> the list are serialised by struct_mutex) and (b) under the atomic
> runtime suspend (which c
> r-b with that bikeshed addressed or not?
Hello Daniel,
I have already acknowledged this comment, and I am publishing next set today
with rebase, addressing this comment.
So you can consider this one done.
Regards
Shashank
-Original Message-
From: Daniel Vetter [mailto:daniel.vet...@f
On 2016.10.17 09:33:19 +0200, Daniel Vetter wrote:
> On Mon, Oct 17, 2016 at 09:30:45AM +0200, Daniel Vetter wrote:
> > On Fri, Oct 14, 2016 at 06:30:30PM +0800, Zhenyu Wang wrote:
> > >
> > > Hi,
> > >
> > > This is first pull request to merge GVT-g device model in i915
> > > which contains core
Internally we allow for using more objects than a single process can
allocate, i.e. we allow for a 64bit GPU address space even on a 32bit
system. Using size_t may oveerflow.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_debugfs.c | 2 +-
drivers/gpu/drm/i915/i915_drv.h | 2 +-
d
In many places, we try to count pages using a 32 bit integer. That
implies if we are asked to create an object larger than 43bits, we will
subtly crash much later. Catch this on the boundary, and add a warning
to remind ourselves later on our exabyte systems.
Signed-off-by: Chris Wilson
---
driv
The scattergather list uses a 32bit size counter, we should avoid
exceeding it.
v2: Also we should use unsigned int to match sg->length.
Fixes: 871dfbd67d4e ("drm/i915: Allow compaction upto SWIOTLB max segment size")
Signed-off-by: Chris Wilson
Reviewed-by: Tvrtko Ursulin # v1
---
drivers/gpu
Op 17-10-16 om 08:05 schreef Daniel Vetter:
> On Thu, Oct 13, 2016 at 05:04:03PM -0300, Paulo Zanoni wrote:
>> Em Qui, 2016-10-13 às 15:39 +0200, Maarten Lankhorst escreveu:
>>> Op 08-10-16 om 02:11 schreef Lyude:
Now that we've make skl_wm_levels make a little more sense, we can
remove a
On Sun, Oct 16, 2016 at 08:48:02PM +0200, Daniel Vetter wrote:
> Hi all,
>
> This is the first step to split out the drm-misc trees into their own git
> repo.
> This just moves the integration trees into drm-nightly.git. Those are separate
> since I really like the idea of sharing one integration
On Fri, Oct 14, 2016 at 08:33:37PM +, Pandiyan, Dhinakaran wrote:
> On Thu, 2016-10-13 at 21:44 +0300, Ville Syrjälä wrote:
> > On Thu, Oct 13, 2016 at 11:04:19AM -0700, Dhinakaran Pandiyan wrote:
> > > According to BSpec, cdclk has to be not less than 432 MHz with DP audio
> > > enabled, port
On Mon, Oct 17, 2016 at 08:55:51AM +0200, Daniel Vetter wrote:
> On Fri, Oct 14, 2016 at 07:27:39PM +, Pandiyan, Dhinakaran wrote:
> > On Thu, 2016-10-13 at 11:30 -0700, Jim Bride wrote:
> > > On Thu, Oct 13, 2016 at 11:04:19AM -0700, Dhinakaran Pandiyan wrote:
> > > > According to BSpec, cdclk
== Series Details ==
Series: series starting with [v2,1/3] drm/i915: Bump object bookkeeping to u64
from size_t
URL : https://patchwork.freedesktop.org/series/13853/
State : warning
== Summary ==
Series 13853v1 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/series/13853
On 14/10/2016 15:42, Chris Wilson wrote:
On Fri, Oct 14, 2016 at 03:35:59PM +0100, Tvrtko Ursulin wrote:
On 14/10/2016 14:53, Chris Wilson wrote:
We do pass NORETRY | NOWARN for the higher order allocations, so it
shouldn't be as bad it seems?
I don't know for sure without looking into the im
On ma, 2016-10-17 at 09:00 +0100, Chris Wilson wrote:
> Internally we allow for using more objects than a single process can
> allocate, i.e. we allow for a 64bit GPU address space even on a 32bit
> system. Using size_t may oveerflow.
>
> Signed-off-by: Chris Wilson
How might we get more than si
Signed-off-by: Petri Latvala
CC: Chris Wilson
---
(Possible duplicate, I didn't see this arrive on the mailing list myself)
To match Chris's intention.
The test itself will receive fixes later, and another module loading
test will be added at the end. This change is just for mitigation.
tes
On Mon, Oct 17, 2016 at 12:48:17PM +0300, Joonas Lahtinen wrote:
> On ma, 2016-10-17 at 09:00 +0100, Chris Wilson wrote:
> > Internally we allow for using more objects than a single process can
> > allocate, i.e. we allow for a 64bit GPU address space even on a 32bit
> > system. Using size_t may ov
On Mon, Oct 17, 2016 at 10:47:09AM +0100, Tvrtko Ursulin wrote:
>
> On 14/10/2016 15:42, Chris Wilson wrote:
> >On Fri, Oct 14, 2016 at 03:35:59PM +0100, Tvrtko Ursulin wrote:
> >>On 14/10/2016 14:53, Chris Wilson wrote:
> >We do pass NORETRY | NOWARN for the higher order allocations, so it
>
On 14/10/2016 15:07, Chris Wilson wrote:
On Fri, Oct 14, 2016 at 02:32:03PM +0100, Tvrtko Ursulin wrote:
On 14/10/2016 13:18, Chris Wilson wrote:
A while ago we switched from a contiguous array of pages into an sglist,
for that was both more convenient for mapping to hardware and avoided
the r
On Mon, Oct 17, 2016 at 12:53:10PM +0300, Petri Latvala wrote:
> Signed-off-by: Petri Latvala
> CC: Chris Wilson
> ---
>
> (Possible duplicate, I didn't see this arrive on the mailing list myself)
>
> To match Chris's intention.
>
> The test itself will receive fixes later, and another module
On Sun, Oct 16, 2016 at 10:51:30PM -0700, Manasi Navare wrote:
> Hi,
>
> I am working on adding a link rate fallback in case of link training failure.
> If the link training fails at a specific link rate/lane count in atomic
> commit phase, I validate the modes again based on the link rate lower t
On pe, 2016-10-14 at 19:56 +0530, Shashank Sharma wrote:
> As per the software design, we are driving lspcon in
> PCON mode. But while resuming from suspend, lspcon can go
> in LS mode (which is its default operating mode on power on)
>
> This patch adds a resume function for lspcon, which makes s
On 14/10/2016 10:43, Chris Wilson wrote:
On Fri, Oct 14, 2016 at 10:28:42AM +0100, Tvrtko Ursulin wrote:
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index 3c22d49005fe..271e63c8f037 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i91
On 14/10/2016 13:18, Chris Wilson wrote:
The plan is to move obj->pages out from under the struct_mutex into its
own per-object lock. We need to prune any assumption of the struct_mutex
from the get_pages/put_pages backends, and to make it easier we pass
around the sg_table to operate on rather
On Mon, Oct 17, 2016 at 10:56:27AM +0100, Tvrtko Ursulin wrote:
>
> On 14/10/2016 15:07, Chris Wilson wrote:
> >On Fri, Oct 14, 2016 at 02:32:03PM +0100, Tvrtko Ursulin wrote:
> >>On 14/10/2016 13:18, Chris Wilson wrote:
> >>>A while ago we switched from a contiguous array of pages into an sglist,
On Mon, Oct 17, 2016 at 11:52:27AM +0100, Tvrtko Ursulin wrote:
>
> On 14/10/2016 10:43, Chris Wilson wrote:
> >On Fri, Oct 14, 2016 at 10:28:42AM +0100, Tvrtko Ursulin wrote:
> >> diff --git a/drivers/gpu/drm/i915/i915_drv.h
> >> b/drivers/gpu/drm/i915/i915_drv.h
> >> index 3c22d49005fe..271e6
Current DRM layer functions don't parse aspect ratio information
while converting a user mode->kernel mode or vice versa. This
causes modeset to pick mode with wrong aspect ratio, eventually
causing failures in HDMI compliance test cases, due to wrong VIC.
This patch adds aspect ratio information
HDMI 2.0/CEA-861-F introduces two new aspect ratios:
- 64:27
- 256:135
This patch adds enumeration for the new aspect ratios
in the existing aspect ratio list.
Signed-off-by: Shashank Sharma
Reviewed-by: Sean Paul
Cc: Daniel Vetter
Cc: Emil Velikov
V2: rebase
V3: rebase
---
drivers/video/hd
HDMI 2.0/CEA-861-F introduces two new aspect ratios:
- 64:27
- 256:135
This patch:
- Adds new DRM flags for to represent these new aspect ratios.
- Adds new cases to handle these aspect ratios while converting
from user->kernel mode or vise versa.
Signed-off-by: Shashank Sharma
Reviewed-by: Se
1 - 100 of 135 matches
Mail list logo
