Re: [Intel-gfx] [PATCH] drm/i915: Sanity check the computed size and base of stolen memory

2017-01-30 Thread Chris Wilson
On Mon, Jan 30, 2017 at 02:41:15PM +0200, Joonas Lahtinen wrote: > On pe, 2017-01-27 at 17:20 +, Chris Wilson wrote: > > Just do a quick check that the stolen memory address range doesn't > > overflow our chosen integer type. > > > > Signed-off-by: Chris Wilson > > > > > @@ -189,7 +189,7 @

Re: [Intel-gfx] [PATCH v2 1/3] drm/i915: Make intel_detect_preproduction_hw easier to extend

2017-01-30 Thread Jani Nikula
On Mon, 30 Jan 2017, Joonas Lahtinen wrote: > On ma, 2017-01-30 at 10:44 +, Chris Wilson wrote: >> As we add new generations, we should keep detecting new pre-production >> system development platforms that were temporarily enabled to facilitate >> initial development and now superseded by pro

Re: [Intel-gfx] [PATCH v2 13/13] drm/i915: Acquire P-Unit access when modifying P-Unit settings

2017-01-30 Thread Ville Syrjälä
On Sat, Jan 28, 2017 at 06:18:45PM +0100, Hans de Goede wrote: > Hi, > > On 01/28/2017 05:25 PM, Hans de Goede wrote: > > Hi, > > > > On 01/27/2017 02:51 PM, Ville Syrjälä wrote: > >> On Mon, Jan 23, 2017 at 10:09:58PM +0100, Hans de Goede wrote: > >>> Make sure the P-Unit or the PMIC i2c bus is n

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/color: un-inline drm_color_lut_extract() (rev2)

2017-01-30 Thread Patchwork
== Series Details == Series: drm/color: un-inline drm_color_lut_extract() (rev2) URL : https://patchwork.freedesktop.org/series/18391/ State : success == Summary == Series 18391v2 drm/color: un-inline drm_color_lut_extract() https://patchwork.freedesktop.org/api/1.0/series/18391/revisions/2/mb

Re: [Intel-gfx] [PATCH i-g-t v2] tests: Validate TEST_ONLY correctness against full atomic commit

2017-01-30 Thread Maarten Lankhorst
Op 30-01-17 om 12:37 schreef Mika Kahola: > Validate atomic commit correctness before actual commit. > > For: VIZ-6956 > > v2: Add flag to toggle TEST_ONLY for atomic commit. > Remove DRM_MODE_PAGE_FLIP_EVENT flag, if enabled, before trying atomic > commit with TEST_ONLY flag (Maarten) > >

Re: [Intel-gfx] [PATCH] drm/i915: Sanity check the computed size and base of stolen memory

2017-01-30 Thread Joonas Lahtinen
On pe, 2017-01-27 at 17:20 +, Chris Wilson wrote: > Just do a quick check that the stolen memory address range doesn't > overflow our chosen integer type. > > Signed-off-by: Chris Wilson > @@ -189,7 +189,7 @@ static dma_addr_t i915_stolen_to_dma(struct > drm_i915_private *dev_priv) >  

Re: [Intel-gfx] [PATCH i-g-t 2/2] aubdump: support new ioctl

2017-01-30 Thread Chris Wilson
On Mon, Jan 30, 2017 at 02:29:07PM +0200, Petri Latvala wrote: > On Sun, Jan 29, 2017 at 11:35:42PM +, Lionel Landwerlin wrote: > > Otherwise we won't log anything with OpenGL applications using a > > recent Mesa driver. > > > > Signed-off-by: Lionel Landwerlin > > --- > > tools/aubdump.c |

Re: [Intel-gfx] [PATCH v2 3/3] drm/i915: Add early BXT sdv to the list of preproduction machines

2017-01-30 Thread Joonas Lahtinen
On ma, 2017-01-30 at 10:44 +, Chris Wilson wrote: > Extend intel_detect_preproduction_hw() to include BXT A and B steppings. > > Signed-off-by: Chris Wilson > Cc: "Zanoni, Paulo R" > Cc: Rodrigo Vivi > Cc: Jani Nikula > Cc: Ville Syrjälä Reviewed-by: Joonas Lahtinen Regards, Joonas --

Re: [Intel-gfx] [PATCH v2 1/3] drm/i915: Make intel_detect_preproduction_hw easier to extend

2017-01-30 Thread Joonas Lahtinen
On ma, 2017-01-30 at 10:44 +, Chris Wilson wrote: > As we add new generations, we should keep detecting new pre-production > system development platforms that were temporarily enabled to facilitate > initial development and now superseded by production systems. To make > it easier to add more p

Re: [Intel-gfx] [PATCH v2 2/3] drm/i915: Mark the kernel as tainted if we fail the preproduction check

2017-01-30 Thread Joonas Lahtinen
On ma, 2017-01-30 at 10:44 +, Chris Wilson wrote: > Preproduction sdv are not supported beyond the release of production > hardware, and continued use is ill-advised. Mark the kernel as tainted > to reinforce the error. > > Signed-off-by: Chris Wilson > Cc: "Zanoni, Paulo R" > Cc: Rodrigo Vi

Re: [Intel-gfx] [PATCH i-g-t 2/2] aubdump: support new ioctl

2017-01-30 Thread Petri Latvala
On Sun, Jan 29, 2017 at 11:35:42PM +, Lionel Landwerlin wrote: > Otherwise we won't log anything with OpenGL applications using a > recent Mesa driver. > > Signed-off-by: Lionel Landwerlin > --- > tools/aubdump.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/too

Re: [Intel-gfx] [PATCH] drm/i915: Check fence status after serialising with the spinlock

2017-01-30 Thread Joonas Lahtinen
On pe, 2017-01-27 at 13:06 +, Chris Wilson wrote: > A very rare problem we encounter is that as retire the request we find                                          we -^ > that it appears to not have been yet executed. Other than malevolent hw, > one possible explanation is that the update of t

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/skl: Add missing SKL ID (rev2)

2017-01-30 Thread Patchwork
== Series Details == Series: drm/i915/skl: Add missing SKL ID (rev2) URL : https://patchwork.freedesktop.org/series/3537/ State : success == Summary == Series 3537v2 drm/i915/skl: Add missing SKL ID https://patchwork.freedesktop.org/api/1.0/series/3537/revisions/2/mbox/ Test kms_pipe_crc_basi

Re: [Intel-gfx] Oops at shutdown in intel_unpin_fb_obj()

2017-01-30 Thread Maarten Lankhorst
Op 30-01-17 om 10:38 schreef Daniel Vetter: > On Sun, Jan 29, 2017 at 11:42:32AM -0800, Linus Torvalds wrote: >> Guys, I've gotten absolutely no response to this, and the problem >> seems to still occur. >> >> I just got a slightly different hang at shutdown, due to a kernel oops >> that seems rela

Re: [Intel-gfx] [PATCH i-g-t] lib/drmtest: make DRIVER_ANY match any driver

2017-01-30 Thread Eric Engestrom
On Monday, 2017-01-30 11:50:52 +, Daniel Stone wrote: > Hi, > > On 30 January 2017 at 11:46, Petri Latvala wrote: > > NAK. > > > > DRIVER_VGEM is omitted from DRIVER_ANY intentionally. Vgem is unable > > to modeset, unable to render, practically it only supports the > > vgem-specific tests. S

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: minor corner case fix to respect user's backlight setting

2017-01-30 Thread Patchwork
== Series Details == Series: drm/i915: minor corner case fix to respect user's backlight setting URL : https://patchwork.freedesktop.org/series/18685/ State : success == Summary == Series 18685v1 drm/i915: minor corner case fix to respect user's backlight setting https://patchwork.freedesktop

Re: [Intel-gfx] [PATCH i-g-t] lib/drmtest: make DRIVER_ANY match any driver

2017-01-30 Thread Daniel Stone
Hi, On 30 January 2017 at 11:46, Petri Latvala wrote: > NAK. > > DRIVER_VGEM is omitted from DRIVER_ANY intentionally. Vgem is unable > to modeset, unable to render, practically it only supports the > vgem-specific tests. See also: lib/drmtest.c, __drm_open_driver(). Yeah, I agree with this. It'

Re: [Intel-gfx] [PATCH i-g-t] lib/drmtest: make DRIVER_ANY match any driver

2017-01-30 Thread Petri Latvala
NAK. DRIVER_VGEM is omitted from DRIVER_ANY intentionally. Vgem is unable to modeset, unable to render, practically it only supports the vgem-specific tests. See also: lib/drmtest.c, __drm_open_driver(). -- Petri Latvala ___ Intel-gfx mailing list Int

[Intel-gfx] [PATCH i-g-t v2] tests: Validate TEST_ONLY correctness against full atomic commit

2017-01-30 Thread Mika Kahola
Validate atomic commit correctness before actual commit. For: VIZ-6956 v2: Add flag to toggle TEST_ONLY for atomic commit. Remove DRM_MODE_PAGE_FLIP_EVENT flag, if enabled, before trying atomic commit with TEST_ONLY flag (Maarten) Cc: Maarten Lankhorst Signed-off-by: Mika Kahola ---

Re: [Intel-gfx] [PATCH v3] drm/edid: Complete CEA modedb(VIC 1-107)

2017-01-30 Thread Sharma, Shashank
I just realized that the CEA spec talks about picking HDMI 2.0/861-F VIC over HDMI14b VIC in case of 3D side-by-side format (not in 2D), In this case, we might have to add a check while loading the VIC filed in AVI IF (as you suggested) Will come back with a V2 (if required), once I have more

[Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Check fence status after serialising with the spinlock

2017-01-30 Thread Patchwork
== Series Details == Series: drm/i915: Check fence status after serialising with the spinlock URL : https://patchwork.freedesktop.org/series/18678/ State : warning == Summary == Series 18678v1 drm/i915: Check fence status after serialising with the spinlock https://patchwork.freedesktop.org/ap

[Intel-gfx] [PATCH] drm/i915: Warn when cleanup the plane if the fb doesn't match the vma

2017-01-30 Thread Chris Wilson
We track the plane state's framebuffer and VMA separately, giving us an opportunity to detect if that pair becomes inconsistent when we cleanup after the atomic modeset. Signed-off-by: Chris Wilson Cc: Maarten Lankhorst Cc: Daniel Vetter --- drivers/gpu/drm/i915/intel_display.c | 2 ++ 1 file

[Intel-gfx] [PATCH i-g-t] tests: Clean up shell scripts

2017-01-30 Thread Joonas Lahtinen
Convert all scripts to use /bin/sh shebang and fix all shellcheck reported problems. Signed-off-by: Joonas Lahtinen Cc: Petri Latvala Cc: Daniel Vetter Cc: Jani Nikula --- tests/check_drm_clients | 15 + tests/ddx_intel_after_fbdev | 42 ++--- t

Re: [Intel-gfx] [PATCH v3] drm/edid: Complete CEA modedb(VIC 1-107)

2017-01-30 Thread Sharma, Shashank
Regards Shashank On 1/30/2017 3:47 PM, Andrzej Hajda wrote: On 30.01.2017 10:42, Sharma, Shashank wrote: Regards Shashank On 1/30/2017 2:52 PM, Andrzej Hajda wrote: On 30.01.2017 09:45, Sharma, Shashank wrote: Regards Shashank On 1/30/2017 1:30 PM, Andrzej Hajda wrote: On 29.01.2017

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Create context desc template when context is created

2017-01-30 Thread Patchwork
== Series Details == Series: drm/i915: Create context desc template when context is created URL : https://patchwork.freedesktop.org/series/18677/ State : success == Summary == Series 18677v1 drm/i915: Create context desc template when context is created https://patchwork.freedesktop.org/api/1.

[Intel-gfx] [PATCH v2 3/3] drm/i915: Add early BXT sdv to the list of preproduction machines

2017-01-30 Thread Chris Wilson
Extend intel_detect_preproduction_hw() to include BXT A and B steppings. Signed-off-by: Chris Wilson Cc: "Zanoni, Paulo R" Cc: Rodrigo Vivi Cc: Jani Nikula Cc: Ville Syrjälä --- drivers/gpu/drm/i915/i915_drv.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/i915_drv.

[Intel-gfx] [PATCH v2 2/3] drm/i915: Mark the kernel as tainted if we fail the preproduction check

2017-01-30 Thread Chris Wilson
Preproduction sdv are not supported beyond the release of production hardware, and continued use is ill-advised. Mark the kernel as tainted to reinforce the error. Signed-off-by: Chris Wilson Cc: "Zanoni, Paulo R" Cc: Rodrigo Vivi Cc: Jani Nikula Cc: Ville Syrjälä Acked-by: Jani Nikula ---

[Intel-gfx] [PATCH v2 1/3] drm/i915: Make intel_detect_preproduction_hw easier to extend

2017-01-30 Thread Chris Wilson
As we add new generations, we should keep detecting new pre-production system development platforms that were temporarily enabled to facilitate initial development and now superseded by production systems. To make it easier to add more platforms, split the if into a series of logical operations. v

Re: [Intel-gfx] [PATCH v3] drm/edid: Complete CEA modedb(VIC 1-107)

2017-01-30 Thread Andrzej Hajda
On 30.01.2017 10:42, Sharma, Shashank wrote: > Regards > > Shashank > > > On 1/30/2017 2:52 PM, Andrzej Hajda wrote: >> On 30.01.2017 09:45, Sharma, Shashank wrote: >>> Regards >>> >>> Shashank >>> >>> >>> On 1/30/2017 1:30 PM, Andrzej Hajda wrote: On 29.01.2017 06:41, Shashank Sharma wrote: >

Re: [Intel-gfx] [PATCH v2 3/3] acpi: lpss: call pwm_add_table() for bsw PWM device

2017-01-30 Thread Thierry Reding
On Mon, Jan 30, 2017 at 11:33:10AM +0200, Mika Westerberg wrote: > On Mon, Jan 30, 2017 at 09:28:09AM +0100, Rafael J. Wysocki wrote: > > On Mon, Jan 30, 2017 at 9:18 AM, Thierry Reding > > wrote: > > > On Sun, Jan 22, 2017 at 05:14:09PM +0100, Hans de Goede wrote: > > >> On x86 we do not have dev

Re: [Intel-gfx] [PATCH 1/4] lib/scatterlist: Fix offset type in sg_alloc_table_from_pages

2017-01-30 Thread Daniel Vetter
Hi all, Ok if we merge the entire series through drm-intel (likely for 4.12, 4.11 is getting a bit late)? We'd like to use this there, and Mauro already reviewed the v4l side ... Thanks, Daniel On Mon, Jan 16, 2017 at 02:12:07PM +, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Scatterli

Re: [Intel-gfx] [PATCH v3] drm/edid: Complete CEA modedb(VIC 1-107)

2017-01-30 Thread Sharma, Shashank
Regards Shashank On 1/30/2017 2:52 PM, Andrzej Hajda wrote: On 30.01.2017 09:45, Sharma, Shashank wrote: Regards Shashank On 1/30/2017 1:30 PM, Andrzej Hajda wrote: On 29.01.2017 06:41, Shashank Sharma wrote: CEA-861-F specs defines new 4k video modes to be used with HDMI 2.0 EDIDs. Thes

Re: [Intel-gfx] Oops at shutdown in intel_unpin_fb_obj()

2017-01-30 Thread Daniel Vetter
On Sun, Jan 29, 2017 at 11:42:32AM -0800, Linus Torvalds wrote: > Guys, I've gotten absolutely no response to this, and the problem > seems to still occur. > > I just got a slightly different hang at shutdown, due to a kernel oops > that seems related. It's not identical - the call trace is very >

Re: [Intel-gfx] [dinq] Request to merge in padovan's fence fd fix

2017-01-30 Thread Daniel Vetter
On Fri, Jan 27, 2017 at 12:36:11PM -0800, Chad Versace wrote: > Chris just pushed execbuffer fence fds to dinq. Could someone please get the > below fix from airlied/drm-fixes into dinq? (What's the usual method here? > Merge drm-fixes -> dinq, cherry-pick drm-fixes -> dinq, rebase?) For testing,

Re: [Intel-gfx] [PATCH v2 3/3] acpi: lpss: call pwm_add_table() for bsw PWM device

2017-01-30 Thread Mika Westerberg
On Mon, Jan 30, 2017 at 09:28:09AM +0100, Rafael J. Wysocki wrote: > On Mon, Jan 30, 2017 at 9:18 AM, Thierry Reding > wrote: > > On Sun, Jan 22, 2017 at 05:14:09PM +0100, Hans de Goede wrote: > >> On x86 we do not have devicetree to link the PWM controller and > >> the display controller together

Re: [Intel-gfx] [PATCH] drm/i915: Get correct display clock on 945gm

2017-01-30 Thread Daniel Vetter
On Fri, Jan 27, 2017 at 09:51:50PM +0200, Ville Syrjälä wrote: > On Fri, Jan 27, 2017 at 06:24:25PM +0100, Arthur Heymans wrote: > > Ville Syrjälä writes: > > > > >> > > >> > Now if someone could figure out where to dig up the DDR and FSB clocks > > >> > we could also fix up the 190 vs. 200 MHz

Re: [Intel-gfx] [PATCH] drm: Don't race connector registration

2017-01-30 Thread Daniel Vetter
On Thu, Jan 26, 2017 at 12:34:29PM -0800, Dave Hansen wrote: > On 01/25/2017 07:38 AM, Daniel Vetter wrote: > > On Wed, Jan 25, 2017 at 07:20:45AM -0800, Dave Hansen wrote: > >> On 01/24/2017 10:21 PM, Daniel Vetter wrote: > >>> Hi Dave, > >>> > >>> Still waiting for your testing results on this on

[Intel-gfx] [PULL] drm-misc-next

2017-01-30 Thread Daniel Vetter
Hi Dave, Another round of -misc stuff: - Noralf debugfs cleanup cleanup (not yet everything, some more driver patches awaiting acks). - More doc work. - edid/infoframe fixes from Ville. - misc 1-patch fixes all over, as usual Noralf needs this for his tinydrm pull request. And as discussed las

Re: [Intel-gfx] [PATCH v3] drm/i915/glk: Program pipe gamma and degamma tables

2017-01-30 Thread Ander Conselvan De Oliveira
On Fri, 2017-01-27 at 16:04 +0200, Ville Syrjälä wrote: > On Fri, Jan 27, 2017 at 11:02:30AM +0200, Ander Conselvan de Oliveira wrote: > > The gamma tables in Geminilake were changed. There is no split-gamma > > mode. Instead, there is a dedicated degamma table that is enabled > > whenever pipe CSC

Re: [Intel-gfx] [PATCH v2 2/5] drm/i915/glk: Plane color correction register changes

2017-01-30 Thread Ander Conselvan De Oliveira
On Thu, 2017-01-26 at 13:32 +0200, Ville Syrjälä wrote: > On Thu, Jan 26, 2017 at 01:24:22PM +0200, Ander Conselvan de Oliveira wrote: > > In Geminilake, the bits for enabling pipe csc, pipe gamma and plane > > gamma moved to a new register. So update the plane update functions > > to set the right

Re: [Intel-gfx] [PATCH v3] drm/i915/glk: Turn on workarounds that apply to Geminilake too

2017-01-30 Thread Ander Conselvan De Oliveira
On Thu, 2017-01-26 at 14:06 +0200, David Weinehall wrote: > On Thu, Jan 26, 2017 at 11:16:58AM +0200, Ander Conselvan de Oliveira wrote: > > Apply workarounds to Geminilake, and annotate those that are applied > > unconditionally when they apply to GLK based on the workaround database. > > > > v2:

Re: [Intel-gfx] [PATCH v2 3/3] acpi: lpss: call pwm_add_table() for bsw PWM device

2017-01-30 Thread Rafael J. Wysocki
On Mon, Jan 30, 2017 at 9:18 AM, Thierry Reding wrote: > On Sun, Jan 22, 2017 at 05:14:09PM +0100, Hans de Goede wrote: >> On x86 we do not have devicetree to link the PWM controller and >> the display controller together. So someone needs to call >> pwm_add_table() to create the link, so that the

Re: [Intel-gfx] [PATCH v2 3/3] acpi: lpss: call pwm_add_table() for bsw PWM device

2017-01-30 Thread Thierry Reding
On Sun, Jan 22, 2017 at 05:14:09PM +0100, Hans de Goede wrote: > On x86 we do not have devicetree to link the PWM controller and > the display controller together. So someone needs to call > pwm_add_table() to create the link, so that the i915 driver's > pwm_get(dev, "pwm_backlight") call returns t

Re: [Intel-gfx] [PATCH 0/3] drm/i915: Handle hanging during nonblocking modeset correctly.

2017-01-30 Thread Daniel Vetter
On Fri, Jan 27, 2017 at 03:08:45PM +, Chris Wilson wrote: > On Fri, Jan 27, 2017 at 03:58:08PM +0100, Daniel Vetter wrote: > > On Fri, Jan 27, 2017 at 02:31:55PM +, Chris Wilson wrote: > > > On Fri, Jan 27, 2017 at 03:21:29PM +0100, Daniel Vetter wrote: > > > > On Fri, Jan 27, 2017 at 09:30

Re: [Intel-gfx] [PATCH v2 2/3] pwm/core: Try to get the module from pwm_get

2017-01-30 Thread Thierry Reding
On Sun, Jan 22, 2017 at 05:14:08PM +0100, Hans de Goede wrote: > Add a module_name string to the pwm_lookup struct and if specified > and pwmchip_find_by_name() does not find the pwmchip try calling > request_module with the specified name. > > Signed-off-by: Hans de Goede > --- > drivers/pwm/co

Re: [Intel-gfx] [PATCH] tools: rename intel_bios_reader to intel_vbt_decode

2017-01-30 Thread Jani Nikula
On Tue, 24 Jan 2017, Jani Nikula wrote: > After all these years intel_bios_reader and intel_bios_dumper still > manage to confuse me. Read or dump, which one decodes. Rename > intel_bios_reader to intel_vbt_decode to be in line with the naming of > all the other tools (particularly the closely rel

<    1   2