On 2/8/2019 1:18 AM, Daniel Vetter wrote:
On Thu, Feb 07, 2019 at 08:40:08PM +0100, Daniel Vetter wrote:
On Thu, Feb 07, 2019 at 02:33:57AM +0530, Ramalingam C wrote:
Defining the mei-i915 interface functions and initialization of
the interface.
v2:
Adjust to the new interface changes. [T
Op 07-02-2019 om 17:27 schreef Ville Syrjälä:
> On Thu, Feb 07, 2019 at 04:49:47PM +0100, Maarten Lankhorst wrote:
>> Op 05-02-2019 om 17:08 schreef Ville Syrjala:
>>> From: Ville Syrjälä
>>>
>>> On pre-HSW gamma mode is configured via PIPECONF. The bits are
>>> the same except shifted up, so we c
On Fri, Feb 8, 2019 at 2:38 AM Stephen Rothwell wrote:
>
> Hi all,
>
> After merging the drm-misc tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> In file included from drivers/gpu/drm/arm/display/komeda/komeda_drv.c:13:
> drivers/gpu/drm/arm/display/komeda/komeda_kms.h
On Fri, Feb 08, 2019 at 12:27:44PM +1100, Stephen Rothwell wrote:
> Hi all,
>
> Today's linux-next merge of the drm-misc tree got a conflict in:
>
> drivers/gpu/drm/i915/intel_display.c
>
> between commit:
>
> 9f58892ea996 ("drm/i915: Pull all the reset functionality together into
> i915_r
Quoting Chris Wilson (2019-02-07 23:03:08)
> Make sure that the SIGARLM handler is captured so that we don't
> accidentally shoot ourselves in the foot instead of merely waking up the
> execbuf.
>
> Signed-off-by: Chris Wilson
Warming up the context ahead of time is reasonable for super low SKUs
Quoting Joonas Lahtinen (2019-02-08 08:59:53)
> Quoting Chris Wilson (2019-02-07 23:03:08)
> > Make sure that the SIGARLM handler is captured so that we don't
> > accidentally shoot ourselves in the foot instead of merely waking up the
> > execbuf.
> >
> > Signed-off-by: Chris Wilson
>
> Warming
Hi Daniel,
On Fri, 8 Feb 2019 09:53:33 +0100 Daniel Vetter wrote:
>
> There isn't a "below" afaics
Oops :-( However the merge diff is not very enlightening in this case
anyway. It makes more sense to look at the final result when I finish
up in a while.
--
Cheers,
Stephen Rothwell
pgp
Chris Wilson writes:
> When declaring the GPU wedged, we do need to hit the GPU with the reset
> hammer so that its state matches our presumed state during cleanup. If
> the reset fails, it fails, and we may be unhappy but wedged. However, if
> we are testing our wedge/unwedged handling, the desy
On Fri, Feb 8, 2019 at 10:26 AM Stephen Rothwell wrote:
>
> Hi Daniel,
>
> On Fri, 8 Feb 2019 09:53:33 +0100 Daniel Vetter wrote:
> >
> > There isn't a "below" afaics
>
> Oops :-( However the merge diff is not very enlightening in this case
> anyway. It makes more sense to look at the fina
Chris Wilson writes:
> On wedging, we mark all executing requests as complete and all pending
> requests completed as soon as they are ready. Before unwedging though we
> wish to flush those pending requests prior to restoring default
> execution, and so we must wait. Do so interruptibly as we do
Quoting Mika Kuoppala (2019-02-08 09:31:48)
> Chris Wilson writes:
>
> > When declaring the GPU wedged, we do need to hit the GPU with the reset
> > hammer so that its state matches our presumed state during cleanup. If
> > the reset fails, it fails, and we may be unhappy but wedged. However, if
Hi Stephen,
On Fri, Feb 08, 2019 at 12:32:04PM +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the drm-misc tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
>
> drivers/gpu/drm/sun4i/sun6i_mipi_dsi.c: In function 'sun6i_dsi_probe':
> drivers/gpu/drm/sun
+ Petri for the fact that igt_fork is actually igt_forkoontula and
messes up set up signal handlers when transitioning to child.
Quoting Chris Wilson (2019-02-08 11:01:55)
> Quoting Joonas Lahtinen (2019-02-08 08:59:53)
> > Quoting Chris Wilson (2019-02-07 23:03:08)
> > > Make sure that the SIGARL
Chris Wilson writes:
> Since we use the debugfs to recover the device after modifying the
> i915.reset parameter, we need to be sure that we apply the reset and not
> piggy-back onto a concurrent one in order for the parameter to take
> effect.
>
> Signed-off-by: Chris Wilson
> ---
> drivers/gp
Quoting Mika Kuoppala (2019-02-08 09:46:01)
> Chris Wilson writes:
>
> > On wedging, we mark all executing requests as complete and all pending
> > requests completed as soon as they are ready. Before unwedging though we
> > wish to flush those pending requests prior to restoring default
> > exec
Quoting Mika Kuoppala (2019-02-08 09:56:59)
> Chris Wilson writes:
>
> > Since we use the debugfs to recover the device after modifying the
> > i915.reset parameter, we need to be sure that we apply the reset and not
> > piggy-back onto a concurrent one in order for the parameter to take
> > effe
From: Tvrtko Ursulin
Two new output modes are added: listing of text data to standard out (-l
on the command line), and dumping of JSON formatted records (-J), also to
standard out.
The first mode is selected automatically when non-interactive standard out
is detected.
Signed-off-by: Tvrtko Urs
Quoting Tvrtko Ursulin (2019-02-08 10:26:12)
> From: Tvrtko Ursulin
>
> Two new output modes are added: listing of text data to standard out (-l
> on the command line), and dumping of JSON formatted records (-J), also to
> standard out.
Any chance of interactive with yaml^W json to file?
-Chris
On Fri, Feb 08, 2019 at 12:27:57AM +0100, Daniel Vetter wrote:
> Component framework is extended to support multiple components for
> a struct device. These will be matched with different masters based on
> its sub component value.
>
> We are introducing this, as I915 needs two different component
Quoting Tvrtko Ursulin (2019-02-08 10:26:12)
> +static unsigned int json_indent_level;
> +
> +static const char *json_indent[] = {
> + "",
> + "\t",
> + "\t\t",
> + "\t\t\t",
> + "\t\t\t\t",
> + "\t\t\t\t\t",
> +};
> +
> +#define ARRAY_SIZE(arr) (sizeof(arr)/size
On 08/02/2019 10:33, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2019-02-08 10:26:12)
From: Tvrtko Ursulin
Two new output modes are added: listing of text data to standard out (-l
on the command line), and dumping of JSON formatted records (-J), also to
standard out.
Any chance of interacti
Quoting Tvrtko Ursulin (2019-02-08 10:52:19)
>
> On 08/02/2019 10:33, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2019-02-08 10:26:12)
> >> From: Tvrtko Ursulin
> >>
> >> Two new output modes are added: listing of text data to standard out (-l
> >> on the command line), and dumping of JSON fo
On 08/02/2019 10:57, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2019-02-08 10:52:19)
On 08/02/2019 10:33, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2019-02-08 10:26:12)
From: Tvrtko Ursulin
Two new output modes are added: listing of text data to standard out (-l
on the command line), an
Still the blighted
Starting subtest: pi-ringfull-bsd
child 0 died with signal 14, Alarm clock
Subtest pi-ringfull-bsd: FAIL (0.133s)
So let's make sure our timer cannot fire later on during exit.
References https://bugs.freedesktop.org/show_bug.cgi?id=109584
Signed-off-by: Chris Wilson
Cc: Joon
On Fri, Feb 08, 2019 at 12:27:57AM +0100, Daniel Vetter wrote:
> Component framework is extended to support multiple components for
> a struct device. These will be matched with different masters based on
> its sub component value.
>
> We are introducing this, as I915 needs two different component
On Fri, Feb 08, 2019 at 12:27:56AM +0100, Daniel Vetter wrote:
> While typing these I think doing an s/component_master/aggregate/
> would be useful:
> - it's shorter :-)
> - I think component/aggregate is much more meaningful naming than
> component/puppetmaster or something like that. At least
From: Tvrtko Ursulin
A new -o command switch enables logging to a file.
Signed-off-by: Tvrtko Ursulin
References: https://bugs.freedesktop.org/show_bug.cgi?id=108689
Cc: Eero Tamminen
Cc: 3.1...@ukr.net
Cc: Chris Wilson
---
man/intel_gpu_top.rst | 18 -
tools/intel_gpu_top.c | 63
From: Tvrtko Ursulin
Two new output modes are added: listing of text data to standard out (-l
on the command line), and dumping of JSON formatted records (-J), also to
standard out.
The first mode is selected automatically when non-interactive standard out
is detected.
Example of text output:
Quoting Tvrtko Ursulin (2019-02-08 12:03:50)
> From: Tvrtko Ursulin
>
> Two new output modes are added: listing of text data to standard out (-l
> on the command line), and dumping of JSON formatted records (-J), also to
> standard out.
>
> The first mode is selected automatically when non-inter
Quoting Tvrtko Ursulin (2019-02-08 12:03:51)
> From: Tvrtko Ursulin
>
> A new -o command switch enables logging to a file.
>
> Signed-off-by: Tvrtko Ursulin
> References: https://bugs.freedesktop.org/show_bug.cgi?id=108689
> Cc: Eero Tamminen
> Cc: 3.1...@ukr.net
> Cc: Chris Wilson
> -
>> >> >-Original Message-
>> >> >From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com]
>> >> >Sent: Tuesday, February 5, 2019 10:02 PM
>> >> >To: Shankar, Uma
>> >> >Cc: intel-gfx@lists.freedesktop.org;
>> >> >dri-de...@lists.freedesktop.org; Syrjala, Ville
>> >> >; Lankhorst, Maarten
On Thu, Feb 07, 2019 at 03:11:59PM -0800, Lucas De Marchi wrote:
> On Thu, Feb 7, 2019 at 9:33 AM Ville Syrjala
> wrote:
> >
> > From: Ville Syrjälä
> >
> > Rather than try to maintain some magic relationship between the link
> > rates and the index into the wrpll params array let's just store
>
Quoting Chris Wilson (2019-02-08 13:41:09)
> Still the blighted
>
> Starting subtest: pi-ringfull-bsd
> child 0 died with signal 14, Alarm clock
> Subtest pi-ringfull-bsd: FAIL (0.133s)
>
> So let's make sure our timer cannot fire later on during exit.
This starts to reek like broken libc.
The
Regards
Shashank
> -Original Message-
> From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of
> Shankar, Uma
> Sent: Friday, February 8, 2019 5:45 PM
> To: Ville Syrjälä
> Cc: intel-gfx@lists.freedesktop.org; Syrjala, Ville
> ; dri-
> de...@lists.freedesktop.org;
Starting from opregion version 2.1 (roughly corresponding to ICL+) the
RVDA field is relative from the beginning of opregion, not absolute
address.
Fix the error path while at it.
v2: Make relative vs. absolute conditional on the opregion version,
bumped for the purpose. Turned out there are
The u32 version field encodes major version in the high word. We've been
checking for version >= 0.2.
Add opregion version logging while at it.
Fixes: 04ebaadb9f2d ("drm/i915/opregion: handle VBT sizes bigger than 6 KB")
Cc: Ville Syrjälä
Cc: Imre Deak
Signed-off-by: Jani Nikula
---
drivers/g
Temporary CI workaround before proper opregion version bump propagates
to the machines.
Cc: Ville Syrjälä
Cc: Imre Deak
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/intel_opregion.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_opregion
On Fri, Feb 08, 2019 at 12:36:25PM +, Sharma, Shashank wrote:
> Regards
> Shashank
>
> > -Original Message-
> > From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf
> > Of
> > Shankar, Uma
> > Sent: Friday, February 8, 2019 5:45 PM
> > To: Ville Syrjälä
> > Cc:
Regards
Shashank
On 2/8/2019 6:22 PM, Ville Syrjälä wrote:
On Fri, Feb 08, 2019 at 12:36:25PM +, Sharma, Shashank wrote:
Regards
Shashank
-Original Message-
From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of
Shankar, Uma
Sent: Friday, February 8, 2019 5
On Tue, Feb 05, 2019 at 03:42:05PM +0200, Jani Nikula wrote:
>
> The following commits have been marked as Cc: stable or fixing something
> in v5.0-rc5 or earlier, but failed to cherry-pick to
> drm-intel-fixes. Please see if they are worth backporting, and please do
> so if they are.
>
> Failed
On Fri, Feb 08, 2019 at 06:36:39PM +0530, Sharma, Shashank wrote:
> Regards
>
> Shashank
>
> On 2/8/2019 6:22 PM, Ville Syrjälä wrote:
> > On Fri, Feb 08, 2019 at 12:36:25PM +, Sharma, Shashank wrote:
> >> Regards
> >> Shashank
> >>
> >>> -Original Message-
> >>> From: Intel-gfx [mail
Hi,
On 8.2.2019 14.03, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin
Two new output modes are added: listing of text data to standard out (-l
on the command line), and dumping of JSON formatted records (-J), also to
standard out.
The first mode is selected automatically when non-interactive stan
If we allocate while iterating the rbtree of active nodes, we may hit
the shrinker and so retire the i915_active reap the rbtree. Modifying
the rbtree as we iterate is not good behaviour, so acquire the
i915_active first to keep the tree intact whenever we allocate.
Fixes: a42375af0a30 ("drm/i915:
From: Juha-Pekka Heikkila
Preparations for enabling P010, P012 and P016 formats. These
formats will extend NV12 for larger bit depths.
Signed-off-by: Juha-Pekka Heikkila
---
drivers/gpu/drm/i915/intel_atomic_plane.c | 2 +-
drivers/gpu/drm/i915/intel_display.c | 27 +-
From: Juha-Pekka Heikkila
Add needed plane control flag definitions for P010, P012 and
P016 formats.
Signed-off-by: Juha-Pekka Heikkila
---
drivers/gpu/drm/i915/i915_reg.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
i
From: Swati Sharma
Added needed plane control flag definitions for Y2xx and Y4xx (10, 12 and
16 bits)
Signed-off-by: Swati Sharma
Reviewed-by: Juha-Pekka Heikkila
---
drivers/gpu/drm/i915/i915_reg.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/dr
From: Swati Sharma
The following pixel formats are packed format that follows 4:2:2
chroma sampling. For memory represenation each component is
allocated 16 bits each. Thus each pixel occupies 32bit.
Y210: For each component, valid data occupies MSB 10 bits.
LSB 6 bits are filled with
From: Swati Sharma
In this patch, apart from enabling Y2xx and Y4xx pixel formats
P0xx pixel formats are added too for ICL.
Signed-off-by: Swati Sharma
Reviewed-by: Juha-Pekka Heikkila
---
drivers/gpu/drm/i915/intel_display.c | 30 ++
drivers/gpu/drm/i915/intel_sprite.c | 60
From: Juha-Pekka Heikkila
Enabling of P010, P012 and P016 formats. These formats will
extend NV12 for larger bit depths.
Signed-off-by: Juha-Pekka Heikkila
---
drivers/gpu/drm/i915/intel_sprite.c | 28 ++--
1 file changed, 26 insertions(+), 2 deletions(-)
diff --git a/
From: Juha-Pekka Heikkila
Add P010 definition, semi-planar yuv format where each component
is 16 bits 10 msb containing color value. First come Y plane [10:6]
followed by 2x2 subsampled Cr:Cb plane [10:6:10:6]
Add P012 definition, semi-planar yuv format where each component
is 16 bits 12 msb con
From: Swati Sharma
In this patch series, Juha Pekka's patch series for Gen10+ P0xx formats
https://patchwork.freedesktop.org/series/56053/ is combined with Swati's
https://patchwork.freedesktop.org/series/55035/ for Gen11+ pixel formats
(Y2xx and Y4xx).
P0xx pixel formats are enabled from GLK wh
This series makes drm_dev_unplug() compatible with the upcoming
devm_drm_dev_init(), fixes a double drm_dev_unregister() situation and
simplifies the drm_device ref handling wrt to the last fd closed after
unregister.
The first version of this patchset removed drm_dev_unplug(), see here
for the di
If userspace has open fd(s) when drm_dev_unplug() is run, it will result
in drm_dev_unregister() being called twice. First in drm_dev_unplug() and
then later in drm_release() through the call to drm_put_dev().
Since userspace already holds a ref on drm_device through the drm_minor,
it's not necess
This makes it possible to use drm_dev_unplug() with the upcoming
devm_drm_dev_init() which will do drm_dev_put() in its release callback.
Cc: Alex Deucher
Cc: Christian König
Cc: David (ChunMing) Zhou
Cc: Dave Airlie
Cc: Sean Paul
Cc: Oleksandr Andrushchenko
Cc: Daniel Vetter
Signed-off-by:
Regards
Shashank
On 2/8/2019 7:00 PM, Ville Syrjälä wrote:
On Fri, Feb 08, 2019 at 06:36:39PM +0530, Sharma, Shashank wrote:
Regards
Shashank
On 2/8/2019 6:22 PM, Ville Syrjälä wrote:
On Fri, Feb 08, 2019 at 12:36:25PM +, Sharma, Shashank wrote:
Regards
Shashank
-Original Message
== Series Details ==
Series: series starting with [1/3] drm/i915/opregion: fix version check
URL : https://patchwork.freedesktop.org/series/56395/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
d6761937ae7b drm/i915/opregion: fix version check
7b8f0501093b drm/i915/opregion: rvd
Op 06-02-2019 om 19:02 schreef Kevin Strasser:
> Add 64 bpp 16:16:16:16 half float pixel formats. Each 16 bit component is
> formatted in IEEE-754 half-precision float (binary16) 1:5:10
> MSb-sign:exponent:fraction form.
>
> This patch attempts to address the feedback provided when 2 of these
> for
== Series Details ==
Series: series starting with [1/3] drm/i915/opregion: fix version check
URL : https://patchwork.freedesktop.org/series/56395/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_5567 -> Patchwork_12175
Summar
Chris Wilson writes:
> Prevent concurrent set-wedge with ongoing resets (and vice versa) by
> taking the same wedge_mutex around both operations.
>
> Signed-off-by: Chris Wilson
Reviewed-by: Mika Kuoppala
> ---
> drivers/gpu/drm/i915/i915_reset.c | 68 ++-
> 1 fil
On 2/8/19 4:01 PM, Noralf Trønnes wrote:
This makes it possible to use drm_dev_unplug() with the upcoming
devm_drm_dev_init() which will do drm_dev_put() in its release callback.
Cc: Alex Deucher
Cc: Christian König
Cc: David (ChunMing) Zhou
Cc: Dave Airlie
Cc: Sean Paul
Cc: Oleksandr Andru
On Fri, Feb 08, 2019 at 07:36:24PM +0530, Sharma, Shashank wrote:
> Regards
>
> Shashank
>
> On 2/8/2019 7:00 PM, Ville Syrjälä wrote:
> > On Fri, Feb 08, 2019 at 06:36:39PM +0530, Sharma, Shashank wrote:
> >> Regards
> >>
> >> Shashank
> >>
> >> On 2/8/2019 6:22 PM, Ville Syrjälä wrote:
> >>> On
Check that the GPU even exists before submitting a batch.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109589
Signed-off-by: Chris Wilson
---
tests/kms_fence_pin_leak.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/kms_fence_pin_leak.c b/tests/kms_fence_pin_leak.c
index 62c
kms_fence_pin_leak tests smooth sharp edges that are i915 specific (and
requires using GEM to do so). It doesn't belong in the general paddock
of all driver tests, so move it into the i915/ stable.
Signed-off-by: Chris Wilson
Cc: Arkadiusz Hiler
Cc: Petri Latvala
---
tests/Makefile.sources
Hi,
On Fri, Feb 01, 2019 at 09:42:57PM +0530, Anshuman Gupta wrote:
> This patch series adds new tests to validate Display C states.
> DC states like DC5 and DC6 are validated during PSR entry/exit
> and during DPMS on/off cycle.
> Sending new revision of patch series after addressing review comme
On Fri, Feb 01, 2019 at 09:42:58PM +0530, Anshuman Gupta wrote:
> From: Jyoti Yadav
>
> It will be used by new test pm_dc.c which will validate Display C States.
> So moving the same to igt_pm library.
>
> v2: Simplify the comment section.
> v3: Remove . from the subject line.
> v4: Rebased and
Chris Wilson writes:
> If we haven't even begun executing the payload of the stalled request,
> then we should not claim that its userspace context was guilty of
> submitting a hanging batch.
>
> v2: Check for context corruption before trying to restart.
> v3: Preserve semaphores on skipping requ
Op 06-02-2019 om 19:02 schreef Kevin Strasser:
> 64 bpp half float formats are supported on hdr planes only and are subject
> to the following restrictions:
> * 90/270 rotation not supported
> * Yf Tiling not supported
> * Frame Buffer Compression not supported
> * Color Keying not supporte
Op 06-02-2019 om 19:02 schreef Kevin Strasser:
> This series defines new formats and adds implementation to the i915 driver.
> Since posting v1 I have removed the pixel normalize property, as it's not
> needed
> for basic functionality. Also, I have been working on adding support to
> userspace, b
On Fri, Feb 01, 2019 at 09:42:59PM +0530, Anshuman Gupta wrote:
> From: Jyoti Yadav
>
> Currently this test validates DC5 upon PSR entry for supported platforms.
> Added new file for compilation inside Makefile and Meson.
>
> v2: Used the debugfs entry for DC counters instead of Registers.
>
== Series Details ==
Series: drm/i915: Protect i915_active iterators from the shrinker
URL : https://patchwork.freedesktop.org/series/56401/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5567 -> Patchwork_12176
Summary
On Fri, Feb 08, 2019 at 02:44:58PM +, Chris Wilson wrote:
> kms_fence_pin_leak tests smooth sharp edges that are i915 specific (and
> requires using GEM to do so). It doesn't belong in the general paddock
> of all driver tests, so move it into the i915/ stable.
>
> Signed-off-by: Chris Wilson
On Fri, Feb 01, 2019 at 09:43:02PM +0530, Anshuman Gupta wrote:
> From: Jyoti Yadav
>
> Added new subtest for DC6 entry during DPMS on/off cycle.
> During DPMS on/off cycle DC6 counter is incremented.
>
> v2: Renamed the subtest name.
> v3: Rebased.
> v4: Rebased and address review comment.
>
>
Quoting Mika Kuoppala (2019-02-08 14:47:13)
> Chris Wilson writes:
>
> > If we haven't even begun executing the payload of the stalled request,
> > then we should not claim that its userspace context was guilty of
> > submitting a hanging batch.
> >
> > v2: Check for context corruption before try
Op 08-02-2019 om 14:47 schreef swati2.sha...@intel.com:
> From: Swati Sharma
>
> The following pixel formats are packed format that follows 4:2:2
> chroma sampling. For memory represenation each component is
> allocated 16 bits each. Thus each pixel occupies 32bit.
>
> Y210: For each component, va
>-Original Message-
>From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com]
>Sent: Friday, February 8, 2019 8:08 PM
>To: Sharma, Shashank
>Cc: Shankar, Uma ; intel-gfx@lists.freedesktop.org;
>Syrjala, Ville ; dri-de...@lists.freedesktop.org;
>Lankhorst,
>Maarten
>Subject: Re: [Inte
Chris Wilson writes:
> Quoting Mika Kuoppala (2019-02-08 09:46:01)
>> Chris Wilson writes:
>>
>> > On wedging, we mark all executing requests as complete and all pending
>> > requests completed as soon as they are ready. Before unwedging though we
>> > wish to flush those pending requests prior
On Fri, 08 Feb 2019, Jani Nikula wrote:
> The u32 version field encodes major version in the high word. We've been
> checking for version >= 0.2.
>
> Add opregion version logging while at it.
>
> Fixes: 04ebaadb9f2d ("drm/i915/opregion: handle VBT sizes bigger than 6 KB")
> Cc: Ville Syrjälä
> Cc
On Fri, Feb 08, 2019 at 05:09:51PM +0200, Jani Nikula wrote:
> On Fri, 08 Feb 2019, Jani Nikula wrote:
> > The u32 version field encodes major version in the high word. We've been
> > checking for version >= 0.2.
> >
> > Add opregion version logging while at it.
> >
> > Fixes: 04ebaadb9f2d ("drm/i
Quoting Mika Kuoppala (2019-02-08 15:07:55)
> Chris Wilson writes:
>
> > Quoting Mika Kuoppala (2019-02-08 09:46:01)
> >> Chris Wilson writes:
> >>
> >> > On wedging, we mark all executing requests as complete and all pending
> >> > requests completed as soon as they are ready. Before unwedging
== Series Details ==
Series: Enable P0xx (planar), Y2xx and Y4xx (packed) pixel formats
URL : https://patchwork.freedesktop.org/series/56402/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
38dc7fc809e3 drm: Add P010, P012, P016 format definitions and fourcc
-:28: WARNING:LONG_LI
== Series Details ==
Series: Enable P0xx (planar), Y2xx and Y4xx (packed) pixel formats
URL : https://patchwork.freedesktop.org/series/56402/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm: Add P010, P012, P016 format definitions and fourcc
Okay!
On Fri, Feb 08, 2019 at 10:28:49AM +, Russell King - ARM Linux admin wrote:
> On Fri, Feb 08, 2019 at 12:27:57AM +0100, Daniel Vetter wrote:
> > Component framework is extended to support multiple components for
> > a struct device. These will be matched with different masters based on
> > its
Op 31-01-2019 om 22:10 schreef Gwan-gyeong Mun:
> Bspec describes that GEN10 only supports capability of YUV 4:2:0 output to
> HDMI port and GEN11 supports capability of YUV 4:2:0 output to both DP and
> HDMI ports.
>
> Signed-off-by: Gwan-gyeong Mun
> ---
> drivers/gpu/drm/i915/intel_dp.c | 3 ++
On Fri, Feb 08, 2019 at 12:51:23PM +0100, Greg Kroah-Hartman wrote:
> On Fri, Feb 08, 2019 at 12:27:56AM +0100, Daniel Vetter wrote:
> > While typing these I think doing an s/component_master/aggregate/
> > would be useful:
> > - it's shorter :-)
> > - I think component/aggregate is much more meani
Op 31-01-2019 om 22:10 schreef Gwan-gyeong Mun:
> pipe_bpp value was assumed RGB therefore it was multiplied with 3.
> But YCbCr 4:2:0 requires multiplier value to 1.5 therefore it divides
> pipe_bpp to 2.
> - RGB bpp = bpc x 3
> - YCbCr 4:2:0 bpp = bpc x 1.5
>
> Signed-off-by: Gwan-gyeong Mun
>
Op 31-01-2019 om 22:10 schreef Gwan-gyeong Mun:
> Function intel_pixel_encoding_setup_vsc handles vsc header and data block
> setup for pixel encoding / colorimetry format.
>
> Setup VSC header and data block in function intel_pixel_encoding_setup_vsc
> for pixel encoding / colorimetry format as pe
== Series Details ==
Series: Enable P0xx (planar), Y2xx and Y4xx (packed) pixel formats
URL : https://patchwork.freedesktop.org/series/56402/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5567 -> Patchwork_12177
Summary
---
Chris Wilson writes:
> Quoting Mika Kuoppala (2019-02-08 14:47:13)
>> Chris Wilson writes:
>>
>> > If we haven't even begun executing the payload of the stalled request,
>> > then we should not claim that its userspace context was guilty of
>> > submitting a hanging batch.
>> >
>> > v2: Check f
On Fri, Feb 08, 2019 at 03:03:34PM +, Shankar, Uma wrote:
>
>
> >-Original Message-
> >From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com]
> >Sent: Friday, February 8, 2019 8:08 PM
> >To: Sharma, Shashank
> >Cc: Shankar, Uma ; intel-gfx@lists.freedesktop.org;
> >Syrjala, Ville
On Fri, Feb 8, 2019 at 4:23 PM Daniel Vetter wrote:
>
> On Fri, Feb 08, 2019 at 10:28:49AM +, Russell King - ARM Linux admin
> wrote:
> > On Fri, Feb 08, 2019 at 12:27:57AM +0100, Daniel Vetter wrote:
> > > Component framework is extended to support multiple components for
> > > a struct devi
Currently, we may simultaneously release the fence register from both
fence_update() and i915_gem_restore_fences(). This is dangerous, so
defer the bookkeeping entirely to i915_gem_restore_fences() when the
device is asleep.
Reported-by: Mika Kuoppala
Signed-off-by: Chris Wilson
Cc: Mika Kuoppal
On wedging, we mark all executing requests as complete and all pending
requests completed as soon as they are ready. Before unwedging though we
wish to flush those pending requests prior to restoring default
execution, and so we must wait. Do so uninterruptibly as we do not provide
the EINTR gracef
If we haven't even begun executing the payload of the stalled request,
then we should not claim that its userspace context was guilty of
submitting a hanging batch.
v2: Check for context corruption before trying to restart.
v3: Preserve semaphores on skipping requests (need to keep the timelines
i
Previously, we were able to rely on the recursive properties of
struct_mutex to allow us to serialise revoking mmaps and reacquiring the
FENCE registers with them being clobbered over a global device reset.
I then proceeded to throw out the baby with the bath water in order to
pursue a struct_mutex
Prevent concurrent set-wedge with ongoing resets (and vice versa) by
taking the same wedge_mutex around both operations.
Signed-off-by: Chris Wilson
Reviewed-by: Mika Kuoppala
---
drivers/gpu/drm/i915/i915_reset.c | 68 ++-
1 file changed, 40 insertions(+), 28 deleti
When declaring the GPU wedged, we do need to hit the GPU with the reset
hammer so that its state matches our presumed state during cleanup. If
the reset fails, it fails, and we may be unhappy but wedged. However, if
we are testing our wedge/unwedged handling, the desync carries over into
the next t
Since we use the debugfs to recover the device after modifying the
i915.reset parameter, we need to be sure that we apply the reset and not
piggy-back onto a concurrent one in order for the parameter to take
effect.
Signed-off-by: Chris Wilson
Reviewed-by: Mika Kuoppala
---
drivers/gpu/drm/i915
On Fri, Feb 8, 2019 at 4:29 PM Daniel Vetter wrote:
>
> On Fri, Feb 08, 2019 at 12:51:23PM +0100, Greg Kroah-Hartman wrote:
> > On Fri, Feb 08, 2019 at 12:27:56AM +0100, Daniel Vetter wrote:
> > > While typing these I think doing an s/component_master/aggregate/
> > > would be useful:
> > > - it's
== Series Details ==
Series: drm/drv: Rework drm_dev_unplug() (was: Remove drm_dev_unplug())
URL : https://patchwork.freedesktop.org/series/56404/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5567 -> Patchwork_12178
Summar
== Series Details ==
Series: series starting with [CI,1/7] drm/i915: Defer removing fence register
tracking to rpm wakeup
URL : https://patchwork.freedesktop.org/series/56412/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/i915: Defer removing fen
1 - 100 of 203 matches
Mail list logo
