== Series Details ==
Series: drm/i915/guc: Define GuC firmware version for Comet Lake (rev2)
URL : https://patchwork.freedesktop.org/series/62969/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6433 -> Patchwork_13572
== Series Details ==
Series: drm/i915/selftests: Set igt_spinner.gt for early exit
URL : https://patchwork.freedesktop.org/series/63414/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6433 -> Patchwork_13573
Summary
---
On Wed, 2019-07-03 at 16:37 -0700, Matt Roper wrote:
> Convert the code that operates directly on gen11 combo PHY's to use
> the
> new namespace. Combo PHY registers are those named "ICL_PORT_*" plus
> ICL_DPHY_CHKN.
>
> Note that a lot of the PHY programming happens in the MIPI DSI code.
> For
On 7/5/19 12:46 AM, Chris Wilson wrote:
Having taken the first step in encapsulating the functionality by moving
the related files under gt/, the next step is to start encapsulating by
passing around the relevant structs rather than the global
drm_i915_private. In this step, we pass intel_gt
== Series Details ==
Series: drm/i915/userptr: Acquire the page lock around set_page_dirty()
URL : https://patchwork.freedesktop.org/series/63383/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431_full -> Patchwork_13563_full
Hi all,
On Mon, 24 Jun 2019 13:53:52 +1000 Stephen Rothwell
wrote:
>
> On Mon, 17 Jun 2019 13:20:27 +1000 Stephen Rothwell
> wrote:
> >
> > Today's linux-next merge of the drm-intel tree got a conflict in:
> >
> > drivers/gpu/drm/i915/i915_drv.h
> >
> > between commit:
> >
> >
On Tue, Jul 02, 2019 at 01:09:45PM -0700, Daniele Ceraolo Spurio wrote:
From: Chris Wilson
Preemption via GuC submission is not being supported with its current
legacy incarnation. The current FW does support a similar pre-emption
flow via H2G, but it is class-based instead of being
== Series Details ==
Series: Initial support for Tiger Lake (rev2)
URL : https://patchwork.freedesktop.org/series/62726/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
00d88902cdbe drm/i915: Add 4th pipe and transcoder
1957212e1a06 drm/i915/tgl: add initial Tiger Lake
== Series Details ==
Series: series starting with [v2,1/2] drm/i915: Clear the shared PLL from the
put_dplls() hook
URL : https://patchwork.freedesktop.org/series/63384/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6431_full -> Patchwork_13564_full
On Mon, 2019-07-08 at 16:16 -0700, Lucas De Marchi wrote:
> From: José Roberto de Souza
>
> On TGL the special EDP transcoder is gone and it should be handled by
> transcoder A.
>
> v2 (Lucas):
> - Reuse POWER_DOMAIN_TRANSCODER_EDP_VDSC (suggested by Ville)
> - Use crtc->dev since
On Mon, 2019-07-08 at 23:59 +, Souza, Jose wrote:
> On Wed, 2019-07-03 at 16:37 -0700, Matt Roper wrote:
> > Our past DDI-based Intel platforms have had a fixed DDI<->PHY
> > mapping.
> > Because of this, both the bspec documentation and our i915 code has
> > used
> > the term "port" when
Hi,
[This is an automated email]
This commit has been processed because it contains a "Fixes:" tag,
fixing commit: 5cc9ed4b9a7a drm/i915: Introduce mapping of user pages into
video memory (userptr) ioctl.
The bot has tested the following trees: v5.1.16, v4.19.57, v4.14.132, v4.9.184,
Hello Ram,
On 7/2/2019 11:24 AM, Ramalingam C wrote:
From Gen12 onwards, HDCP HW block is implemented within transcoders.
Till Gen11 HDCP HW block was part of DDI.
Hence required changes in HW programming is handled here.
v2:
_MMIO_TRANS is used [Lucas and Daniel]
platform check is
On Wed, 2019-07-03 at 16:37 -0700, Matt Roper wrote:
> Transition the remaining uses of intel_port_is_* over to the
> equivalent
> intel_phy_is_* functions and drop the port functions.
Awesome
Reviewed-by: José Roberto de Souza
>
> Cc: José Roberto de Souza
> Signed-off-by: Matt Roper
>
On Tue, Jul 02, 2019 at 01:09:46PM -0700, Daniele Ceraolo Spurio wrote:
We originally added support, in some cases partial, for different modes
of operations via guc clients:
- proxy vs direct submission;
- variable engine mask per-client.
We only ever used one flow (all submissions via a
== Series Details ==
Series: drm/i915: Explicitly track active fw_domain timers (rev3)
URL : https://patchwork.freedesktop.org/series/63331/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431_full -> Patchwork_13566_full
From: Vandita Kulkarni
Add 2 new PLLs for additional TC ports. The names for the PLLs on TGL
changed, but most registers remained the same, like MGPLL5_ENABLE,
MGPLL6_ENABLE. So continue to use the name from ICL.
Cc: Madhav Chauhan
Cc: Rodrigo Vivi
Signed-off-by: Vandita Kulkarni
According to the spec when initializing the display in TGL we should not
set PORT_CL_DW12 for the Aux channel of the combo PHYs. We will re-use the
power well hooks from ICL so just check for IS_TIGERLAKE() inside it.
Cc: Imre Deak
Signed-off-by: Lucas De Marchi
---
From: Michel Thierry
Reuse Gen11 stolen memory changes since Tiger Lake uses the same BSM
register (and format).
Cc: Rodrigo Vivi
Signed-off-by: Michel Thierry
Signed-off-by: Lucas De Marchi
---
arch/x86/kernel/early-quirks.c | 1 +
1 file changed, 1 insertion(+)
diff --git
From: Imre Deak
The patch adds the new power wells introduced by TGL (GEN 12) and
maps these to existing/new power domains. The changes for GEN 12 wrt
to GEN 11 are the following:
- Transcoder#EDP removed from power well#1 (Transcoder#A used in
low-power mode instead)
- Transcoder#A is now
Add pipe D and transcoder D to prepare for platforms having them.
Cc: Rodrigo Vivi
Signed-off-by: Lucas De Marchi
Reviewed-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_display.c | 3 ++-
drivers/gpu/drm/i915/display/intel_display.h | 4
drivers/gpu/drm/i915/i915_reg.h
From: Radhakrishna Sripada
Add the enum additions to TGP.
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Cc: David Weinehall
Cc: James Ausmus
Signed-off-by: Radhakrishna Sripada
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/i915_drv.c | 4
drivers/gpu/drm/i915/i915_drv.h | 3 +++
2
From: José Roberto de Souza
This register definition changed from ICL and has now another meaning.
Use the right bits on TGL.
Signed-off-by: José Roberto de Souza
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_dpll_mgr.c | 8 ++--
drivers/gpu/drm/i915/i915_reg.h
Current list of PCI IDs for Tiger Lake.
Cc: Rodrigo Vivi
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/i915_pci.c | 1 +
include/drm/i915_pciids.h | 10 ++
2 files changed, 11 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
From: Mahesh Kumar
Create a helper function to get ddc pin according to port number.
Cc: Anusha Srivatsa
Signed-off-by: Mahesh Kumar
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_hdmi.c | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff
From: Daniele Ceraolo Spurio
Tiger Lake is a Intel® Processor containing Intel® HD Graphics.
This is just an initial Tiger Lake definition. PCI IDs, generic support
and new features coming in following patches.
v2 (Lucas):
- Remove modular FIA - feature will be re-introduced in future
Cc:
From: José Roberto de Souza
On TGL the special EDP transcoder is gone and it should be handled by
transcoder A.
v2 (Lucas):
- Reuse POWER_DOMAIN_TRANSCODER_EDP_VDSC (suggested by Ville)
- Use crtc->dev since new_crtc_state->state may be NULL on atomic
commit (suggested by Maarten)
Cc:
From: Mahesh Kumar
This patch initializes DDI PORT A, B & C for Tiger lake. Other
TC ports need to be initialized later once corresponding code is there.
Cc: Madhav Chauhan
Signed-off-by: Mahesh Kumar
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_display.c | 9
From: Mahesh Kumar
Assume PCH_TGP when platform is TGL.
Cc: Rodrigo Vivi
Signed-off-by: Mahesh Kumar
Signed-off-by: Lucas De Marchi
Reviewed-by: Anusha Srivatsa
---
drivers/gpu/drm/i915/i915_drv.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
From: Mahesh Kumar
TGL has 3 combophy ports, so extend check for tigerlake in
intel_port_is_combophy/tc function.
Cc: Mika Kahola
Signed-off-by: Mahesh Kumar
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_display.c | 12 +---
1 file changed, 9 insertions(+), 3
Add port C to workaround to cover Tiger Lake.
Cc: Rodrigo Vivi
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_display_power.c | 11 ---
drivers/gpu/drm/i915/i915_reg.h| 4 +++-
2 files changed, 11 insertions(+), 4 deletions(-)
diff --git
From: Vandita Kulkarni
Add a new pll array for Tiger Lake. The TC pll functions for type C will
be covered in later patches after its phy is implemented.
Cc: Madhav Chauhan
Cc: Rodrigo Vivi
Signed-off-by: Vandita Kulkarni
Signed-off-by: Lucas De Marchi
---
From: Mahesh Kumar
Add VBT-value to DDC bus pin mapping for the same.
Signed-off-by: Mahesh Kumar
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_bios.c | 17 -
drivers/gpu/drm/i915/display/intel_vbt_defs.h | 3 +++
2 files changed, 19
From: Mahesh Kumar
Bit definitions for port-select got changed for TRANS_CLK_SEL &
TRANS_DDI_FUNC_CTL registers in TGL.
Signed-off-by: Mahesh Kumar
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_ddi.c | 48 +++-
drivers/gpu/drm/i915/i915_reg.h
From: Rodrigo Vivi
Previously, the recommended B credit for all platforms was 24 / number
of pipes, which would give 6 for newer platforms with 4 pipes. However 6
is not enough and we need 12 on these cases.
We also need a different BW credit for these platforms.
Cc: Arthur J Runyan
From: Mahesh Kumar
Add default GPIO pin mapping for all ports. Tiger Lake has 3 combophy
ports and 6 TC ports, gpio pin1-3 are mapped to combophy & pin9-14 are
mapped to TC ports.
Cc: Anusha Srivatsa
Cc: Rodrigo Vivi
Signed-off-by: Mahesh Kumar
Signed-off-by: Lucas De Marchi
---
From: Vandita Kulkarni
There are 2 new additional typeC ports in Tiger Lake and PORT-C is now a
combophy port. This results in 6 typeC ports and 3 combophy ports.
These 6 TC ports can be DP alternate mode, DP over thunderbolt, native
DP on legacy DP connector or native HDMI on legacy connector.
On TGL the port programming for combophy is very similar to ICL, so
adapt the callers to possibly use the different register values.
Cc: Vandita Kulkarni
Cc: Rodrigo Vivi
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_dpll_mgr.c | 24 +++
From: José Roberto de Souza
On Tiger Lake there is one more pipe - check if it's fused.
Signed-off-by: José Roberto de Souza
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_device_info.c | 3 +++
2 files changed, 4 insertions(+)
From: Mahesh Kumar
In GEN 12 PORT_C DDI clk_off bit is not equally distanced to A/B,
it's at offset 24. Similarly TC port (5/6) clk off bits are at
offset 22/23. Extend the macros to cover the additional ports.
Signed-off-by: Mahesh Kumar
Signed-off-by: Lucas De Marchi
---
v2 of https://patchwork.freedesktop.org/series/62726/
- Remove patches already reviewed
- Remove modular FIA - it's handled in a separate series now
- Add r-b on some patches
- Handle comments on power well definitions
Patches are from their original authors, modified as per review on
From: Mika Kahola
Add power well 5 to support 4th pipe and transcoder on TGL.
Cc: James Ausmus
Cc: Imre Deak
Signed-off-by: Mika Kahola
Signed-off-by: Lucas De Marchi
---
.../drm/i915/display/intel_display_power.c| 30 ---
.../drm/i915/display/intel_display_power.h|
== Series Details ==
Series: Initial support for Tiger Lake (rev2)
URL : https://patchwork.freedesktop.org/series/62726/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6433 -> Patchwork_13574
Summary
---
**SUCCESS**
On Wed, 2019-07-03 at 16:37 -0700, Matt Roper wrote:
> Our past DDI-based Intel platforms have had a fixed DDI<->PHY
> mapping.
> Because of this, both the bspec documentation and our i915 code has
> used
> the term "port" when talking about either DDI's or PHY's; it was
> always
> easy to tell
On Wed, 2019-07-03 at 18:06 -0700, Matt Roper wrote:
> Although the register name implies that it operates on DDI's,
> DPCLKA_CFGCR0_ICL actually needs to be programmed according to the
> PHY
> that's in use. I.e., when using EHL's DDI-D on combo PHY A, the bits
> described as "port A" in the
== Series Details ==
Series: drm/i915/selftests: Reorder error cleanup for whitelist checking (rev2)
URL : https://patchwork.freedesktop.org/series/63394/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431_full -> Patchwork_13567_full
== Series Details ==
Series: drm: Try to fix encoder possible_clones/crtc
URL : https://patchwork.freedesktop.org/series/63399/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431_full -> Patchwork_13568_full
Summary
Quoting Chris Wilson (2019-07-08 14:47:05)
> I'm considering how best to add lockdep to highlight that. I think if I
> add a struct mutex *lock to i915_active_request, and make it only exist
> under debug?
The problem with that is that I need to find some way to communicate
when I'm deliberately
On Fri, Jul 05, 2019 at 01:33:10PM +0300, Ville Syrjälä wrote:
On Thu, Jul 04, 2019 at 08:55:51AM -0700, Lucas De Marchi wrote:
On Thu, Jul 04, 2019 at 06:09:04PM +0300, Ville Syrjälä wrote:
>On Thu, Jul 04, 2019 at 07:54:26AM -0700, Lucas De Marchi wrote:
>> On Thu, Jul 04, 2019 at 12:18:11PM
From: Ville Syrjälä
It's not at all clear what cloning options this driver supports.
So let's just clear possible_clones instead of setting it to some
bogus value.
Cc: Philipp Zabel
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/imx/imx-drm-core.c | 2 +-
1 file changed, 1 insertion(+), 1
From: Ville Syrjälä
Don't advertize non-exisiting crtcs in the encoder possible_crtcs
bitmask.
Reviewed-by: Dhinakaran Pandiyan
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_display.c | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git
From: Ville Syrjälä
Replace the hand rolled stuff with drm_encoder_mask() when populating
possible_clones, and rename the function to
intel_encoder_possible_clones() to make it clear what it's used for.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_display.c | 13
From: Ville Syrjälä
Each fake MST encoder is tied to a specific pipe. Fix the encoder's
crtc_mask to reflect that fact.
Reviewed-by: Dhinakaran Pandiyan
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_dp_mst.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
From: Ville Syrjälä
Replace the hand rolled encoder bitmask thing with drm_encoder_mask()
Cc: Inki Dae
Cc: Joonyoung Shim
Cc: Seung-Woo Kim
Cc: Kyungmin Park
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/exynos/exynos_drm_drv.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
From: Ville Syrjälä
WARN if the encoder possible_crtcs is effectively empty or contains
bits for non-existing crtcs.
TODO: Or should we perhapst just filter out any bit for a
non-exisiting crtc?
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/drm_encoder.c | 18 ++
1 file
From: Ville Syrjälä
Many drivers are populating encoder->possible_clones wrong. Let's
persuade them to get it right by adding some loud WARNs.
We'll cross check the bits between any two encoders. So either
both encoders can clone with the other, or neither can.
We'll also complain about
From: Ville Syrjälä
Use BIT(pipe) for better legibility when populating the crtc_mask
for encoders.
Also remove the redundant possible_crtcs setup for the TV encoder.
Reviewed-by: Jani Nikula
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_crt.c | 4 ++--
From: Ville Syrjälä
We don't need to special case PCH vs. gen4 when setting up the LVDS
crtc_mask. Just claim pipes A|B|C work and intel_encoder_crtcs() drop
out any crtc that doesn't exist.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_lvds.c | 4 +---
1 file changed, 1
On 2019-07-08 at 13:16:23 +0300, Pekka Paalanen wrote:
> On Fri, 5 Jul 2019 06:16:42 +0530
> Ramalingam C wrote:
>
> > In the kernel documentation, HDCP specifications links are shared as a
> > reference for SRM table format.
> >
> > Signed-off-by: Ramalingam C
> > ---
> >
From: Ville Syrjälä
Rename the encoder->crtc_mask to encoder->pipe_mask to better
reflect what it actually contains.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/icl_dsi.c | 2 +-
drivers/gpu/drm/i915/display/intel_crt.c | 4 ++--
== Series Details ==
Series: drm: Try to fix encoder possible_clones/crtc
URL : https://patchwork.freedesktop.org/series/63399/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431 -> Patchwork_13568
Summary
---
Stop guessing over whether we have an extra wakeref held by the delayed
fw put, and track it explicitly for the sake of debug.
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
Reviewed-by: Mika Kuoppala
---
drivers/gpu/drm/i915/intel_uncore.c | 15 ---
== Series Details ==
Series: drm/i915/selftests: Reorder error cleanup for whitelist checking
URL : https://patchwork.freedesktop.org/series/63394/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431 -> Patchwork_13565
Chris Wilson writes:
> Reorder the error paths so that we unwind all the locals from any error
> path and so avoid setting off divers alarum in case we find an error in
> case we find an error.
Divers alarum got past me. But for finding an error in case
we find error: It is fitting enough that
== Series Details ==
Series: drm/i915/selftests: Reorder error cleanup for whitelist checking (rev2)
URL : https://patchwork.freedesktop.org/series/63394/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431 -> Patchwork_13567
PORT_TX_DFLEXDPMLE1 is a FIA register so move it to intel_tc.c where we
access other FIA registers. In Tiger Lake we have multiple/modular FIAs
so it makes sense to start moving all access to their registers to a
common place.
While at it, make it clear that we will only ever call this function
v2:
- Fix sparse warning
- Do not try to make header self-contained
- Fix coding style while moving code
Anusha Srivatsa (1):
drm/i915: Add modular FIA
Lucas De Marchi (3):
drm/i915: make new intel_tc.c use uncore accessors
drm/i915: fix include order in intel_tc.*
drm/i915: move
Make intel_tc.h the first include so we guarantee it's self-contained.
Sort the rest. Same principle applies for includes in the header.
v2: don't make intel_tc.h be the first include
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_tc.c | 2 +-
From: Anusha Srivatsa
Some platforms may have Modular FIA. If Modular FIA is used in the SOC,
then Display Driver will access the additional instances of
FIA based on pre-assigned offset in GTTMADDR space.
Each Modular FIA instance has its own IOSF Sideband Port ID
and it houses only 2 Type-C
Let's make the just created intel_tc.c already follow the trend of using
i915 instead of dev_priv and calling the intel_uncore_*() functions.
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/display/intel_tc.c | 57 ++---
1 file changed, 31 insertions(+), 26
On 2019-07-08 at 12:59:59 +0300, Pekka Paalanen wrote:
> On Mon, 8 Jul 2019 12:52:17 +0300
> Pekka Paalanen wrote:
>
> > On Fri, 5 Jul 2019 06:16:37 +0530
> > Ramalingam C wrote:
> >
> > > This patch adds a DRM ENUM property to the selected connectors.
> > > This property is used for
From: Ville Syrjälä
There's no point in the cast for accessing the base class. Just
take the address of the struct instead.
Cc: Benjamin Gaignard
Cc: Vincent Abriou
Acked-by: Benjamin Gaignard
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/sti/sti_tvout.c | 6 +++---
1 file changed, 3
From: Ville Syrjälä
The current possible_clones setup doesn't look sensible. I'm assuming
the 0 and 1 are supposed to refer to the indexes of the hdmi and hda
encoders? So it kinda looks like we want hda+hdmi cloning, but then
dvo also claims to be cloneable with hdmi, but hdmi won't
From: Ville Syrjälä
The docs say possible_clones should always include the encoder itself.
Since most drivers don't want to deal with the complexities of cloning
let's allow them to set possible_clones=0 and instead we'll fix that
up in the core.
We can't put this special case into
From: Ville Syrjälä
Resurrecting my possible_clones/crtcs cleanup from ~1 year ago.
Some prep work from the previous posting did get merged in the
meantime.
I have a feeling the WARNs from the new validation code may end up
triggering on some drivers, but I don't imagine this stuff would
get
From: Ville Syrjälä
I doubt the DP+DP and SDVO+SDVO cloning works for this driver.
i915 at least doesn't do those. Truthfully there could be some very
specific circumstances where some of them would do doable, but
genereally it's too much pain to deal with so we've chose not to
bother. Let's use
On 2019-07-08 at 13:09:14 +0300, Pekka Paalanen wrote:
> On Fri, 5 Jul 2019 06:16:40 +0530
> Ramalingam C wrote:
>
> > drm function is defined and exported to update a connector's
> > content protection property state and to generate a uevent along
> > with it.
> >
> > Need ACK for the uevent
== Series Details ==
Series: drm/i915/selftests: Fill in a little more of the dummy fence
URL : https://patchwork.freedesktop.org/series/63365/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6429_full -> Patchwork_13560_full
== Series Details ==
Series: drm/i915: Explicitly track active fw_domain timers (rev3)
URL : https://patchwork.freedesktop.org/series/63331/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431 -> Patchwork_13566
Summary
== Series Details ==
Series: i915 vgpu PV to improve vgpu performance
URL : https://patchwork.freedesktop.org/series/6/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6428_full -> Patchwork_13557_full
Summary
---
== Series Details ==
Series: drm/i915: Plane cdclk requirements and fp16 for gen4+
URL : https://patchwork.freedesktop.org/series/63373/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6429 -> Patchwork_13562
Summary
---
For symmetry with the get_dplls() hook which sets the shared_dpll
pointer clear the same pointer from the put_dplls() hook.
While at it also constify the old crtc state.
v2:
- Constify the old crtc state. (Ville)
Signed-off-by: Imre Deak
Reviewed-by: Ville Syrjälä
---
For consistency clear the icl_port_dplls from the new crtc state, when
releasing the DPLLs from the old crtc state. Leaving them set could
result in releasing the same PLLs multiple times from the same CRTC
state incorrectly (if the same CRTC was first used for a TypeC port then
for a combo PHY
Reorder the error paths so that we unwind all the locals from any error
path and so avoid setting off divers alarum in case we find an error in
case we find an error.
References: https://bugs.freedesktop.org/show_bug.cgi?id=111048
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
---
Chris Wilson writes:
> Stop guessing over whether we have an extra wakeref held by the delayed
> fw put, and track it explicitly for the sake of debug.
>
> Signed-off-by: Chris Wilson
> Cc: Tvrtko Ursulin
> ---
> drivers/gpu/drm/i915/intel_uncore.c | 13 ++---
>
Quoting Mika Kuoppala (2019-07-08 16:40:30)
> Chris Wilson writes:
>
> > Stop guessing over whether we have an extra wakeref held by the delayed
> > fw put, and track it explicitly for the sake of debug.
> >
> > Signed-off-by: Chris Wilson
> > Cc: Tvrtko Ursulin
> > ---
> >
== Series Details ==
Series: drm/i915: Plane cdclk requirements and fp16 for gen4+
URL : https://patchwork.freedesktop.org/series/63373/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm: Add drm_modeset_lock_assert_held()
Okay!
Commit:
Quoting Tvrtko Ursulin (2019-07-08 13:24:39)
>
> On 08/07/2019 13:16, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2019-07-08 13:10:34)
> >>
> >> On 08/07/2019 12:19, Chris Wilson wrote:
> >>> Avoid a global idle barrier by reconfiguring each context by rewriting
> >>> them with MI_STORE_DWORD
Quoting Tvrtko Ursulin (2019-07-08 14:40:18)
>
> On 08/07/2019 14:32, Chris Wilson wrote:
> > Quoting Chris Wilson (2019-07-08 13:22:01)
> >> Quoting Tvrtko Ursulin (2019-07-08 13:18:02)
> >>>
> >>> On 08/07/2019 12:39, Chris Wilson wrote:
> v2: Though it doesn't affect the current users,
On Thu, Jul 04, 2019 at 04:56:41PM +0300, Imre Deak wrote:
On Wed, Jul 03, 2019 at 05:06:47PM -0700, Lucas De Marchi wrote:
Make intel_tc.h the first include so we guarantee it's self-contained.
Sort the rest. Same principle applies for includes in the header.
Signed-off-by: Lucas De Marchi
On 08/07/2019 14:32, Chris Wilson wrote:
Quoting Chris Wilson (2019-07-08 13:22:01)
Quoting Tvrtko Ursulin (2019-07-08 13:18:02)
On 08/07/2019 12:39, Chris Wilson wrote:
v2: Though it doesn't affect the current users, contexts may share
timelines so check if we already hold the right mutex.
Hi Chunyu,
First of all sorry for the delay on the response here.
Many of us were out on vacation and holidays.
Also in general please use https://bugs.freedesktop.org/
to report bugs.
Please make sure you read this before:
https://01.org/linuxgraphics/documentation/how-report-bugs
(more
Chris Wilson writes:
> Leave a few more breadcrumbs in the dmesg log to try and spot where the
> delay occurs.
>
> Signed-off-by: Chris Wilson
Reviewed-by: Mika Kuoppala
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
== Series Details ==
Series: series starting with [v2,1/2] drm/i915: Clear the shared PLL from the
put_dplls() hook
URL : https://patchwork.freedesktop.org/series/63384/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431 -> Patchwork_13564
On Wed, Jul 03, 2019 at 05:02:11PM -0700, Matt Roper wrote:
> Transition the remaining uses of intel_port_is_* over to the equivalent
> intel_phy_is_* functions and drop the port functions.
>
> v5: Fix a call in a debug function that's only called when
> CONFIG_DRM_I915_DEBUG_RUNTIME_PM is
set_page_dirty says:
For pages with a mapping this should be done under the page lock
for the benefit of asynchronous memory errors who prefer a
consistent dirty state. This rule can be broken in some special
cases, but should be better not to.
If the
On Mon, Jul 08, 2019 at 07:02:49AM -0700, Lucas De Marchi wrote:
> On Fri, Jul 05, 2019 at 01:33:10PM +0300, Ville Syrjälä wrote:
> >On Thu, Jul 04, 2019 at 08:55:51AM -0700, Lucas De Marchi wrote:
> >> On Thu, Jul 04, 2019 at 06:09:04PM +0300, Ville Syrjälä wrote:
> >> >On Thu, Jul 04, 2019 at
== Series Details ==
Series: drm/i915/userptr: Acquire the page lock around set_page_dirty()
URL : https://patchwork.freedesktop.org/series/63383/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6431 -> Patchwork_13563
On Thu, Jul 04, 2019 at 07:21:38PM +0200, Michal Wajdeczko wrote:
On Thu, 04 Jul 2019 02:06:47 +0200, Lucas De Marchi
wrote:
Make intel_tc.h the first include so we guarantee it's self-contained.
Sort the rest. Same principle applies for includes in the header.
Signed-off-by: Lucas De
Leave a few more breadcrumbs in the dmesg log to try and spot where the
delay occurs.
Signed-off-by: Chris Wilson
---
tests/i915/gem_eio.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/i915/gem_eio.c b/tests/i915/gem_eio.c
index dc7afb0fd..4d7362d8f 100644
---
1 - 100 of 165 matches
Mail list logo