Re: [Intel-gfx] [PATCH 11/13] drm/vmwgfx: Drop drm_vblank_cleanup

2017-06-26 Thread Thomas Hellstrom
With the below fixed, Reviewed-by: Thomas Hellstrom On 06/23/2017 04:10 PM, Sean Paul wrote: On Wed, Jun 21, 2017 at 10:28:48AM +0200, Daniel Vetter wrote: Again stopping the vblank before uninstalling the irq handler is kinda the wrong way round, but the fb_off stuff

[Intel-gfx] [PATCH i-g-t 1/7] tests/chamelium: Update connector state without reprobe when possible

2017-06-26 Thread Paul Kocialkowski
This renames the reprobe_connector function to update_connector and ensures that full reprobe of the connector is only done when really necessary (e.g. when changing the EDID). A full reprobe takes time and is not required for updating the connector state. Thus, this allows executing tests

Re: [Intel-gfx] [PATCH i-g-t 4/7] lib/igt_chamelium: Add support for configurable DUT suspend/resume delay

2017-06-26 Thread Paul Kocialkowski
On Mon, 2017-06-26 at 17:25 +0300, Martin Peres wrote: > On 26/06/17 16:59, Paul Kocialkowski wrote: > > This adds support for reading a SuspendResumeDelay property (under > > [DUT]) in the IGT configuration (igtrc) and exposing it through a > > chamelium_get_suspend_resume_delay function. > > >

[Intel-gfx] [PATCH] drm/i915: Synchronize connectors states when switching from poll to irq

2017-06-26 Thread Paul Kocialkowski
After detecting an IRQ storm, hotplug detection will switch from irq-based detection to poll-based detection. After a short delay or when resetting storm detection from debugfs, detection will switch back to being irq-based. However, it may occur that polling does not have enough time to detect

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Synchronize connectors states when switching from poll to irq

2017-06-26 Thread Patchwork
== Series Details == Series: drm/i915: Synchronize connectors states when switching from poll to irq URL : https://patchwork.freedesktop.org/series/26361/ State : success == Summary == Series 26361v1 drm/i915: Synchronize connectors states when switching from poll to irq

[Intel-gfx] [Regression report] Weekly regression report WW25

2017-06-26 Thread elizabethx . de . la . torre . mena
Link to FDO regression list:

Re: [Intel-gfx] [PATCH v12 3/3] drm/i915: Add option to support dynamic backlight via DPCD

2017-06-26 Thread David Weinehall
On Thu, Jun 22, 2017 at 12:03:39PM -0700, Puthikorn Voravootivat wrote: > This patch adds option to enable dynamic backlight for eDP > panel that supports this feature via DPCD register and > set minimum / maximum brightness to 0% and 100% of the > normal brightness. This patch causes a

[Intel-gfx] [PATCH i-g-t 5/7] tests/chamelium: Use configurable DUT suspend/resume delay

2017-06-26 Thread Paul Kocialkowski
Now that the suspend/resume delay can be configured in the IGT configuration (igtrc), use it in the chamelium suspend/resume tests. Signed-off-by: Paul Kocialkowski --- tests/chamelium.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-)

[Intel-gfx] [PATCH i-g-t 6/7] tests/chamelium: Reduce the simple hotplug test toggle count for VGA

2017-06-26 Thread Paul Kocialkowski
Since VGA hpd detection is done through RGB lines load detection and nowadays often goes through a bridge to some digital interface, HPD detection usually takes a while (up to a couple seconds). Thus, it is not as relevant to stress it as many as 15 times. This brings the toggle count down to 5

[Intel-gfx] [PATCH i-g-t 4/7] lib/igt_chamelium: Add support for configurable DUT suspend/resume delay

2017-06-26 Thread Paul Kocialkowski
This adds support for reading a SuspendResumeDelay property (under [DUT]) in the IGT configuration (igtrc) and exposing it through a chamelium_get_suspend_resume_delay function. Signed-off-by: Paul Kocialkowski --- lib/igt_chamelium.c | 31

[Intel-gfx] [PATCH i-g-t 3/7] tests/chamelium: Use 50 ms delay to wait for connector change

2017-06-26 Thread Paul Kocialkowski
Using a 1 s delay seems too large for detecting a connector change, that may happen faster than this. Since the constraints on execution time are high, it is preferable to use a much smaller sleep time. Signed-off-by: Paul Kocialkowski --- tests/chamelium.c |

[Intel-gfx] [PATCH i-g-t 2/7] tests/chamelium: Check all connectors state for basic hotplug

2017-06-26 Thread Paul Kocialkowski
Without doing a full reprobe, hpd toggles are sent without much delay between them. With a VGA connector attached, the reset occurring before the test will toggle its state, with a delay (inherent to hpd detection on VGA). It often occurs that this VGA state toggle is detected in the middle of

[Intel-gfx] [PATCH i-g-t 7/7] tests/chamelium: Disconnect connectors without extra reset

2017-06-26 Thread Paul Kocialkowski
This removes the reset call from the reset_state function and renames it to disconnect_connector. Since a call to reset is already done in chamelium_init, there is no need to do an extra reset in each test. This allows reducing the execution time a bit. Signed-off-by: Paul Kocialkowski

Re: [Intel-gfx] [PATCH i-g-t 4/7] lib/igt_chamelium: Add support for configurable DUT suspend/resume delay

2017-06-26 Thread Martin Peres
On 26/06/17 16:59, Paul Kocialkowski wrote: This adds support for reading a SuspendResumeDelay property (under [DUT]) in the IGT configuration (igtrc) and exposing it through a chamelium_get_suspend_resume_delay function. Signed-off-by: Paul Kocialkowski ---

[Intel-gfx] [Bug Report] Weekly progress report WW25

2017-06-26 Thread De La Torre Mena, ElizabethX
Highlights: - 15 bugs open during the week - 11 bugs closed during the week - 350 total bugs remain open - 27 bugs labeled ReadyForDev during the week - 111 total bugs labeled ReadyForDev remain open Details: - Bugs open during the week

Re: [Intel-gfx] [PATCH i-g-t 4/7] lib/igt_chamelium: Add support for configurable DUT suspend/resume delay

2017-06-26 Thread Martin Peres
On 26/06/17 17:33, Paul Kocialkowski wrote: On Mon, 2017-06-26 at 17:25 +0300, Martin Peres wrote: On 26/06/17 16:59, Paul Kocialkowski wrote: This adds support for reading a SuspendResumeDelay property (under [DUT]) in the IGT configuration (igtrc) and exposing it through a

[Intel-gfx] [PATCH 1/2] drm/vmwgfx: Drop drm_vblank_cleanup

2017-06-26 Thread Daniel Vetter
Again stopping the vblank before uninstalling the irq handler is kinda the wrong way round, but the fb_off stuff should take care of disabling the dsiplay at least in most cases. So drop the drm_vblank_cleanup code since it's not really doing anything, it looks all cargo-culted. v2: Appease gcc

[Intel-gfx] [RFC v4 1/2] drm/i915: Engine discovery uAPI

2017-06-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Engine discovery uAPI allows userspace to probe for engine configuration and features without needing to maintain the internal PCI id based database. This enables removal of code duplications across userspace components. Probing is done via the

[Intel-gfx] [RFC v6 2/2] drm/i915: Select engines via class and instance in execbuffer2

2017-06-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Building on top of the previous patch which exported the concept of engine classes and instances, we can also use this instead of the current awkward engine selection uAPI. This is primarily interesting for the VCS engine selection which is a)

Re: [Intel-gfx] [RFC i-g-t 0/4] Redundant test pruning

2017-06-26 Thread Daniel Vetter
On Fri, Jun 23, 2017 at 12:31:39PM +0100, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Small series which saves test execution time by removing the redundant tests. > > Tvrtko Ursulin (4): > igt: Remove default from the engine list > gem_exec_basic: Exercise

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915/dp: Remove -1/+1 from t11_t12 for Gen9_LP/CNP case

2017-06-26 Thread Ville Syrjälä
On Thu, Jun 22, 2017 at 12:25:24PM -0700, Manasi Navare wrote: > Now the VBT.seq->t11_t12 value adds 100ms to both Gen9_LP > as well as non Gen9_LP cases so no need to special case > and do -1 during HW readout and +1 during pp_div write > for Gen9_LP/CNP case. > > Signed-off-by: Manasi Navare

Re: [Intel-gfx] [RFC i-g-t 1/2] extended.testlist: Remove default and render engine test duplicates

2017-06-26 Thread Daniel Vetter
On Fri, Jun 23, 2017 at 12:21:48PM +0100, Chris Wilson wrote: > Quoting Daniel Vetter (2017-06-23 12:14:49) > > On Thu, Jun 22, 2017 at 12:45 PM, Tvrtko Ursulin > > wrote: > > > > > > On 22/06/2017 10:54, Daniel Vetter wrote: > > >> > > >> On Fri, Jun 16, 2017 at

[Intel-gfx] ✓ Fi.CI.BAT: success for New engine discovery and execbuffer2 engine selection uAPI (rev8)

2017-06-26 Thread Patchwork
== Series Details == Series: New engine discovery and execbuffer2 engine selection uAPI (rev8) URL : https://patchwork.freedesktop.org/series/23189/ State : success == Summary == Series 23189v8 New engine discovery and execbuffer2 engine selection uAPI

Re: [Intel-gfx] [PATCH] drm/i915: Prevent kernel panic when reading/writing compliance debugfs files, v2.

2017-06-26 Thread Pandiyan, Dhinakaran
On Mon, 2017-06-26 at 10:18 +0200, Maarten Lankhorst wrote: > When reading all debugfs files on a system with DP-MST the kernel panics > on a null pointer dereference because intel_dp is null for a DP-MST > connector. Detect this case and skip those connectors. > > Also fix the write for the

[Intel-gfx] [RFC i-g-t 1/2] igt/gem_engine_info: Test the new uAPI

2017-06-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Test that the engine info ioctl works. Signed-off-by: Tvrtko Ursulin --- tests/Makefile.sources | 1 + tests/gem_engine_info.c | 317 2 files changed, 318 insertions(+)

[Intel-gfx] [RFC i-g-t 2/2] igt/gem_exec_class_instance: Test the new uAPI

2017-06-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Submit a few valid and invalid no-op batches using the new class- instance execbuf interface. Signed-off-by: Tvrtko Ursulin --- tests/Makefile.sources | 1 + tests/gem_exec_class_instance.c | 177

[Intel-gfx] [PATCH 2/2] drm/vblank: Unexport drm_vblank_cleanup

2017-06-26 Thread Daniel Vetter
There's no reason for drivers to call this, and all the ones I've removed looked very fishy: - Proper quiescenting of the vblank machinery should be done by calling drm_crtc_vblank_off(), which is best done by shutting down the entire display engine with drm_atomic_helper_shutdown. -

Re: [Intel-gfx] [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-26 Thread Daniel Vetter
On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: > On Wed, 21 Jun 2017 17:23:57 +0200, > Chris Wilson wrote: > > > > Quoting Daniel Vetter (2017-06-21 16:08:54) > > > So back when the i915 power well support landed in > > > > > > commit 99a2008d0b32d72dfc2a54e7be1eb698dd2e3bd6 > > >

Re: [Intel-gfx] [PATCH v9 5/7] vfio: Define vfio based dma-buf operations

2017-06-26 Thread Alex Williamson
On Mon, 26 Jun 2017 08:39:17 +0200 Gerd Hoffmann wrote: > Hi, > > > > With the generation we can also do something different:  Pass in > > > plane_type and > > > generation, and have VFIO_DEVICE_GET_DMABUF_FD return an error in > > > case > > > the generation doesn't match. 

Re: [Intel-gfx] [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-26 Thread Takashi Iwai
On Mon, 26 Jun 2017 18:16:30 +0200, Daniel Vetter wrote: > > On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: > > On Wed, 21 Jun 2017 17:23:57 +0200, > > Chris Wilson wrote: > > > > > > Quoting Daniel Vetter (2017-06-21 16:08:54) > > > > So back when the i915 power well support

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [RESEND,1/2] drm/i915/dp: Fix the t11_t12 panel power cycle delay from VBT read

2017-06-26 Thread Patchwork
== Series Details == Series: series starting with [RESEND,1/2] drm/i915/dp: Fix the t11_t12 panel power cycle delay from VBT read URL : https://patchwork.freedesktop.org/series/26388/ State : success == Summary == Series 26388v1 Series without cover letter

Re: [Intel-gfx] [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-26 Thread Daniel Vetter
On Mon, Jun 26, 2017 at 7:47 PM, Takashi Iwai wrote: > On Mon, 26 Jun 2017 18:16:30 +0200, > Daniel Vetter wrote: >> >> On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: >> > On Wed, 21 Jun 2017 17:23:57 +0200, >> > Chris Wilson wrote: >> > > >> > > Quoting Daniel

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/vmwgfx: Drop drm_vblank_cleanup

2017-06-26 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/vmwgfx: Drop drm_vblank_cleanup URL : https://patchwork.freedesktop.org/series/26379/ State : success == Summary == Series 26379v1 Series without cover letter https://patchwork.freedesktop.org/api/1.0/series/26379/revisions/1/mbox/

Re: [Intel-gfx] [PATCH] drm/i915: Make DP-MST connector info work

2017-06-26 Thread Pandiyan, Dhinakaran
Patch looks correct, Reviewed-by: Dhinakaran Pandiyan On Mon, 2017-06-26 at 10:33 +0200, Maarten Lankhorst wrote: > Commit 9a148a96fc3a ("drm/i915/debugfs: add dp mst info") adds support > for DP-MST to intel_connector_info, but forgot to remove the early >

[Intel-gfx] [RESEND 2/2] drm/i915/dp: Remove -1/+1 from t11_t12 for Gen9_LP/CNP case

2017-06-26 Thread Manasi Navare
Now the VBT.seq->t11_t12 value adds 100ms to both Gen9_LP as well as non Gen9_LP cases so no need to special case and do -1 during HW readout and +1 during pp_div write for Gen9_LP/CNP case. Reviewed-by: Ville Syrjala Signed-off-by: Manasi Navare

[Intel-gfx] [RESEND 1/2] drm/i915/dp: Fix the t11_t12 panel power cycle delay from VBT read

2017-06-26 Thread Manasi Navare
When we read the VBT t11_t12 value for panel power cycle delay, it is a zero based value so we need to 100ms to that. And then it needs to be multiplied by 10 to store it in 100usecs unit same as SW VBT. v3: * Add it as part of series v2: * Change the VBT value instead of HW readout and pp div

Re: [Intel-gfx] [PATCH i-g-t 4/7] lib/igt_chamelium: Add support for configurable DUT suspend/resume delay

2017-06-26 Thread Lyude Paul
> > > > Fair enough, but at this point, only lib/igt_chamelium.c is using > > igtrc and all > > the parsing is implemented there. > > > > So one course of action here would be to move igtrc parsing outside > > of > > igt_chamelium (maybe to igt_core?) and use some global variable for > > storing

Re: [Intel-gfx] [PATCH i-g-t 2/7] tests/chamelium: Check all connectors state for basic hotplug

2017-06-26 Thread Lyude Paul
rb'd and pushed, thanks! On Mon, 2017-06-26 at 16:59 +0300, Paul Kocialkowski wrote: > Without doing a full reprobe, hpd toggles are sent without much delay > between them. With a VGA connector attached, the reset occurring > before > the test will toggle its state, with a delay (inherent to hpd

Re: [Intel-gfx] [PATCH v2] drm/core: Fail atomic IOCTL with no CRTC state but with signaling.

2017-06-26 Thread Harry Wentland
On 2017-06-20 01:57 PM, Andrey Grodzovsky wrote: > Problem : While running IGT kms_atomic_transition test suite i encountered > a hang in drmHandleEvent immediately following an atomic_commit. > After dumping the atomic state I relized that in this case there was > not even one CRTC attached to

[Intel-gfx] [PATCH] i2c: i801: Allow ACPI SystemIO OpRegion to conflict harder

2017-06-26 Thread Lyude
There's quite a number of machines on the market, mainly Lenovo ThinkPads, that make the horrible mistake in their firmware of reusing the PCIBAR space reserved for the SMBus for things that are completely unrelated to the SMBus controller, such as the OpRegion used for i915. This is very bad and

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Disable MSI for all pre-gen5

2017-06-26 Thread Patchwork
== Series Details == Series: drm/i915: Disable MSI for all pre-gen5 URL : https://patchwork.freedesktop.org/series/26392/ State : failure == Summary == Series 26392v1 drm/i915: Disable MSI for all pre-gen5 https://patchwork.freedesktop.org/api/1.0/series/26392/revisions/1/mbox/ Test

Re: [Intel-gfx] [PATCH] i2c: i801: Allow ACPI SystemIO OpRegion to conflict harder

2017-06-26 Thread Rafael J. Wysocki
On Monday, June 26, 2017 04:40:08 PM Lyude wrote: > There's quite a number of machines on the market, mainly Lenovo > ThinkPads, that make the horrible mistake in their firmware of reusing > the PCIBAR space reserved for the SMBus for things that are completely > unrelated to the SMBus controller,

Re: [Intel-gfx] [PATCH i-g-t 3/7] tests/chamelium: Use 50 ms delay to wait for connector change

2017-06-26 Thread Lyude Paul
rb'd and pushed, thanks! On Mon, 2017-06-26 at 16:59 +0300, Paul Kocialkowski wrote: > Using a 1 s delay seems too large for detecting a connector change, > that > may happen faster than this. Since the constraints on execution time > are > high, it is preferable to use a much smaller sleep time.

Re: [Intel-gfx] [PATCH v2 00/14] improve the fb_setcmap helper

2017-06-26 Thread Dave Airlie
> > I'm traveling and cannot make progress this week. The merge window is > also real close so this series will therefore probably miss it unless > something unexpected happens... Don't worry you missed the merge window for drm already, we don't merge things after -rc6. Please remember the Linus

Re: [Intel-gfx] [PATCH v2 00/14] improve the fb_setcmap helper

2017-06-26 Thread Peter Rosin
On 2017-06-26 11:35, Daniel Vetter wrote: > On Thu, Jun 22, 2017 at 08:06:23AM +0200, Peter Rosin wrote: >> Hi! >> >> While trying to get CLUT support for the atmel_hlcdc driver, and >> specifically for the emulated fbdev interface, I received some >> push-back that my feeble in-driver attempts

Re: [Intel-gfx] [PATCH i-g-t 7/7] tests/chamelium: Disconnect connectors without extra reset

2017-06-26 Thread Lyude Paul
On Mon, 2017-06-26 at 16:59 +0300, Paul Kocialkowski wrote: > This removes the reset call from the reset_state function and renames > it > to disconnect_connector. Since a call to reset is already done in > chamelium_init, there is no need to do an extra reset in each test. > > This allows

[Intel-gfx] [PATCH] drm/i915: Disable MSI for all pre-gen5

2017-06-26 Thread ville . syrjala
From: Ville Syrjälä We have pretty clear evidence that MSIs are getting lost on g4x and somehow the interrupt logic doesn't seem to recover from that state even if we try hard to clear the IIR. Disabling IER around the normal IIR clearing in the irq handler isn't

Re: [Intel-gfx] [PATCH i-g-t 1/5] lib/igt_chamelium: Add support for HPD toggle scheduling

2017-06-26 Thread Lyude Paul
This all looks great! I'm happy to see us finally getting rid of the very hacky workaround I had to make async hotplugging work (to be fair, it was only partly broken ;). Thanks for dealing with chameleond upstream too, I couldn't stand how ugly the code in that thing is... Anyway, this whole

Re: [Intel-gfx] [PATCH i-g-t 1/3] lib: Force global reset + uevents for hang detector

2017-06-26 Thread Antonio Argenziano
On 20/06/17 11:25, Michel Thierry wrote: From: Chris Wilson The hang detector relies on a uevent for notification and aborting the test. As proposed, fine-grained resets may not produce a global uevent and so this hang detection becomes void. As we don't expect any

[Intel-gfx] [PATCH] drm/i915: Prevent kernel panic when reading/writing compliance debugfs files, v2.

2017-06-26 Thread Maarten Lankhorst
When reading all debugfs files on a system with DP-MST the kernel panics on a null pointer dereference because intel_dp is null for a DP-MST connector. Detect this case and skip those connectors. Also fix the write for the DP compliance file in the same way. Changes since v1: - Fix

[Intel-gfx] [PATCH] drm/i915: Make DP-MST connector info work

2017-06-26 Thread Maarten Lankhorst
Commit 9a148a96fc3a ("drm/i915/debugfs: add dp mst info") adds support for DP-MST to intel_connector_info, but forgot to remove the early return for DP-MST. Remove it, and print out MST connectors directly. Fixes: 9a148a96fc3a ("drm/i915/debugfs: add dp mst info") Cc: #

Re: [Intel-gfx] [RFC i-g-t 1/4] igt: Remove default from the engine list

2017-06-26 Thread Szwichtenberg, Radoslaw
On Fri, 2017-06-23 at 15:35 +0100, Tvrtko Ursulin wrote: > On 23/06/2017 15:17, Szwichtenberg, Radoslaw wrote: > > On Fri, 2017-06-23 at 12:31 +0100, Tvrtko Ursulin wrote: > > > From: Tvrtko Ursulin > > > > > > Default is not an engine but an ABI alias for RCS. Remove

Re: [Intel-gfx] [PATCH v2 02/14] drm/fb-helper: remove drm_fb_helper_save_lut_atomic

2017-06-26 Thread Daniel Vetter
On Thu, Jun 22, 2017 at 08:06:25AM +0200, Peter Rosin wrote: > drm_fb_helper_save_lut_atomic is redundant since the .gamma_store is > now always kept up to date by drm_fb_helper_setcmap. > > Signed-off-by: Peter Rosin Also note that this is for kgdb support only and so likely

[Intel-gfx] [PATCH i-g-t 4/5] tests/chamelium: Test HPD toggle after suspend and hibernate for VGA

2017-06-26 Thread Paul Kocialkowski
This adds HPD toggle after suspend and hibernate testing for VGA, in the same fashion as currently done for DP and HDMI. --- tests/chamelium.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/tests/chamelium.c b/tests/chamelium.c index c7114987..07dc4849 100644

[Intel-gfx] [PATCH i-g-t 5/5] tests/chamelium: Also test VGA for common suspend and hibernate tests

2017-06-26 Thread Paul Kocialkowski
Now that HPD toggle during suspend and hibernate can be tested for VGA as well, there is no need to exclude it in the common tests. Signed-off-by: Paul Kocialkowski --- tests/chamelium.c | 9 - 1 file changed, 9 deletions(-) diff --git

[Intel-gfx] [PATCH i-g-t 2/5] tests/chamelium: Use HPD toggle scheduling instead of async pulses

2017-06-26 Thread Paul Kocialkowski
This switches over to HPD toggle scheduling instead of starting pulses with an asynchronous call, suspending and dealing with the result at resume. The XMLRPC library does not guarantee that the call will be made before caring for its outcome and this is in fact what was happening: the call was

[Intel-gfx] [PATCH i-g-t 3/5] lib/igt_chamelium: Get rid of async hpd pulse functions

2017-06-26 Thread Paul Kocialkowski
This removes the functions related to async hpd pulse handling, that were used for hpd toggle during suspend testing. Since the XMLRPC library does not guarantee that the call will be made before caring for its result, this function did not work properly. These functions are deprecated in favor

Re: [Intel-gfx] [PATCH v9 5/7] vfio: Define vfio based dma-buf operations

2017-06-26 Thread Gerd Hoffmann
Hi, > > With the generation we can also do something different:  Pass in > > plane_type and > > generation, and have VFIO_DEVICE_GET_DMABUF_FD return an error in > > case > > the generation doesn't match.  In that case it doesn't make much > > sense any > > more to have a separate plane_info

Re: [Intel-gfx] [PATCH 02/17] drm/i915: s/GEN3/GEN5/

2017-06-26 Thread Maarten Lankhorst
Op 22-06-17 om 14:40 schreef Chris Wilson: > Quoting ville.syrj...@linux.intel.com (2017-06-22 12:55:40) >> From: Ville Syrjälä >> >> The GEN5_IRQ_RESET/INIT macros are perfectly suitable even for >> gen3/4 hardware as those have 32 bit interrupt registers. Let's >>

[Intel-gfx] [PATCH i-g-t 1/5] lib/igt_chamelium: Add support for HPD toggle scheduling

2017-06-26 Thread Paul Kocialkowski
This adds support for the newly-introduced ScheduleHpdToggle XMLRPC method of the Chamelium's interface. It allows scheduling an HPD toggle, for which the call will return immediately. This is especially useful for testing HPD during suspend/resume. Signed-off-by: Paul Kocialkowski

Re: [Intel-gfx] [PATCH v9 5/7] vfio: Define vfio based dma-buf operations

2017-06-26 Thread Gerd Hoffmann
Hi, > > So maybe a "enum plane_state" (instead of "bool is_enabled")?  So > > we > > can clearly disturgish ENABLED, DISABLED, NOT_SUPPORTED cases? > > What's the difference between NOT_SUPPORTED and -ENOTTY on the ioctl? > Perhaps a bit in a flags field could specify EN/DIS-ABLED and leave >

Re: [Intel-gfx] [PATCH v2 03/14] drm/fb-helper: do a generic fb_setcmap helper in terms of crtc .gamma_set

2017-06-26 Thread Daniel Vetter
On Thu, Jun 22, 2017 at 08:06:26AM +0200, Peter Rosin wrote: > This makes the redundant fb helpers .load_lut, .gamma_set and .gamma_get > totally obsolete. > > Signed-off-by: Peter Rosin > --- > drivers/gpu/drm/drm_fb_helper.c | 154 > >

Re: [Intel-gfx] [PATCH v2 01/14] drm/fb-helper: keep the .gamma_store updated in drm_fb_helper_setcmap

2017-06-26 Thread Daniel Vetter
On Thu, Jun 22, 2017 at 08:06:24AM +0200, Peter Rosin wrote: > I think the gamma_store can end up invalid on error. But the way I read > it, that can happen in drm_mode_gamma_set_ioctl as well, so why should > this pesky legacy fbdev stuff be any better? > > Signed-off-by: Peter Rosin

Re: [Intel-gfx] [PATCH v2 00/14] improve the fb_setcmap helper

2017-06-26 Thread Daniel Vetter
On Thu, Jun 22, 2017 at 08:06:23AM +0200, Peter Rosin wrote: > Hi! > > While trying to get CLUT support for the atmel_hlcdc driver, and > specifically for the emulated fbdev interface, I received some > push-back that my feeble in-driver attempts should be solved > by the core. This is my attempt

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915: Wait for pending modesets to complete before trying link training, v2.

2017-06-26 Thread Maarten Lankhorst
Op 14-06-17 om 15:44 schreef Ville Syrjälä: > On Wed, Jun 14, 2017 at 11:08:42AM +0200, Maarten Lankhorst wrote: >> The nonblocking modeset tests were failing on systems with a DP output, >> because lijnk training could occur during the modeset. >> >> With normal modesets we're protected by the

Re: [Intel-gfx] [PATCH v2 03/14] drm/fb-helper: do a generic fb_setcmap helper in terms of crtc .gamma_set

2017-06-26 Thread Daniel Vetter
On Thu, Jun 22, 2017 at 12:22:10PM +0200, Peter Rosin wrote: > This makes the redundant fb helpers .load_lut, .gamma_set and .gamma_get > totally obsolete. > > Signed-off-by: Peter Rosin > --- > drivers/gpu/drm/drm_fb_helper.c | 151 > +--- >

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Prevent kernel panic on reading out compliance debugfs files (rev2)

2017-06-26 Thread Patchwork
== Series Details == Series: drm/i915: Prevent kernel panic on reading out compliance debugfs files (rev2) URL : https://patchwork.freedesktop.org/series/26139/ State : success == Summary == Series 26139v2 drm/i915: Prevent kernel panic on reading out compliance debugfs files

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Make DP-MST connector info work

2017-06-26 Thread Patchwork
== Series Details == Series: drm/i915: Make DP-MST connector info work URL : https://patchwork.freedesktop.org/series/26352/ State : success == Summary == Series 26352v1 drm/i915: Make DP-MST connector info work https://patchwork.freedesktop.org/api/1.0/series/26352/revisions/1/mbox/ Test

Re: [Intel-gfx] [PATCH v4 10/15] drm/i915: add compute-config for YCBCR outputs

2017-06-26 Thread Ander Conselvan De Oliveira
On Wed, 2017-06-21 at 21:19 +0530, Sharma, Shashank wrote: > Regards > > Shashank > > > On 6/20/2017 7:50 PM, Ander Conselvan De Oliveira wrote: > > On Mon, 2017-06-19 at 21:38 +0530, Shashank Sharma wrote: > > > This patch checks encoder level support for HDMI YCBCR outputs. > > > HDMI output

Re: [Intel-gfx] [PATCH 3/4] drm/i915: Add format modifiers for Intel

2017-06-26 Thread Emil Velikov
On 23 June 2017 at 17:45, Ben Widawsky wrote: > This was based on a patch originally by Kristian. It has been modified > pretty heavily to use the new callbacks from the previous patch. > > v2: > - Add LINEAR and Yf modifiers to list (Ville) > - Combine i8xx and i965 into