* Re: [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
2024-09-27 0:01 [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state Alessandro Zanni
@ 2024-09-27 8:20 ` Jani Nikula
2024-09-27 11:57 ` Ville Syrjälä
2024-09-27 8:26 ` ✓ CI.Patch_applied: success for " Patchwork
` (3 subsequent siblings)
4 siblings, 1 reply; 10+ messages in thread
From: Jani Nikula @ 2024-09-27 8:20 UTC (permalink / raw)
To: Alessandro Zanni, rodrigo.vivi, joonas.lahtinen, tursulin,
airlied, simona
Cc: Alessandro Zanni, intel-gfx, intel-xe, dri-devel, linux-kernel,
skhan, anupnewsmail, Ville Syrjala
On Fri, 27 Sep 2024, Alessandro Zanni <alessandro.zanni87@gmail.com> wrote:
> This fix solves multiple Smatch errors:
>
> drivers/gpu/drm/i915/display/intel_atomic_plane.c:660
> intel_plane_atomic_check_with_state() error:
> we previously assumed 'fb' could be null (see line 648)
>
> drivers/gpu/drm/i915/display/intel_atomic_plane.c:664
> intel_plane_atomic_check_with_state()
> error: we previously assumed 'fb' could be null (see line 659)
>
> drivers/gpu/drm/i915/display/intel_atomic_plane.c:671
> intel_plane_atomic_check_with_state()
> error: we previously assumed 'fb' could be null (see line 663)
>
> We should check first if fb is not null before to access its properties.
new_plane_state->uapi.visible && !fb should not be possible, but it's
probably too hard for smatch to figure out. It's not exactly trivial for
humans to figure out either.
I'm thinking something like below to help both.
Ville, thoughts?
BR,
Jani.
diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
index 3505a5b52eb9..d9da47aed55d 100644
--- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
@@ -629,6 +629,9 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
if (ret)
return ret;
+ if (drm_WARN_ON(display->drm, new_plane_state->uapi.visible && !fb))
+ return -EINVAL;
+
if (fb)
new_crtc_state->enabled_planes |= BIT(plane->id);
>
> Signed-off-by: Alessandro Zanni <alessandro.zanni87@gmail.com>
> ---
> drivers/gpu/drm/i915/display/intel_atomic_plane.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> index e979786aa5cf..1606f79b39e6 100644
> --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> @@ -656,18 +656,18 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
> intel_plane_is_scaled(new_plane_state))
> new_crtc_state->scaled_planes |= BIT(plane->id);
>
> - if (new_plane_state->uapi.visible &&
> + if (new_plane_state->uapi.visible && fb &&
> intel_format_info_is_yuv_semiplanar(fb->format, fb->modifier))
> new_crtc_state->nv12_planes |= BIT(plane->id);
>
> - if (new_plane_state->uapi.visible &&
> + if (new_plane_state->uapi.visible && fb &&
> fb->format->format == DRM_FORMAT_C8)
> new_crtc_state->c8_planes |= BIT(plane->id);
>
> if (new_plane_state->uapi.visible || old_plane_state->uapi.visible)
> new_crtc_state->update_planes |= BIT(plane->id);
>
> - if (new_plane_state->uapi.visible &&
> + if (new_plane_state->uapi.visible && fb &&
> intel_format_info_is_yuv_semiplanar(fb->format, fb->modifier)) {
> new_crtc_state->data_rate_y[plane->id] =
> intel_plane_data_rate(new_crtc_state, new_plane_state, 0);
--
Jani Nikula, Intel
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
2024-09-27 8:20 ` Jani Nikula
@ 2024-09-27 11:57 ` Ville Syrjälä
2024-09-27 13:14 ` Jani Nikula
0 siblings, 1 reply; 10+ messages in thread
From: Ville Syrjälä @ 2024-09-27 11:57 UTC (permalink / raw)
To: Jani Nikula
Cc: Alessandro Zanni, rodrigo.vivi, joonas.lahtinen, tursulin,
airlied, simona, intel-gfx, intel-xe, dri-devel, linux-kernel,
skhan, anupnewsmail
On Fri, Sep 27, 2024 at 11:20:32AM +0300, Jani Nikula wrote:
> On Fri, 27 Sep 2024, Alessandro Zanni <alessandro.zanni87@gmail.com> wrote:
> > This fix solves multiple Smatch errors:
> >
> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:660
> > intel_plane_atomic_check_with_state() error:
> > we previously assumed 'fb' could be null (see line 648)
> >
> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:664
> > intel_plane_atomic_check_with_state()
> > error: we previously assumed 'fb' could be null (see line 659)
> >
> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:671
> > intel_plane_atomic_check_with_state()
> > error: we previously assumed 'fb' could be null (see line 663)
> >
> > We should check first if fb is not null before to access its properties.
>
> new_plane_state->uapi.visible && !fb should not be possible, but it's
> probably too hard for smatch to figure out. It's not exactly trivial for
> humans to figure out either.
>
> I'm thinking something like below to help both.
>
> Ville, thoughts?
>
>
> BR,
> Jani.
>
>
> diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> index 3505a5b52eb9..d9da47aed55d 100644
> --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> @@ -629,6 +629,9 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
> if (ret)
> return ret;
>
> + if (drm_WARN_ON(display->drm, new_plane_state->uapi.visible && !fb))
> + return -EINVAL;
> +
We have probably 100 places that would need this. So it's going
to be extremely ugly.
One approach I could maybe tolerate is something like
intel_plane_is_visible(plane_state)
{
if (drm_WARN_ON(visible && !fb))
return false;
return plane_state->visible;
}
+ s/plane_state->visible/intel_plane_is_visible(plane_state)/
But is that going to help these obtuse tools?
> if (fb)
> new_crtc_state->enabled_planes |= BIT(plane->id);
>
>
>
> >
> > Signed-off-by: Alessandro Zanni <alessandro.zanni87@gmail.com>
> > ---
> > drivers/gpu/drm/i915/display/intel_atomic_plane.c | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> > index e979786aa5cf..1606f79b39e6 100644
> > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> > @@ -656,18 +656,18 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
> > intel_plane_is_scaled(new_plane_state))
> > new_crtc_state->scaled_planes |= BIT(plane->id);
> >
> > - if (new_plane_state->uapi.visible &&
> > + if (new_plane_state->uapi.visible && fb &&
> > intel_format_info_is_yuv_semiplanar(fb->format, fb->modifier))
> > new_crtc_state->nv12_planes |= BIT(plane->id);
> >
> > - if (new_plane_state->uapi.visible &&
> > + if (new_plane_state->uapi.visible && fb &&
> > fb->format->format == DRM_FORMAT_C8)
> > new_crtc_state->c8_planes |= BIT(plane->id);
> >
> > if (new_plane_state->uapi.visible || old_plane_state->uapi.visible)
> > new_crtc_state->update_planes |= BIT(plane->id);
> >
> > - if (new_plane_state->uapi.visible &&
> > + if (new_plane_state->uapi.visible && fb &&
> > intel_format_info_is_yuv_semiplanar(fb->format, fb->modifier)) {
> > new_crtc_state->data_rate_y[plane->id] =
> > intel_plane_data_rate(new_crtc_state, new_plane_state, 0);
>
> --
> Jani Nikula, Intel
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
2024-09-27 11:57 ` Ville Syrjälä
@ 2024-09-27 13:14 ` Jani Nikula
2024-09-27 13:45 ` Ville Syrjälä
0 siblings, 1 reply; 10+ messages in thread
From: Jani Nikula @ 2024-09-27 13:14 UTC (permalink / raw)
To: Ville Syrjälä
Cc: Alessandro Zanni, rodrigo.vivi, joonas.lahtinen, tursulin,
airlied, simona, intel-gfx, intel-xe, dri-devel, linux-kernel,
skhan, anupnewsmail
On Fri, 27 Sep 2024, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> On Fri, Sep 27, 2024 at 11:20:32AM +0300, Jani Nikula wrote:
>> On Fri, 27 Sep 2024, Alessandro Zanni <alessandro.zanni87@gmail.com> wrote:
>> > This fix solves multiple Smatch errors:
>> >
>> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:660
>> > intel_plane_atomic_check_with_state() error:
>> > we previously assumed 'fb' could be null (see line 648)
>> >
>> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:664
>> > intel_plane_atomic_check_with_state()
>> > error: we previously assumed 'fb' could be null (see line 659)
>> >
>> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:671
>> > intel_plane_atomic_check_with_state()
>> > error: we previously assumed 'fb' could be null (see line 663)
>> >
>> > We should check first if fb is not null before to access its properties.
>>
>> new_plane_state->uapi.visible && !fb should not be possible, but it's
>> probably too hard for smatch to figure out. It's not exactly trivial for
>> humans to figure out either.
>>
>> I'm thinking something like below to help both.
>>
>> Ville, thoughts?
>>
>>
>> BR,
>> Jani.
>>
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
>> index 3505a5b52eb9..d9da47aed55d 100644
>> --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
>> +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
>> @@ -629,6 +629,9 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
>> if (ret)
>> return ret;
>>
>> + if (drm_WARN_ON(display->drm, new_plane_state->uapi.visible && !fb))
>> + return -EINVAL;
>> +
>
> We have probably 100 places that would need this. So it's going
> to be extremely ugly.
>
> One approach I could maybe tolerate is something like
> intel_plane_is_visible(plane_state)
> {
> if (drm_WARN_ON(visible && !fb))
> return false;
>
> return plane_state->visible;
> }
>
> + s/plane_state->visible/intel_plane_is_visible(plane_state)/
>
> But is that going to help these obtuse tools?
That does help people, which is more important. :)
I think the problem is first checking if fb is NULL, and then
dereferencing it anyway.
visible always means fb != NULL, but I forget, is the reverse true? Can
we have fb != NULL and !visible? I mean could we change the fb check to
visible check?
BR,
Jani.
>
>> if (fb)
>> new_crtc_state->enabled_planes |= BIT(plane->id);
>>
>>
>>
>> >
>> > Signed-off-by: Alessandro Zanni <alessandro.zanni87@gmail.com>
>> > ---
>> > drivers/gpu/drm/i915/display/intel_atomic_plane.c | 6 +++---
>> > 1 file changed, 3 insertions(+), 3 deletions(-)
>> >
>> > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
>> > index e979786aa5cf..1606f79b39e6 100644
>> > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
>> > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
>> > @@ -656,18 +656,18 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
>> > intel_plane_is_scaled(new_plane_state))
>> > new_crtc_state->scaled_planes |= BIT(plane->id);
>> >
>> > - if (new_plane_state->uapi.visible &&
>> > + if (new_plane_state->uapi.visible && fb &&
>> > intel_format_info_is_yuv_semiplanar(fb->format, fb->modifier))
>> > new_crtc_state->nv12_planes |= BIT(plane->id);
>> >
>> > - if (new_plane_state->uapi.visible &&
>> > + if (new_plane_state->uapi.visible && fb &&
>> > fb->format->format == DRM_FORMAT_C8)
>> > new_crtc_state->c8_planes |= BIT(plane->id);
>> >
>> > if (new_plane_state->uapi.visible || old_plane_state->uapi.visible)
>> > new_crtc_state->update_planes |= BIT(plane->id);
>> >
>> > - if (new_plane_state->uapi.visible &&
>> > + if (new_plane_state->uapi.visible && fb &&
>> > intel_format_info_is_yuv_semiplanar(fb->format, fb->modifier)) {
>> > new_crtc_state->data_rate_y[plane->id] =
>> > intel_plane_data_rate(new_crtc_state, new_plane_state, 0);
>>
>> --
>> Jani Nikula, Intel
--
Jani Nikula, Intel
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
2024-09-27 13:14 ` Jani Nikula
@ 2024-09-27 13:45 ` Ville Syrjälä
2024-09-27 14:07 ` Ville Syrjälä
0 siblings, 1 reply; 10+ messages in thread
From: Ville Syrjälä @ 2024-09-27 13:45 UTC (permalink / raw)
To: Jani Nikula
Cc: Alessandro Zanni, rodrigo.vivi, joonas.lahtinen, tursulin,
airlied, simona, intel-gfx, intel-xe, dri-devel, linux-kernel,
skhan, anupnewsmail
On Fri, Sep 27, 2024 at 04:14:17PM +0300, Jani Nikula wrote:
> On Fri, 27 Sep 2024, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> > On Fri, Sep 27, 2024 at 11:20:32AM +0300, Jani Nikula wrote:
> >> On Fri, 27 Sep 2024, Alessandro Zanni <alessandro.zanni87@gmail.com> wrote:
> >> > This fix solves multiple Smatch errors:
> >> >
> >> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:660
> >> > intel_plane_atomic_check_with_state() error:
> >> > we previously assumed 'fb' could be null (see line 648)
> >> >
> >> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:664
> >> > intel_plane_atomic_check_with_state()
> >> > error: we previously assumed 'fb' could be null (see line 659)
> >> >
> >> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:671
> >> > intel_plane_atomic_check_with_state()
> >> > error: we previously assumed 'fb' could be null (see line 663)
> >> >
> >> > We should check first if fb is not null before to access its properties.
> >>
> >> new_plane_state->uapi.visible && !fb should not be possible, but it's
> >> probably too hard for smatch to figure out. It's not exactly trivial for
> >> humans to figure out either.
> >>
> >> I'm thinking something like below to help both.
> >>
> >> Ville, thoughts?
> >>
> >>
> >> BR,
> >> Jani.
> >>
> >>
> >> diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> >> index 3505a5b52eb9..d9da47aed55d 100644
> >> --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> >> +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> >> @@ -629,6 +629,9 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
> >> if (ret)
> >> return ret;
> >>
> >> + if (drm_WARN_ON(display->drm, new_plane_state->uapi.visible && !fb))
> >> + return -EINVAL;
> >> +
> >
> > We have probably 100 places that would need this. So it's going
> > to be extremely ugly.
> >
> > One approach I could maybe tolerate is something like
> > intel_plane_is_visible(plane_state)
> > {
> > if (drm_WARN_ON(visible && !fb))
> > return false;
> >
> > return plane_state->visible;
> > }
> >
> > + s/plane_state->visible/intel_plane_is_visible(plane_state)/
> >
> > But is that going to help these obtuse tools?
>
> That does help people, which is more important. :)
>
> I think the problem is first checking if fb is NULL, and then
> dereferencing it anyway.
>
> visible always means fb != NULL, but I forget, is the reverse true? Can
> we have fb != NULL and !visible? I mean could we change the fb check to
> visible check?
No, the reverse does not hold. A plane can be invisible
while still having a valid fb. Eg. the plane could be
positioned completely offscreen, or the entire crtc may
be inactive (DPMS off).
And whenever we have an fb we want to do all the check to make sure
it satisfies all the requirements, whether the plane is visible or
not. Otherwise we could end up confusing userspace with something
like this:
1. Usespace assigns some unsupported fb to the plane
but positions the plane offscreen -> success
2. Userspace moves the plane to somewhere onscreen -> fail
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
2024-09-27 13:45 ` Ville Syrjälä
@ 2024-09-27 14:07 ` Ville Syrjälä
0 siblings, 0 replies; 10+ messages in thread
From: Ville Syrjälä @ 2024-09-27 14:07 UTC (permalink / raw)
To: Jani Nikula
Cc: Alessandro Zanni, rodrigo.vivi, joonas.lahtinen, tursulin,
airlied, simona, intel-gfx, intel-xe, dri-devel, linux-kernel,
skhan, anupnewsmail
On Fri, Sep 27, 2024 at 04:45:44PM +0300, Ville Syrjälä wrote:
> On Fri, Sep 27, 2024 at 04:14:17PM +0300, Jani Nikula wrote:
> > On Fri, 27 Sep 2024, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> > > On Fri, Sep 27, 2024 at 11:20:32AM +0300, Jani Nikula wrote:
> > >> On Fri, 27 Sep 2024, Alessandro Zanni <alessandro.zanni87@gmail.com> wrote:
> > >> > This fix solves multiple Smatch errors:
> > >> >
> > >> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:660
> > >> > intel_plane_atomic_check_with_state() error:
> > >> > we previously assumed 'fb' could be null (see line 648)
> > >> >
> > >> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:664
> > >> > intel_plane_atomic_check_with_state()
> > >> > error: we previously assumed 'fb' could be null (see line 659)
> > >> >
> > >> > drivers/gpu/drm/i915/display/intel_atomic_plane.c:671
> > >> > intel_plane_atomic_check_with_state()
> > >> > error: we previously assumed 'fb' could be null (see line 663)
> > >> >
> > >> > We should check first if fb is not null before to access its properties.
> > >>
> > >> new_plane_state->uapi.visible && !fb should not be possible, but it's
> > >> probably too hard for smatch to figure out. It's not exactly trivial for
> > >> humans to figure out either.
> > >>
> > >> I'm thinking something like below to help both.
> > >>
> > >> Ville, thoughts?
> > >>
> > >>
> > >> BR,
> > >> Jani.
> > >>
> > >>
> > >> diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> > >> index 3505a5b52eb9..d9da47aed55d 100644
> > >> --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> > >> +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> > >> @@ -629,6 +629,9 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
> > >> if (ret)
> > >> return ret;
> > >>
> > >> + if (drm_WARN_ON(display->drm, new_plane_state->uapi.visible && !fb))
> > >> + return -EINVAL;
> > >> +
> > >
> > > We have probably 100 places that would need this. So it's going
> > > to be extremely ugly.
> > >
> > > One approach I could maybe tolerate is something like
> > > intel_plane_is_visible(plane_state)
> > > {
> > > if (drm_WARN_ON(visible && !fb))
> > > return false;
> > >
> > > return plane_state->visible;
> > > }
> > >
> > > + s/plane_state->visible/intel_plane_is_visible(plane_state)/
> > >
> > > But is that going to help these obtuse tools?
> >
> > That does help people, which is more important. :)
> >
> > I think the problem is first checking if fb is NULL, and then
> > dereferencing it anyway.
> >
> > visible always means fb != NULL, but I forget, is the reverse true? Can
> > we have fb != NULL and !visible? I mean could we change the fb check to
> > visible check?
>
> No, the reverse does not hold. A plane can be invisible
> while still having a valid fb. Eg. the plane could be
> positioned completely offscreen, or the entire crtc may
> be inactive (DPMS off).
>
> And whenever we have an fb we want to do all the check to make sure
> it satisfies all the requirements, whether the plane is visible or
> not. Otherwise we could end up confusing userspace with something
> like this:
>
> 1. Usespace assigns some unsupported fb to the plane
> but positions the plane offscreen -> success
> 2. Userspace moves the plane to somewhere onscreen -> fail
Basically planes should have three different "enabled" states:
logically_enabled: fb!=NULL (also the crtc must be logically enabled,
but drm_atomic_plane_check() guarantees
this for us)
visible: logically_enabled && dst rectangle is at least
partially within pipe_src rectangle
active: visible && crtc_is_active
Currently we try to make the proper distinction between
logically_enabled vs. invisible, but we do not properly
handle the visible vs. active case. That is, we currently
mark the plane as invisible if the crtc is inactive.
That means we eg. calculate watermarks as if the plane was
invisible. That may cause a subsequent "DPMS on" operation
to fail unexpectedly because all of a sudden we realize
that we don't have enough FIFO space for this particular
plane configuration. There's a FIXME somewhere in the plane
code about this.
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 10+ messages in thread
* ✓ CI.Patch_applied: success for gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
2024-09-27 0:01 [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state Alessandro Zanni
2024-09-27 8:20 ` Jani Nikula
@ 2024-09-27 8:26 ` Patchwork
2024-09-27 8:26 ` ✗ CI.checkpatch: warning " Patchwork
` (2 subsequent siblings)
4 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2024-09-27 8:26 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-xe
== Series Details ==
Series: gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
URL : https://patchwork.freedesktop.org/series/139198/
State : success
== Summary ==
=== Applying kernel patches on branch 'drm-tip' with base: ===
Base commit: c114fec0af7e drm-tip: 2024y-09m-27d-07h-08m-52s UTC integration manifest
=== git am output follows ===
Applying: gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
^ permalink raw reply [flat|nested] 10+ messages in thread* ✗ CI.checkpatch: warning for gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
2024-09-27 0:01 [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state Alessandro Zanni
2024-09-27 8:20 ` Jani Nikula
2024-09-27 8:26 ` ✓ CI.Patch_applied: success for " Patchwork
@ 2024-09-27 8:26 ` Patchwork
2024-09-27 8:28 ` ✓ CI.KUnit: success " Patchwork
2024-09-27 8:32 ` ✗ CI.Build: failure " Patchwork
4 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2024-09-27 8:26 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-xe
== Series Details ==
Series: gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
URL : https://patchwork.freedesktop.org/series/139198/
State : warning
== Summary ==
+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit c44d377fcea6a6ec5c0b37bf778604da26f5fada
Author: Jani Nikula <jani.nikula@linux.intel.com>
Date: Fri Sep 27 11:20:32 2024 +0300
gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
On Fri, 27 Sep 2024, Alessandro Zanni <alessandro.zanni87@gmail.com> wrote:
> This fix solves multiple Smatch errors:
>
> drivers/gpu/drm/i915/display/intel_atomic_plane.c:660
> intel_plane_atomic_check_with_state() error:
> we previously assumed 'fb' could be null (see line 648)
>
> drivers/gpu/drm/i915/display/intel_atomic_plane.c:664
> intel_plane_atomic_check_with_state()
> error: we previously assumed 'fb' could be null (see line 659)
>
> drivers/gpu/drm/i915/display/intel_atomic_plane.c:671
> intel_plane_atomic_check_with_state()
> error: we previously assumed 'fb' could be null (see line 663)
>
> We should check first if fb is not null before to access its properties.
new_plane_state->uapi.visible && !fb should not be possible, but it's
probably too hard for smatch to figure out. It's not exactly trivial for
humans to figure out either.
I'm thinking something like below to help both.
Ville, thoughts?
BR,
Jani.
>
> Signed-off-by: Alessandro Zanni <alessandro.zanni87@gmail.com>
> ---
> drivers/gpu/drm/i915/display/intel_atomic_plane.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> index e979786aa5cf..1606f79b39e6 100644
> --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> @@ -656,18 +656,18 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
> intel_plane_is_scaled(new_plane_state))
> new_crtc_state->scaled_planes |= BIT(plane->id);
>
> - if (new_plane_state->uapi.visible &&
> + if (new_plane_state->uapi.visible && fb &&
> intel_format_info_is_yuv_semiplanar(fb->format, fb->modifier))
> new_crtc_state->nv12_planes |= BIT(plane->id);
>
> - if (new_plane_state->uapi.visible &&
> + if (new_plane_state->uapi.visible && fb &&
> fb->format->format == DRM_FORMAT_C8)
> new_crtc_state->c8_planes |= BIT(plane->id);
>
> if (new_plane_state->uapi.visible || old_plane_state->uapi.visible)
> new_crtc_state->update_planes |= BIT(plane->id);
>
> - if (new_plane_state->uapi.visible &&
> + if (new_plane_state->uapi.visible && fb &&
> intel_format_info_is_yuv_semiplanar(fb->format, fb->modifier)) {
> new_crtc_state->data_rate_y[plane->id] =
> intel_plane_data_rate(new_crtc_state, new_plane_state, 0);
+ /mt/dim checkpatch c114fec0af7e702962ba625f659e4647a9887194 drm-intel
c44d377fcea6 gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
-:41: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#41:
> diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
-:81: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)
total: 1 errors, 1 warnings, 0 checks, 9 lines checked
^ permalink raw reply [flat|nested] 10+ messages in thread* ✓ CI.KUnit: success for gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
2024-09-27 0:01 [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state Alessandro Zanni
` (2 preceding siblings ...)
2024-09-27 8:26 ` ✗ CI.checkpatch: warning " Patchwork
@ 2024-09-27 8:28 ` Patchwork
2024-09-27 8:32 ` ✗ CI.Build: failure " Patchwork
4 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2024-09-27 8:28 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-xe
== Series Details ==
Series: gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
URL : https://patchwork.freedesktop.org/series/139198/
State : success
== Summary ==
+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[08:26:56] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[08:27:00] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make ARCH=um O=.kunit --jobs=48
../lib/iomap.c:156:5: warning: no previous prototype for ‘ioread64_lo_hi’ [-Wmissing-prototypes]
156 | u64 ioread64_lo_hi(const void __iomem *addr)
| ^~~~~~~~~~~~~~
../lib/iomap.c:163:5: warning: no previous prototype for ‘ioread64_hi_lo’ [-Wmissing-prototypes]
163 | u64 ioread64_hi_lo(const void __iomem *addr)
| ^~~~~~~~~~~~~~
../lib/iomap.c:170:5: warning: no previous prototype for ‘ioread64be_lo_hi’ [-Wmissing-prototypes]
170 | u64 ioread64be_lo_hi(const void __iomem *addr)
| ^~~~~~~~~~~~~~~~
../lib/iomap.c:178:5: warning: no previous prototype for ‘ioread64be_hi_lo’ [-Wmissing-prototypes]
178 | u64 ioread64be_hi_lo(const void __iomem *addr)
| ^~~~~~~~~~~~~~~~
../lib/iomap.c:264:6: warning: no previous prototype for ‘iowrite64_lo_hi’ [-Wmissing-prototypes]
264 | void iowrite64_lo_hi(u64 val, void __iomem *addr)
| ^~~~~~~~~~~~~~~
../lib/iomap.c:272:6: warning: no previous prototype for ‘iowrite64_hi_lo’ [-Wmissing-prototypes]
272 | void iowrite64_hi_lo(u64 val, void __iomem *addr)
| ^~~~~~~~~~~~~~~
../lib/iomap.c:280:6: warning: no previous prototype for ‘iowrite64be_lo_hi’ [-Wmissing-prototypes]
280 | void iowrite64be_lo_hi(u64 val, void __iomem *addr)
| ^~~~~~~~~~~~~~~~~
../lib/iomap.c:288:6: warning: no previous prototype for ‘iowrite64be_hi_lo’ [-Wmissing-prototypes]
288 | void iowrite64be_hi_lo(u64 val, void __iomem *addr)
| ^~~~~~~~~~~~~~~~~
[08:27:27] Starting KUnit Kernel (1/1)...
[08:27:27] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[08:27:28] =================== guc_dbm (7 subtests) ===================
[08:27:28] [PASSED] test_empty
[08:27:28] [PASSED] test_default
[08:27:28] ======================== test_size ========================
[08:27:28] [PASSED] 4
[08:27:28] [PASSED] 8
[08:27:28] [PASSED] 32
[08:27:28] [PASSED] 256
[08:27:28] ==================== [PASSED] test_size ====================
[08:27:28] ======================= test_reuse ========================
[08:27:28] [PASSED] 4
[08:27:28] [PASSED] 8
[08:27:28] [PASSED] 32
[08:27:28] [PASSED] 256
[08:27:28] =================== [PASSED] test_reuse ====================
[08:27:28] =================== test_range_overlap ====================
[08:27:28] [PASSED] 4
[08:27:28] [PASSED] 8
[08:27:28] [PASSED] 32
[08:27:28] [PASSED] 256
[08:27:28] =============== [PASSED] test_range_overlap ================
[08:27:28] =================== test_range_compact ====================
[08:27:28] [PASSED] 4
[08:27:28] [PASSED] 8
[08:27:28] [PASSED] 32
[08:27:28] [PASSED] 256
[08:27:28] =============== [PASSED] test_range_compact ================
[08:27:28] ==================== test_range_spare =====================
[08:27:28] [PASSED] 4
[08:27:28] [PASSED] 8
[08:27:28] [PASSED] 32
[08:27:28] [PASSED] 256
[08:27:28] ================ [PASSED] test_range_spare =================
[08:27:28] ===================== [PASSED] guc_dbm =====================
[08:27:28] =================== guc_idm (6 subtests) ===================
[08:27:28] [PASSED] bad_init
[08:27:28] [PASSED] no_init
[08:27:28] [PASSED] init_fini
[08:27:28] [PASSED] check_used
[08:27:28] [PASSED] check_quota
[08:27:28] [PASSED] check_all
[08:27:28] ===================== [PASSED] guc_idm =====================
[08:27:28] ================== no_relay (3 subtests) ===================
[08:27:28] [PASSED] xe_drops_guc2pf_if_not_ready
[08:27:28] [PASSED] xe_drops_guc2vf_if_not_ready
[08:27:28] [PASSED] xe_rejects_send_if_not_ready
[08:27:28] ==================== [PASSED] no_relay =====================
[08:27:28] ================== pf_relay (14 subtests) ==================
[08:27:28] [PASSED] pf_rejects_guc2pf_too_short
[08:27:28] [PASSED] pf_rejects_guc2pf_too_long
[08:27:28] [PASSED] pf_rejects_guc2pf_no_payload
[08:27:28] [PASSED] pf_fails_no_payload
[08:27:28] [PASSED] pf_fails_bad_origin
[08:27:28] [PASSED] pf_fails_bad_type
[08:27:28] [PASSED] pf_txn_reports_error
[08:27:28] [PASSED] pf_txn_sends_pf2guc
[08:27:28] [PASSED] pf_sends_pf2guc
[08:27:28] [SKIPPED] pf_loopback_nop
[08:27:28] [SKIPPED] pf_loopback_echo
[08:27:28] [SKIPPED] pf_loopback_fail
[08:27:28] [SKIPPED] pf_loopback_busy
[08:27:28] [SKIPPED] pf_loopback_retry
[08:27:28] ==================== [PASSED] pf_relay =====================
[08:27:28] ================== vf_relay (3 subtests) ===================
[08:27:28] [PASSED] vf_rejects_guc2vf_too_short
[08:27:28] [PASSED] vf_rejects_guc2vf_too_long
[08:27:28] [PASSED] vf_rejects_guc2vf_no_payload
[08:27:28] ==================== [PASSED] vf_relay =====================
[08:27:28] ================= pf_service (11 subtests) =================
[08:27:28] [PASSED] pf_negotiate_any
[08:27:28] [PASSED] pf_negotiate_base_match
[08:27:28] [PASSED] pf_negotiate_base_newer
[08:27:28] [PASSED] pf_negotiate_base_next
[08:27:28] [SKIPPED] pf_negotiate_base_older
[08:27:28] [PASSED] pf_negotiate_base_prev
[08:27:28] [PASSED] pf_negotiate_latest_match
[08:27:28] [PASSED] pf_negotiate_latest_newer
[08:27:28] [PASSED] pf_negotiate_latest_next
[08:27:28] [SKIPPED] pf_negotiate_latest_older
[08:27:28] [SKIPPED] pf_negotiate_latest_prev
[08:27:28] =================== [PASSED] pf_service ====================
[08:27:28] ===================== lmtt (1 subtest) =====================
[08:27:28] ======================== test_ops =========================
[08:27:28] [PASSED] 2-level
[08:27:28] [PASSED] multi-level
[08:27:28] ==================== [PASSED] test_ops =====================
[08:27:28] ====================== [PASSED] lmtt =======================
[08:27:28] =================== xe_mocs (2 subtests) ===================
[08:27:28] ================ xe_live_mocs_kernel_kunit ================
[08:27:28] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[08:27:28] ================ xe_live_mocs_reset_kunit =================
[08:27:28] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[08:27:28] ==================== [SKIPPED] xe_mocs =====================
[08:27:28] ================= xe_migrate (2 subtests) ==================
[08:27:28] ================= xe_migrate_sanity_kunit =================
[08:27:28] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[08:27:28] ================== xe_validate_ccs_kunit ==================
[08:27:28] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[08:27:28] =================== [SKIPPED] xe_migrate ===================
[08:27:28] ================== xe_dma_buf (1 subtest) ==================
[08:27:28] ==================== xe_dma_buf_kunit =====================
[08:27:28] ================ [SKIPPED] xe_dma_buf_kunit ================
[08:27:28] =================== [SKIPPED] xe_dma_buf ===================
[08:27:28] ==================== xe_bo (3 subtests) ====================
[08:27:28] ================== xe_ccs_migrate_kunit ===================
[08:27:28] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[08:27:28] ==================== xe_bo_evict_kunit ====================
[08:27:28] =============== [SKIPPED] xe_bo_evict_kunit ================
[08:27:28] =================== xe_bo_shrink_kunit ====================
[08:27:28] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[08:27:28] ===================== [SKIPPED] xe_bo ======================
[08:27:28] ==================== args (11 subtests) ====================
[08:27:28] [PASSED] count_args_test
[08:27:28] [PASSED] call_args_example
[08:27:28] [PASSED] call_args_test
[08:27:28] [PASSED] drop_first_arg_example
[08:27:28] [PASSED] drop_first_arg_test
[08:27:28] [PASSED] first_arg_example
[08:27:28] [PASSED] first_arg_test
[08:27:28] [PASSED] last_arg_example
[08:27:28] [PASSED] last_arg_test
[08:27:28] [PASSED] pick_arg_example
[08:27:28] [PASSED] sep_comma_example
stty: 'standard input': Inappropriate ioctl for device
[08:27:28] ====================== [PASSED] args =======================
[08:27:28] =================== xe_pci (2 subtests) ====================
[08:27:28] [PASSED] xe_gmdid_graphics_ip
[08:27:28] [PASSED] xe_gmdid_media_ip
[08:27:28] ===================== [PASSED] xe_pci ======================
[08:27:28] =================== xe_rtp (2 subtests) ====================
[08:27:28] =============== xe_rtp_process_to_sr_tests ================
[08:27:28] [PASSED] coalesce-same-reg
[08:27:28] [PASSED] no-match-no-add
[08:27:28] [PASSED] match-or
[08:27:28] [PASSED] match-or-xfail
[08:27:28] [PASSED] no-match-no-add-multiple-rules
[08:27:28] [PASSED] two-regs-two-entries
[08:27:28] [PASSED] clr-one-set-other
[08:27:28] [PASSED] set-field
[08:27:28] [PASSED] conflict-duplicate
[08:27:28] [PASSED] conflict-not-disjoint
[08:27:28] [PASSED] conflict-reg-type
[08:27:28] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[08:27:28] ================== xe_rtp_process_tests ===================
[08:27:28] [PASSED] active1
[08:27:28] [PASSED] active2
[08:27:28] [PASSED] active-inactive
[08:27:28] [PASSED] inactive-active
[08:27:28] [PASSED] inactive-1st_or_active-inactive
[08:27:28] [PASSED] inactive-2nd_or_active-inactive
[08:27:28] [PASSED] inactive-last_or_active-inactive
[08:27:28] [PASSED] inactive-no_or_active-inactive
[08:27:28] ============== [PASSED] xe_rtp_process_tests ===============
[08:27:28] ===================== [PASSED] xe_rtp ======================
[08:27:28] ==================== xe_wa (1 subtest) =====================
[08:27:28] ======================== xe_wa_gt =========================
[08:27:28] [PASSED] TIGERLAKE (B0)
[08:27:28] [PASSED] DG1 (A0)
[08:27:28] [PASSED] DG1 (B0)
[08:27:28] [PASSED] ALDERLAKE_S (A0)
[08:27:28] [PASSED] ALDERLAKE_S (B0)
[08:27:28] [PASSED] ALDERLAKE_S (C0)
[08:27:28] [PASSED] ALDERLAKE_S (D0)
[08:27:28] [PASSED] ALDERLAKE_P (A0)
[08:27:28] [PASSED] ALDERLAKE_P (B0)
[08:27:28] [PASSED] ALDERLAKE_P (C0)
[08:27:28] [PASSED] ALDERLAKE_S_RPLS (D0)
[08:27:28] [PASSED] ALDERLAKE_P_RPLU (E0)
[08:27:28] [PASSED] DG2_G10 (C0)
[08:27:28] [PASSED] DG2_G11 (B1)
[08:27:28] [PASSED] DG2_G12 (A1)
[08:27:28] [PASSED] METEORLAKE (g:A0, m:A0)
[08:27:28] [PASSED] METEORLAKE (g:A0, m:A0)
[08:27:28] [PASSED] METEORLAKE (g:A0, m:A0)
[08:27:28] [PASSED] LUNARLAKE (g:A0, m:A0)
[08:27:28] [PASSED] LUNARLAKE (g:B0, m:A0)
[08:27:28] [PASSED] BATTLEMAGE (g:A0, m:A1)
[08:27:28] ==================== [PASSED] xe_wa_gt =====================
[08:27:28] ====================== [PASSED] xe_wa ======================
[08:27:28] ============================================================
[08:27:28] Testing complete. Ran 122 tests: passed: 106, skipped: 16
[08:27:28] Elapsed time: 31.862s total, 4.146s configuring, 27.449s building, 0.215s running
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[08:27:28] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[08:27:29] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make ARCH=um O=.kunit --jobs=48
../lib/iomap.c:156:5: warning: no previous prototype for ‘ioread64_lo_hi’ [-Wmissing-prototypes]
156 | u64 ioread64_lo_hi(const void __iomem *addr)
| ^~~~~~~~~~~~~~
../lib/iomap.c:163:5: warning: no previous prototype for ‘ioread64_hi_lo’ [-Wmissing-prototypes]
163 | u64 ioread64_hi_lo(const void __iomem *addr)
| ^~~~~~~~~~~~~~
../lib/iomap.c:170:5: warning: no previous prototype for ‘ioread64be_lo_hi’ [-Wmissing-prototypes]
170 | u64 ioread64be_lo_hi(const void __iomem *addr)
| ^~~~~~~~~~~~~~~~
../lib/iomap.c:178:5: warning: no previous prototype for ‘ioread64be_hi_lo’ [-Wmissing-prototypes]
178 | u64 ioread64be_hi_lo(const void __iomem *addr)
| ^~~~~~~~~~~~~~~~
../lib/iomap.c:264:6: warning: no previous prototype for ‘iowrite64_lo_hi’ [-Wmissing-prototypes]
264 | void iowrite64_lo_hi(u64 val, void __iomem *addr)
| ^~~~~~~~~~~~~~~
../lib/iomap.c:272:6: warning: no previous prototype for ‘iowrite64_hi_lo’ [-Wmissing-prototypes]
272 | void iowrite64_hi_lo(u64 val, void __iomem *addr)
| ^~~~~~~~~~~~~~~
../lib/iomap.c:280:6: warning: no previous prototype for ‘iowrite64be_lo_hi’ [-Wmissing-prototypes]
280 | void iowrite64be_lo_hi(u64 val, void __iomem *addr)
| ^~~~~~~~~~~~~~~~~
../lib/iomap.c:288:6: warning: no previous prototype for ‘iowrite64be_hi_lo’ [-Wmissing-prototypes]
288 | void iowrite64be_hi_lo(u64 val, void __iomem *addr)
| ^~~~~~~~~~~~~~~~~
[08:27:51] Starting KUnit Kernel (1/1)...
[08:27:51] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[08:27:51] ============ drm_test_pick_cmdline (2 subtests) ============
[08:27:51] [PASSED] drm_test_pick_cmdline_res_1920_1080_60
[08:27:51] =============== drm_test_pick_cmdline_named ===============
[08:27:51] [PASSED] NTSC
[08:27:51] [PASSED] NTSC-J
[08:27:51] [PASSED] PAL
[08:27:51] [PASSED] PAL-M
[08:27:51] =========== [PASSED] drm_test_pick_cmdline_named ===========
[08:27:51] ============== [PASSED] drm_test_pick_cmdline ==============
[08:27:51] ================== drm_buddy (7 subtests) ==================
[08:27:51] [PASSED] drm_test_buddy_alloc_limit
[08:27:51] [PASSED] drm_test_buddy_alloc_optimistic
[08:27:51] [PASSED] drm_test_buddy_alloc_pessimistic
[08:27:51] [PASSED] drm_test_buddy_alloc_pathological
[08:27:51] [PASSED] drm_test_buddy_alloc_contiguous
[08:27:51] [PASSED] drm_test_buddy_alloc_clear
[08:27:51] [PASSED] drm_test_buddy_alloc_range_bias
[08:27:51] ==================== [PASSED] drm_buddy ====================
[08:27:51] ============= drm_cmdline_parser (40 subtests) =============
[08:27:51] [PASSED] drm_test_cmdline_force_d_only
[08:27:51] [PASSED] drm_test_cmdline_force_D_only_dvi
[08:27:51] [PASSED] drm_test_cmdline_force_D_only_hdmi
[08:27:51] [PASSED] drm_test_cmdline_force_D_only_not_digital
[08:27:51] [PASSED] drm_test_cmdline_force_e_only
[08:27:51] [PASSED] drm_test_cmdline_res
[08:27:51] [PASSED] drm_test_cmdline_res_vesa
[08:27:51] [PASSED] drm_test_cmdline_res_vesa_rblank
[08:27:51] [PASSED] drm_test_cmdline_res_rblank
[08:27:51] [PASSED] drm_test_cmdline_res_bpp
[08:27:51] [PASSED] drm_test_cmdline_res_refresh
[08:27:51] [PASSED] drm_test_cmdline_res_bpp_refresh
[08:27:51] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced
[08:27:51] [PASSED] drm_test_cmdline_res_bpp_refresh_margins
[08:27:51] [PASSED] drm_test_cmdline_res_bpp_refresh_force_off
[08:27:51] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on
[08:27:51] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_analog
[08:27:51] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_digital
[08:27:51] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced_margins_force_on
[08:27:51] [PASSED] drm_test_cmdline_res_margins_force_on
[08:27:51] [PASSED] drm_test_cmdline_res_vesa_margins
[08:27:51] [PASSED] drm_test_cmdline_name
[08:27:51] [PASSED] drm_test_cmdline_name_bpp
[08:27:51] [PASSED] drm_test_cmdline_name_option
[08:27:51] [PASSED] drm_test_cmdline_name_bpp_option
[08:27:51] [PASSED] drm_test_cmdline_rotate_0
[08:27:51] [PASSED] drm_test_cmdline_rotate_90
[08:27:51] [PASSED] drm_test_cmdline_rotate_180
[08:27:51] [PASSED] drm_test_cmdline_rotate_270
[08:27:51] [PASSED] drm_test_cmdline_hmirror
[08:27:51] [PASSED] drm_test_cmdline_vmirror
[08:27:51] [PASSED] drm_test_cmdline_margin_options
[08:27:51] [PASSED] drm_test_cmdline_multiple_options
[08:27:51] [PASSED] drm_test_cmdline_bpp_extra_and_option
[08:27:51] [PASSED] drm_test_cmdline_extra_and_option
[08:27:51] [PASSED] drm_test_cmdline_freestanding_options
[08:27:51] [PASSED] drm_test_cmdline_freestanding_force_e_and_options
[08:27:51] [PASSED] drm_test_cmdline_panel_orientation
[08:27:51] ================ drm_test_cmdline_invalid =================
[08:27:51] [PASSED] margin_only
[08:27:51] [PASSED] interlace_only
[08:27:51] [PASSED] res_missing_x
[08:27:51] [PASSED] res_missing_y
[08:27:51] [PASSED] res_bad_y
[08:27:51] [PASSED] res_missing_y_bpp
[08:27:51] [PASSED] res_bad_bpp
[08:27:51] [PASSED] res_bad_refresh
[08:27:51] [PASSED] res_bpp_refresh_force_on_off
[08:27:51] [PASSED] res_invalid_mode
[08:27:51] [PASSED] res_bpp_wrong_place_mode
[08:27:51] [PASSED] name_bpp_refresh
[08:27:51] [PASSED] name_refresh
[08:27:51] [PASSED] name_refresh_wrong_mode
[08:27:51] [PASSED] name_refresh_invalid_mode
[08:27:51] [PASSED] rotate_multiple
[08:27:51] [PASSED] rotate_invalid_val
[08:27:51] [PASSED] rotate_truncated
[08:27:51] [PASSED] invalid_option
[08:27:51] [PASSED] invalid_tv_option
[08:27:51] [PASSED] truncated_tv_option
[08:27:51] ============ [PASSED] drm_test_cmdline_invalid =============
[08:27:51] =============== drm_test_cmdline_tv_options ===============
[08:27:51] [PASSED] NTSC
[08:27:51] [PASSED] NTSC_443
[08:27:51] [PASSED] NTSC_J
[08:27:51] [PASSED] PAL
[08:27:51] [PASSED] PAL_M
[08:27:51] [PASSED] PAL_N
[08:27:51] [PASSED] SECAM
[08:27:51] [PASSED] MONO_525
[08:27:51] [PASSED] MONO_625
[08:27:51] =========== [PASSED] drm_test_cmdline_tv_options ===========
[08:27:51] =============== [PASSED] drm_cmdline_parser ================
[08:27:51] ========== drmm_connector_hdmi_init (19 subtests) ==========
[08:27:51] [PASSED] drm_test_connector_hdmi_init_valid
[08:27:51] [PASSED] drm_test_connector_hdmi_init_bpc_8
[08:27:51] [PASSED] drm_test_connector_hdmi_init_bpc_10
[08:27:51] [PASSED] drm_test_connector_hdmi_init_bpc_12
[08:27:51] [PASSED] drm_test_connector_hdmi_init_bpc_invalid
[08:27:51] [PASSED] drm_test_connector_hdmi_init_bpc_null
[08:27:51] [PASSED] drm_test_connector_hdmi_init_formats_empty
[08:27:51] [PASSED] drm_test_connector_hdmi_init_formats_no_rgb
[08:27:51] [PASSED] drm_test_connector_hdmi_init_null_ddc
[08:27:51] [PASSED] drm_test_connector_hdmi_init_null_product
[08:27:51] [PASSED] drm_test_connector_hdmi_init_null_vendor
[08:27:51] [PASSED] drm_test_connector_hdmi_init_product_length_exact
[08:27:51] [PASSED] drm_test_connector_hdmi_init_product_length_too_long
[08:27:51] [PASSED] drm_test_connector_hdmi_init_product_valid
[08:27:51] [PASSED] drm_test_connector_hdmi_init_vendor_length_exact
[08:27:51] [PASSED] drm_test_connector_hdmi_init_vendor_length_too_long
[08:27:51] [PASSED] drm_test_connector_hdmi_init_vendor_valid
[08:27:51] ========= drm_test_connector_hdmi_init_type_valid =========
[08:27:51] [PASSED] HDMI-A
[08:27:51] [PASSED] HDMI-B
[08:27:51] ===== [PASSED] drm_test_connector_hdmi_init_type_valid =====
[08:27:51] ======== drm_test_connector_hdmi_init_type_invalid ========
[08:27:51] [PASSED] Unknown
[08:27:51] [PASSED] VGA
[08:27:51] [PASSED] DVI-I
[08:27:51] [PASSED] DVI-D
[08:27:51] [PASSED] DVI-A
[08:27:51] [PASSED] Composite
[08:27:51] [PASSED] SVIDEO
[08:27:51] [PASSED] LVDS
[08:27:51] [PASSED] Component
[08:27:51] [PASSED] DIN
[08:27:51] [PASSED] DP
[08:27:51] [PASSED] TV
[08:27:51] [PASSED] eDP
[08:27:51] [PASSED] Virtual
[08:27:51] [PASSED] DSI
[08:27:51] [PASSED] DPI
[08:27:51] [PASSED] Writeback
[08:27:51] [PASSED] SPI
[08:27:51] [PASSED] USB
[08:27:51] ==== [PASSED] drm_test_connector_hdmi_init_type_invalid ====
[08:27:51] ============ [PASSED] drmm_connector_hdmi_init =============
[08:27:51] ============= drmm_connector_init (3 subtests) =============
[08:27:51] [PASSED] drm_test_drmm_connector_init
[08:27:51] [PASSED] drm_test_drmm_connector_init_null_ddc
[08:27:51] ========= drm_test_drmm_connector_init_type_valid =========
[08:27:51] [PASSED] Unknown
[08:27:51] [PASSED] VGA
[08:27:51] [PASSED] DVI-I
[08:27:51] [PASSED] DVI-D
[08:27:51] [PASSED] DVI-A
[08:27:51] [PASSED] Composite
[08:27:51] [PASSED] SVIDEO
[08:27:51] [PASSED] LVDS
[08:27:51] [PASSED] Component
[08:27:51] [PASSED] DIN
[08:27:51] [PASSED] DP
[08:27:51] [PASSED] HDMI-A
[08:27:51] [PASSED] HDMI-B
[08:27:51] [PASSED] TV
[08:27:51] [PASSED] eDP
[08:27:51] [PASSED] Virtual
[08:27:51] [PASSED] DSI
[08:27:51] [PASSED] DPI
[08:27:51] [PASSED] Writeback
[08:27:51] [PASSED] SPI
[08:27:51] [PASSED] USB
[08:27:51] ===== [PASSED] drm_test_drmm_connector_init_type_valid =====
[08:27:51] =============== [PASSED] drmm_connector_init ===============
[08:27:51] = drm_connector_attach_broadcast_rgb_property (2 subtests) =
[08:27:51] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property
[08:27:51] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property_hdmi_connector
[08:27:51] === [PASSED] drm_connector_attach_broadcast_rgb_property ===
[08:27:51] ========== drm_get_tv_mode_from_name (2 subtests) ==========
[08:27:51] ========== drm_test_get_tv_mode_from_name_valid ===========
[08:27:51] [PASSED] NTSC
[08:27:51] [PASSED] NTSC-443
[08:27:51] [PASSED] NTSC-J
[08:27:51] [PASSED] PAL
[08:27:51] [PASSED] PAL-M
[08:27:51] [PASSED] PAL-N
[08:27:51] [PASSED] SECAM
[08:27:51] [PASSED] Mono
[08:27:51] ====== [PASSED] drm_test_get_tv_mode_from_name_valid =======
[08:27:51] [PASSED] drm_test_get_tv_mode_from_name_truncated
[08:27:51] ============ [PASSED] drm_get_tv_mode_from_name ============
[08:27:51] = drm_test_connector_hdmi_compute_mode_clock (12 subtests) =
[08:27:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb
[08:27:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc
[08:27:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc_vic_1
[08:27:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc
[08:27:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc_vic_1
[08:27:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_double
[08:27:51] = drm_test_connector_hdmi_compute_mode_clock_yuv420_valid =
[08:27:51] [PASSED] VIC 96
[08:27:51] [PASSED] VIC 97
[08:27:51] [PASSED] VIC 101
[08:27:51] [PASSED] VIC 102
[08:27:51] [PASSED] VIC 106
[08:27:51] [PASSED] VIC 107
[08:27:51] === [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_valid ===
[08:27:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_10_bpc
[08:27:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_12_bpc
[08:27:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_8_bpc
[08:27:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_10_bpc
[08:27:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_12_bpc
[08:27:51] === [PASSED] drm_test_connector_hdmi_compute_mode_clock ====
[08:27:51] == drm_hdmi_connector_get_broadcast_rgb_name (2 subtests) ==
[08:27:51] === drm_test_drm_hdmi_connector_get_broadcast_rgb_name ====
[08:27:51] [PASSED] Automatic
[08:27:51] [PASSED] Full
[08:27:51] [PASSED] Limited 16:235
[08:27:51] === [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name ===
[08:27:51] [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name_invalid
[08:27:51] ==== [PASSED] drm_hdmi_connector_get_broadcast_rgb_name ====
[08:27:51] == drm_hdmi_connector_get_output_format_name (2 subtests) ==
[08:27:51] === drm_test_drm_hdmi_connector_get_output_format_name ====
[08:27:51] [PASSED] RGB
[08:27:51] [PASSED] YUV 4:2:0
[08:27:51] [PASSED] YUV 4:2:2
[08:27:51] [PASSED] YUV 4:4:4
[08:27:51] === [PASSED] drm_test_drm_hdmi_connector_get_output_format_name ===
[08:27:51] [PASSED] drm_test_drm_hdmi_connector_get_output_format_name_invalid
[08:27:51] ==== [PASSED] drm_hdmi_connector_get_output_format_name ====
[08:27:51] ============= drm_damage_helper (21 subtests) ==============
[08:27:51] [PASSED] drm_test_damage_iter_no_damage
[08:27:51] [PASSED] drm_test_damage_iter_no_damage_fractional_src
[08:27:51] [PASSED] drm_test_damage_iter_no_damage_src_moved
[08:27:51] [PASSED] drm_test_damage_iter_no_damage_fractional_src_moved
[08:27:51] [PASSED] drm_test_damage_iter_no_damage_not_visible
[08:27:51] [PASSED] drm_test_damage_iter_no_damage_no_crtc
[08:27:51] [PASSED] drm_test_damage_iter_no_damage_no_fb
[08:27:51] [PASSED] drm_test_damage_iter_simple_damage
[08:27:51] [PASSED] drm_test_damage_iter_single_damage
[08:27:51] [PASSED] drm_test_damage_iter_single_damage_intersect_src
[08:27:51] [PASSED] drm_test_damage_iter_single_damage_outside_src
[08:27:51] [PASSED] drm_test_damage_iter_single_damage_fractional_src
[08:27:51] [PASSED] drm_test_damage_iter_single_damage_intersect_fractional_src
[08:27:51] [PASSED] drm_test_damage_iter_single_damage_outside_fractional_src
[08:27:51] [PASSED] drm_test_damage_iter_single_damage_src_moved
[08:27:51] [PASSED] drm_test_damage_iter_single_damage_fractional_src_moved
[08:27:51] [PASSED] drm_test_damage_iter_damage
[08:27:51] [PASSED] drm_test_damage_iter_damage_one_intersect
[08:27:51] [PASSED] drm_test_damage_iter_damage_one_outside
[08:27:51] [PASSED] drm_test_damage_iter_damage_src_moved
[08:27:51] [PASSED] drm_test_damage_iter_damage_not_visible
[08:27:51] ================ [PASSED] drm_damage_helper ================
[08:27:51] ============== drm_dp_mst_helper (3 subtests) ==============
[08:27:51] ============== drm_test_dp_mst_calc_pbn_mode ==============
[08:27:51] [PASSED] Clock 154000 BPP 30 DSC disabled
[08:27:51] [PASSED] Clock 234000 BPP 30 DSC disabled
[08:27:51] [PASSED] Clock 297000 BPP 24 DSC disabled
[08:27:51] [PASSED] Clock 332880 BPP 24 DSC enabled
[08:27:51] [PASSED] Clock 324540 BPP 24 DSC enabled
[08:27:51] ========== [PASSED] drm_test_dp_mst_calc_pbn_mode ==========
[08:27:51] ============== drm_test_dp_mst_calc_pbn_div ===============
[08:27:51] [PASSED] Link rate 2000000 lane count 4
[08:27:51] [PASSED] Link rate 2000000 lane count 2
[08:27:51] [PASSED] Link rate 2000000 lane count 1
[08:27:51] [PASSED] Link rate 1350000 lane count 4
[08:27:51] [PASSED] Link rate 1350000 lane count 2
[08:27:51] [PASSED] Link rate 1350000 lane count 1
[08:27:51] [PASSED] Link rate 1000000 lane count 4
[08:27:51] [PASSED] Link rate 1000000 lane count 2
[08:27:51] [PASSED] Link rate 1000000 lane count 1
[08:27:51] [PASSED] Link rate 810000 lane count 4
[08:27:51] [PASSED] Link rate 810000 lane count 2
[08:27:51] [PASSED] Link rate 810000 lane count 1
[08:27:51] [PASSED] Link rate 540000 lane count 4
[08:27:51] [PASSED] Link rate 540000 lane count 2
[08:27:51] [PASSED] Link rate 540000 lane count 1
[08:27:51] [PASSED] Link rate 270000 lane count 4
[08:27:51] [PASSED] Link rate 270000 lane count 2
[08:27:51] [PASSED] Link rate 270000 lane count 1
[08:27:51] [PASSED] Link rate 162000 lane count 4
[08:27:51] [PASSED] Link rate 162000 lane count 2
[08:27:51] [PASSED] Link rate 162000 lane count 1
[08:27:51] ========== [PASSED] drm_test_dp_mst_calc_pbn_div ===========
[08:27:51] ========= drm_test_dp_mst_sideband_msg_req_decode =========
[08:27:51] [PASSED] DP_ENUM_PATH_RESOURCES with port number
[08:27:51] [PASSED] DP_POWER_UP_PHY with port number
[08:27:51] [PASSED] DP_POWER_DOWN_PHY with port number
[08:27:51] [PASSED] DP_ALLOCATE_PAYLOAD with SDP stream sinks
[08:27:51] [PASSED] DP_ALLOCATE_PAYLOAD with port number
[08:27:51] [PASSED] DP_ALLOCATE_PAYLOAD with VCPI
[08:27:51] [PASSED] DP_ALLOCATE_PAYLOAD with PBN
[08:27:51] [PASSED] DP_QUERY_PAYLOAD with port number
[08:27:51] [PASSED] DP_QUERY_PAYLOAD with VCPI
[08:27:51] [PASSED] DP_REMOTE_DPCD_READ with port number
[08:27:51] [PASSED] DP_REMOTE_DPCD_READ with DPCD address
[08:27:51] [PASSED] DP_REMOTE_DPCD_READ with max number of bytes
[08:27:51] [PASSED] DP_REMOTE_DPCD_WRITE with port number
[08:27:51] [PASSED] DP_REMOTE_DPCD_WRITE with DPCD address
[08:27:51] [PASSED] DP_REMOTE_DPCD_WRITE with data array
[08:27:51] [PASSED] DP_REMOTE_I2C_READ with port number
[08:27:51] [PASSED] DP_REMOTE_I2C_READ with I2C device ID
[08:27:51] [PASSED] DP_REMOTE_I2C_READ with transactions array
[08:27:51] [PASSED] DP_REMOTE_I2C_WRITE with port number
[08:27:51] [PASSED] DP_REMOTE_I2C_WRITE with I2C device ID
[08:27:51] [PASSED] DP_REMOTE_I2C_WRITE with data array
[08:27:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream ID
[08:27:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with client ID
[08:27:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream event
[08:27:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with valid stream event
[08:27:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream behavior
[08:27:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with a valid stream behavior
[08:27:51] ===== [PASSED] drm_test_dp_mst_sideband_msg_req_decode =====
[08:27:51] ================ [PASSED] drm_dp_mst_helper ================
[08:27:51] ================== drm_exec (7 subtests) ===================
[08:27:51] [PASSED] sanitycheck
[08:27:51] [PASSED] test_lock
[08:27:51] [PASSED] test_lock_unlock
[08:27:51] [PASSED] test_duplicates
[08:27:51] [PASSED] test_prepare
[08:27:51] [PASSED] test_prepare_array
[08:27:51] [PASSED] test_multiple_loops
[08:27:51] ==================== [PASSED] drm_exec =====================
[08:27:51] =========== drm_format_helper_test (17 subtests) ===========
[08:27:51] ============== drm_test_fb_xrgb8888_to_gray8 ==============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ========== [PASSED] drm_test_fb_xrgb8888_to_gray8 ==========
[08:27:51] ============= drm_test_fb_xrgb8888_to_rgb332 ==============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb332 ==========
[08:27:51] ============= drm_test_fb_xrgb8888_to_rgb565 ==============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb565 ==========
[08:27:51] ============ drm_test_fb_xrgb8888_to_xrgb1555 =============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ======== [PASSED] drm_test_fb_xrgb8888_to_xrgb1555 =========
[08:27:51] ============ drm_test_fb_xrgb8888_to_argb1555 =============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ======== [PASSED] drm_test_fb_xrgb8888_to_argb1555 =========
[08:27:51] ============ drm_test_fb_xrgb8888_to_rgba5551 =============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ======== [PASSED] drm_test_fb_xrgb8888_to_rgba5551 =========
[08:27:51] ============= drm_test_fb_xrgb8888_to_rgb888 ==============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb888 ==========
[08:27:51] ============ drm_test_fb_xrgb8888_to_argb8888 =============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ======== [PASSED] drm_test_fb_xrgb8888_to_argb8888 =========
[08:27:51] =========== drm_test_fb_xrgb8888_to_xrgb2101010 ===========
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ======= [PASSED] drm_test_fb_xrgb8888_to_xrgb2101010 =======
[08:27:51] =========== drm_test_fb_xrgb8888_to_argb2101010 ===========
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ======= [PASSED] drm_test_fb_xrgb8888_to_argb2101010 =======
[08:27:51] ============== drm_test_fb_xrgb8888_to_mono ===============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ========== [PASSED] drm_test_fb_xrgb8888_to_mono ===========
[08:27:51] ==================== drm_test_fb_swab =====================
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ================ [PASSED] drm_test_fb_swab =================
[08:27:51] ============ drm_test_fb_xrgb8888_to_xbgr8888 =============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ======== [PASSED] drm_test_fb_xrgb8888_to_xbgr8888 =========
[08:27:51] ============ drm_test_fb_xrgb8888_to_abgr8888 =============
[08:27:51] [PASSED] single_pixel_source_buffer
[08:27:51] [PASSED] single_pixel_clip_rectangle
[08:27:51] [PASSED] well_known_colors
[08:27:51] [PASSED] destination_pitch
[08:27:51] ======== [PASSED] drm_test_fb_xrgb8888_to_abgr8888 =========
[08:27:51] ================= drm_test_fb_clip_offset =================
[08:27:51] [PASSED] pass through
[08:27:51] [PASSED] horizontal offset
[08:27:51] [PASSED] vertical offset
[08:27:51] [PASSED] horizontal and vertical offset
[08:27:51] [PASSED] horizontal offset (custom pitch)
[08:27:51] [PASSED] vertical offset (custom pitch)
[08:27:51] [PASSED] horizontal and vertical offset (custom pitch)
[08:27:51] ============= [PASSED] drm_test_fb_clip_offset =============
[08:27:51] ============== drm_test_fb_build_fourcc_list ==============
[08:27:51] [PASSED] no native formats
[08:27:51] [PASSED] XRGB8888 as native format
[08:27:51] [PASSED] remove duplicates
[08:27:51] [PASSED] convert alpha formats
[08:27:51] [PASSED] random formats
[08:27:51] ========== [PASSED] drm_test_fb_build_fourcc_list ==========
[08:27:51] =================== drm_test_fb_memcpy ====================
[08:27:51] [PASSED] single_pixel_source_buffer: XR24 little-endian (0x34325258)
[08:27:51] [PASSED] single_pixel_source_buffer: XRA8 little-endian (0x38415258)
[08:27:51] [PASSED] single_pixel_source_buffer: YU24 little-endian (0x34325559)
[08:27:51] [PASSED] single_pixel_clip_rectangle: XB24 little-endian (0x34324258)
[08:27:51] [PASSED] single_pixel_clip_rectangle: XRA8 little-endian (0x38415258)
[08:27:51] [PASSED] single_pixel_clip_rectangle: YU24 little-endian (0x34325559)
[08:27:51] [PASSED] well_known_colors: XB24 little-endian (0x34324258)
[08:27:51] [PASSED] well_known_colors: XRA8 little-endian (0x38415258)
[08:27:51] [PASSED] well_known_colors: YU24 little-endian (0x34325559)
[08:27:51] [PASSED] destination_pitch: XB24 little-endian (0x34324258)
[08:27:51] [PASSED] destination_pitch: XRA8 little-endian (0x38415258)
[08:27:51] [PASSED] destination_pitch: YU24 little-endian (0x34325559)
[08:27:51] =============== [PASSED] drm_test_fb_memcpy ================
[08:27:51] ============= [PASSED] drm_format_helper_test ==============
[08:27:51] ================= drm_format (18 subtests) =================
[08:27:51] [PASSED] drm_test_format_block_width_invalid
[08:27:51] [PASSED] drm_test_format_block_width_one_plane
[08:27:51] [PASSED] drm_test_format_block_width_two_plane
[08:27:51] [PASSED] drm_test_format_block_width_three_plane
[08:27:51] [PASSED] drm_test_format_block_width_tiled
[08:27:51] [PASSED] drm_test_format_block_height_invalid
[08:27:51] [PASSED] drm_test_format_block_height_one_plane
[08:27:51] [PASSED] drm_test_format_block_height_two_plane
[08:27:51] [PASSED] drm_test_format_block_height_three_plane
[08:27:51] [PASSED] drm_test_format_block_height_tiled
[08:27:51] [PASSED] drm_test_format_min_pitch_invalid
[08:27:51] [PASSED] drm_test_format_min_pitch_one_plane_8bpp
[08:27:51] [PASSED] drm_test_format_min_pitch_one_plane_16bpp
[08:27:51] [PASSED] drm_test_format_min_pitch_one_plane_24bpp
[08:27:51] [PASSED] drm_test_format_min_pitch_one_plane_32bpp
[08:27:51] [PASSED] drm_test_format_min_pitch_two_plane
[08:27:51] [PASSED] drm_test_format_min_pitch_three_plane_8bpp
[08:27:51] [PASSED] drm_test_format_min_pitch_tiled
[08:27:51] =================== [PASSED] drm_format ====================
[08:27:51] ============== drm_framebuffer (10 subtests) ===============
[08:27:51] ========== drm_test_framebuffer_check_src_coords ==========
[08:27:51] [PASSED] Success: source fits into fb
[08:27:51] [PASSED] Fail: overflowing fb with x-axis coordinate
[08:27:51] [PASSED] Fail: overflowing fb with y-axis coordinate
[08:27:51] [PASSED] Fail: overflowing fb with source width
[08:27:51] [PASSED] Fail: overflowing fb with source height
[08:27:51] ====== [PASSED] drm_test_framebuffer_check_src_coords ======
[08:27:51] [PASSED] drm_test_framebuffer_cleanup
[08:27:51] =============== drm_test_framebuffer_create ===============
[08:27:51] [PASSED] ABGR8888 normal sizes
[08:27:51] [PASSED] ABGR8888 max sizes
[08:27:51] [PASSED] ABGR8888 pitch greater than min required
[08:27:51] [PASSED] ABGR8888 pitch less than min required
[08:27:51] [PASSED] ABGR8888 Invalid width
[08:27:51] [PASSED] ABGR8888 Invalid buffer handle
[08:27:51] [PASSED] No pixel format
[08:27:51] [PASSED] ABGR8888 Width 0
[08:27:51] [PASSED] ABGR8888 Height 0
[08:27:51] [PASSED] ABGR8888 Out of bound height * pitch combination
[08:27:51] [PASSED] ABGR8888 Large buffer offset
[08:27:51] [PASSED] ABGR8888 Buffer offset for inexistent plane
[08:27:51] [PASSED] ABGR8888 Invalid flag
[08:27:51] [PASSED] ABGR8888 Set DRM_MODE_FB_MODIFIERS without modifiers
[08:27:51] [PASSED] ABGR8888 Valid buffer modifier
[08:27:51] [PASSED] ABGR8888 Invalid buffer modifier(DRM_FORMAT_MOD_SAMSUNG_64_32_TILE)
[08:27:51] [PASSED] ABGR8888 Extra pitches without DRM_MODE_FB_MODIFIERS
[08:27:51] [PASSED] ABGR8888 Extra pitches with DRM_MODE_FB_MODIFIERS
[08:27:51] [PASSED] NV12 Normal sizes
[08:27:51] [PASSED] NV12 Max sizes
[08:27:51] [PASSED] NV12 Invalid pitch
[08:27:51] [PASSED] NV12 Invalid modifier/missing DRM_MODE_FB_MODIFIERS flag
[08:27:51] [PASSED] NV12 different modifier per-plane
[08:27:51] [PASSED] NV12 with DRM_FORMAT_MOD_SAMSUNG_64_32_TILE
[08:27:51] [PASSED] NV12 Valid modifiers without DRM_MODE_FB_MODIFIERS
[08:27:51] [PASSED] NV12 Modifier for inexistent plane
[08:27:51] [PASSED] NV12 Handle for inexistent plane
[08:27:51] [PASSED] NV12 Handle for inexistent plane without DRM_MODE_FB_MODIFIERS
[08:27:51] [PASSED] YVU420 DRM_MODE_FB_MODIFIERS set without modifier
[08:27:51] [PASSED] YVU420 Normal sizes
[08:27:51] [PASSED] YVU420 Max sizes
[08:27:51] [PASSED] YVU420 Invalid pitch
[08:27:51] [PASSED] YVU420 Different pitches
[08:27:51] [PASSED] YVU420 Different buffer offsets/pitches
[08:27:51] [PASSED] YVU420 Modifier set just for plane 0, without DRM_MODE_FB_MODIFIERS
[08:27:51] [PASSED] YVU420 Modifier set just for planes 0, 1, without DRM_MODE_FB_MODIFIERS
[08:27:51] [PASSED] YVU420 Modifier set just for plane 0, 1, with DRM_MODE_FB_MODIFIERS
[08:27:51] [PASSED] YVU420 Valid modifier
[08:27:51] [PASSED] YVU420 Different modifiers per plane
[08:27:51] [PASSED] YVU420 Modifier for inexistent plane
[08:27:51] [PASSED] YUV420_10BIT Invalid modifier(DRM_FORMAT_MOD_LINEAR)
[08:27:51] [PASSED] X0L2 Normal sizes
[08:27:51] [PASSED] X0L2 Max sizes
[08:27:51] [PASSED] X0L2 Invalid pitch
[08:27:51] [PASSED] X0L2 Pitch greater than minimum required
[08:27:51] [PASSED] X0L2 Handle for inexistent plane
[08:27:51] [PASSED] X0L2 Offset for inexistent plane, without DRM_MODE_FB_MODIFIERS set
[08:27:51] [PASSED] X0L2 Modifier without DRM_MODE_FB_MODIFIERS set
[08:27:51] [PASSED] X0L2 Valid modifier
[08:27:51] [PASSED] X0L2 Modifier for inexistent plane
[08:27:51] =========== [PASSED] drm_test_framebuffer_create ===========
[08:27:51] [PASSED] drm_test_framebuffer_free
[08:27:51] [PASSED] drm_test_framebuffer_init
[08:27:51] [PASSED] drm_test_framebuffer_init_bad_format
[08:27:51] [PASSED] drm_test_framebuffer_init_dev_mismatch
[08:27:51] [PASSED] drm_test_framebuffer_lookup
[08:27:51] [PASSED] drm_test_framebuffer_lookup_inexistent
[08:27:51] [PASSED] drm_test_framebuffer_modifiers_not_supported
[08:27:51] ================= [PASSED] drm_framebuffer =================
[08:27:51] ================ drm_gem_shmem (8 subtests) ================
[08:27:51] [PASSED] drm_gem_shmem_test_obj_create
[08:27:51] [PASSED] drm_gem_shmem_test_obj_create_private
[08:27:51] [PASSED] drm_gem_shmem_test_pin_pages
[08:27:51] [PASSED] drm_gem_shmem_test_vmap
[08:27:51] [PASSED] drm_gem_shmem_test_get_pages_sgt
[08:27:51] [PASSED] drm_gem_shmem_test_get_sg_table
[08:27:51] [PASSED] drm_gem_shmem_test_madvise
[08:27:51] [PASSED] drm_gem_shmem_test_purge
[08:27:51] ================== [PASSED] drm_gem_shmem ==================
[08:27:51] === drm_atomic_helper_connector_hdmi_check (22 subtests) ===
[08:27:51] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode
[08:27:51] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode_vic_1
[08:27:51] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode
[08:27:51] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode_vic_1
[08:27:51] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode
[08:27:51] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode_vic_1
[08:27:51] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_changed
[08:27:51] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_not_changed
[08:27:51] [PASSED] drm_test_check_hdmi_funcs_reject_rate
[08:27:51] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback
[08:27:51] [PASSED] drm_test_check_max_tmds_rate_format_fallback
[08:27:51] [PASSED] drm_test_check_output_bpc_crtc_mode_changed
[08:27:51] [PASSED] drm_test_check_output_bpc_crtc_mode_not_changed
[08:27:51] [PASSED] drm_test_check_output_bpc_dvi
[08:27:51] [PASSED] drm_test_check_output_bpc_format_vic_1
[08:27:51] [PASSED] drm_test_check_output_bpc_format_display_8bpc_only
[08:27:51] [PASSED] drm_test_check_output_bpc_format_display_rgb_only
[08:27:51] [PASSED] drm_test_check_output_bpc_format_driver_8bpc_only
[08:27:51] [PASSED] drm_test_check_output_bpc_format_driver_rgb_only
[08:27:51] [PASSED] drm_test_check_tmds_char_rate_rgb_8bpc
[08:27:51] [PASSED] drm_test_check_tmds_char_rate_rgb_10bpc
[08:27:51] [PASSED] drm_test_check_tmds_char_rate_rgb_12bpc
[08:27:51] ===== [PASSED] drm_atomic_helper_connector_hdmi_check ======
[08:27:51] === drm_atomic_helper_connector_hdmi_reset (6 subtests) ====
[08:27:51] [PASSED] drm_test_check_broadcast_rgb_value
[08:27:51] [PASSED] drm_test_check_bpc_8_value
[08:27:51] [PASSED] drm_test_check_bpc_10_value
[08:27:51] [PASSED] drm_test_check_bpc_12_value
[08:27:51] [PASSED] drm_test_check_format_value
[08:27:51] [PASSED] drm_test_check_tmds_char_value
[08:27:51] ===== [PASSED] drm_atomic_helper_connector_hdmi_reset ======
[08:27:51] ================= drm_managed (2 subtests) =================
[08:27:51] [PASSED] drm_test_managed_release_action
[08:27:51] [PASSED] drm_test_managed_run_action
[08:27:51] =================== [PASSED] drm_managed ===================
[08:27:51] =================== drm_mm (6 subtests) ====================
[08:27:51] [PASSED] drm_test_mm_init
[08:27:51] [PASSED] drm_test_mm_debug
[08:27:51] [PASSED] drm_test_mm_align32
[08:27:51] [PASSED] drm_test_mm_align64
[08:27:51] [PASSED] drm_test_mm_lowest
[08:27:51] [PASSED] drm_test_mm_highest
[08:27:51] ===================== [PASSED] drm_mm ======================
[08:27:51] ============= drm_modes_analog_tv (5 subtests) =============
stty: 'standard input': Inappropriate ioctl for device
[08:27:51] [PASSED] drm_test_modes_analog_tv_mono_576i
[08:27:51] [PASSED] drm_test_modes_analog_tv_ntsc_480i
[08:27:51] [PASSED] drm_test_modes_analog_tv_ntsc_480i_inlined
[08:27:51] [PASSED] drm_test_modes_analog_tv_pal_576i
[08:27:51] [PASSED] drm_test_modes_analog_tv_pal_576i_inlined
[08:27:51] =============== [PASSED] drm_modes_analog_tv ===============
[08:27:51] ============== drm_plane_helper (2 subtests) ===============
[08:27:51] =============== drm_test_check_plane_state ================
[08:27:51] [PASSED] clipping_simple
[08:27:51] [PASSED] clipping_rotate_reflect
[08:27:51] [PASSED] positioning_simple
[08:27:51] [PASSED] upscaling
[08:27:51] [PASSED] downscaling
[08:27:51] [PASSED] rounding1
[08:27:51] [PASSED] rounding2
[08:27:51] [PASSED] rounding3
[08:27:51] [PASSED] rounding4
[08:27:51] =========== [PASSED] drm_test_check_plane_state ============
[08:27:51] =========== drm_test_check_invalid_plane_state ============
[08:27:51] [PASSED] positioning_invalid
[08:27:51] [PASSED] upscaling_invalid
[08:27:51] [PASSED] downscaling_invalid
[08:27:51] ======= [PASSED] drm_test_check_invalid_plane_state ========
[08:27:51] ================ [PASSED] drm_plane_helper =================
[08:27:51] ====== drm_connector_helper_tv_get_modes (1 subtest) =======
[08:27:51] ====== drm_test_connector_helper_tv_get_modes_check =======
[08:27:51] [PASSED] None
[08:27:51] [PASSED] PAL
[08:27:51] [PASSED] NTSC
[08:27:51] [PASSED] Both, NTSC Default
[08:27:51] [PASSED] Both, PAL Default
[08:27:51] [PASSED] Both, NTSC Default, with PAL on command-line
[08:27:51] [PASSED] Both, PAL Default, with NTSC on command-line
[08:27:51] == [PASSED] drm_test_connector_helper_tv_get_modes_check ===
[08:27:51] ======== [PASSED] drm_connector_helper_tv_get_modes ========
[08:27:51] ================== drm_rect (9 subtests) ===================
[08:27:51] [PASSED] drm_test_rect_clip_scaled_div_by_zero
[08:27:51] [PASSED] drm_test_rect_clip_scaled_not_clipped
[08:27:51] [PASSED] drm_test_rect_clip_scaled_clipped
[08:27:51] [PASSED] drm_test_rect_clip_scaled_signed_vs_unsigned
[08:27:51] ================= drm_test_rect_intersect =================
[08:27:51] [PASSED] top-left x bottom-right: 2x2+1+1 x 2x2+0+0
[08:27:51] [PASSED] top-right x bottom-left: 2x2+0+0 x 2x2+1-1
[08:27:51] [PASSED] bottom-left x top-right: 2x2+1-1 x 2x2+0+0
[08:27:51] [PASSED] bottom-right x top-left: 2x2+0+0 x 2x2+1+1
[08:27:51] [PASSED] right x left: 2x1+0+0 x 3x1+1+0
[08:27:51] [PASSED] left x right: 3x1+1+0 x 2x1+0+0
[08:27:51] [PASSED] up x bottom: 1x2+0+0 x 1x3+0-1
[08:27:51] [PASSED] bottom x up: 1x3+0-1 x 1x2+0+0
[08:27:51] [PASSED] touching corner: 1x1+0+0 x 2x2+1+1
[08:27:51] [PASSED] touching side: 1x1+0+0 x 1x1+1+0
[08:27:51] [PASSED] equal rects: 2x2+0+0 x 2x2+0+0
[08:27:51] [PASSED] inside another: 2x2+0+0 x 1x1+1+1
[08:27:51] [PASSED] far away: 1x1+0+0 x 1x1+3+6
[08:27:51] [PASSED] points intersecting: 0x0+5+10 x 0x0+5+10
[08:27:51] [PASSED] points not intersecting: 0x0+0+0 x 0x0+5+10
[08:27:51] ============= [PASSED] drm_test_rect_intersect =============
[08:27:51] ================ drm_test_rect_calc_hscale ================
[08:27:51] [PASSED] normal use
[08:27:51] [PASSED] out of max range
[08:27:51] [PASSED] out of min range
[08:27:51] [PASSED] zero dst
[08:27:51] [PASSED] negative src
[08:27:51] [PASSED] negative dst
[08:27:51] ============ [PASSED] drm_test_rect_calc_hscale ============
[08:27:51] ================ drm_test_rect_calc_vscale ================
[08:27:51] [PASSED] normal use
[08:27:51] [PASSED] out of max range
[08:27:51] [PASSED] out of min range
[08:27:51] [PASSED] zero dst
[08:27:51] [PASSED] negative src
[08:27:51] [PASSED] negative dst
[08:27:51] ============ [PASSED] drm_test_rect_calc_vscale ============
[08:27:51] ================== drm_test_rect_rotate ===================
[08:27:51] [PASSED] reflect-x
[08:27:51] [PASSED] reflect-y
[08:27:51] [PASSED] rotate-0
[08:27:51] [PASSED] rotate-90
[08:27:51] [PASSED] rotate-180
[08:27:51] [PASSED] rotate-270
[08:27:51] ============== [PASSED] drm_test_rect_rotate ===============
[08:27:51] ================ drm_test_rect_rotate_inv =================
[08:27:51] [PASSED] reflect-x
[08:27:51] [PASSED] reflect-y
[08:27:51] [PASSED] rotate-0
[08:27:51] [PASSED] rotate-90
[08:27:51] [PASSED] rotate-180
[08:27:51] [PASSED] rotate-270
[08:27:51] ============ [PASSED] drm_test_rect_rotate_inv =============
[08:27:51] ==================== [PASSED] drm_rect =====================
[08:27:51] ============================================================
[08:27:51] Testing complete. Ran 531 tests: passed: 531
[08:27:51] Elapsed time: 23.650s total, 1.384s configuring, 22.093s building, 0.171s running
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/ttm/tests/.kunitconfig
[08:27:52] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[08:27:53] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make ARCH=um O=.kunit --jobs=48
[08:28:00] Starting KUnit Kernel (1/1)...
[08:28:00] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[08:28:00] ================= ttm_device (5 subtests) ==================
[08:28:00] [PASSED] ttm_device_init_basic
[08:28:00] [PASSED] ttm_device_init_multiple
[08:28:00] [PASSED] ttm_device_fini_basic
[08:28:00] [PASSED] ttm_device_init_no_vma_man
[08:28:00] ================== ttm_device_init_pools ==================
[08:28:00] [PASSED] No DMA allocations, no DMA32 required
[08:28:00] [PASSED] DMA allocations, DMA32 required
[08:28:00] [PASSED] No DMA allocations, DMA32 required
[08:28:00] [PASSED] DMA allocations, no DMA32 required
[08:28:00] ============== [PASSED] ttm_device_init_pools ==============
[08:28:00] =================== [PASSED] ttm_device ====================
[08:28:00] ================== ttm_pool (8 subtests) ===================
[08:28:00] ================== ttm_pool_alloc_basic ===================
[08:28:00] [PASSED] One page
[08:28:00] [PASSED] More than one page
[08:28:00] [PASSED] Above the allocation limit
[08:28:00] [PASSED] One page, with coherent DMA mappings enabled
[08:28:00] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[08:28:00] ============== [PASSED] ttm_pool_alloc_basic ===============
[08:28:00] ============== ttm_pool_alloc_basic_dma_addr ==============
[08:28:00] [PASSED] One page
[08:28:00] [PASSED] More than one page
[08:28:00] [PASSED] Above the allocation limit
[08:28:00] [PASSED] One page, with coherent DMA mappings enabled
[08:28:00] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[08:28:00] ========== [PASSED] ttm_pool_alloc_basic_dma_addr ==========
[08:28:00] [PASSED] ttm_pool_alloc_order_caching_match
[08:28:00] [PASSED] ttm_pool_alloc_caching_mismatch
[08:28:00] [PASSED] ttm_pool_alloc_order_mismatch
[08:28:00] [PASSED] ttm_pool_free_dma_alloc
[08:28:00] [PASSED] ttm_pool_free_no_dma_alloc
[08:28:00] [PASSED] ttm_pool_fini_basic
[08:28:00] ==================== [PASSED] ttm_pool =====================
[08:28:00] ================ ttm_resource (8 subtests) =================
[08:28:00] ================= ttm_resource_init_basic =================
[08:28:00] [PASSED] Init resource in TTM_PL_SYSTEM
[08:28:00] [PASSED] Init resource in TTM_PL_VRAM
[08:28:00] [PASSED] Init resource in a private placement
[08:28:00] [PASSED] Init resource in TTM_PL_SYSTEM, set placement flags
[08:28:00] ============= [PASSED] ttm_resource_init_basic =============
[08:28:00] [PASSED] ttm_resource_init_pinned
[08:28:00] [PASSED] ttm_resource_fini_basic
[08:28:00] [PASSED] ttm_resource_manager_init_basic
[08:28:00] [PASSED] ttm_resource_manager_usage_basic
[08:28:00] [PASSED] ttm_resource_manager_set_used_basic
[08:28:00] [PASSED] ttm_sys_man_alloc_basic
[08:28:00] [PASSED] ttm_sys_man_free_basic
[08:28:00] ================== [PASSED] ttm_resource ===================
[08:28:00] =================== ttm_tt (15 subtests) ===================
[08:28:00] ==================== ttm_tt_init_basic ====================
[08:28:00] [PASSED] Page-aligned size
[08:28:00] [PASSED] Extra pages requested
[08:28:00] ================ [PASSED] ttm_tt_init_basic ================
[08:28:00] [PASSED] ttm_tt_init_misaligned
[08:28:00] [PASSED] ttm_tt_fini_basic
[08:28:00] [PASSED] ttm_tt_fini_sg
[08:28:00] [PASSED] ttm_tt_fini_shmem
[08:28:00] [PASSED] ttm_tt_create_basic
[08:28:00] [PASSED] ttm_tt_create_invalid_bo_type
[08:28:00] [PASSED] ttm_tt_create_ttm_exists
[08:28:00] [PASSED] ttm_tt_create_failed
[08:28:00] [PASSED] ttm_tt_destroy_basic
[08:28:00] [PASSED] ttm_tt_populate_null_ttm
[08:28:00] [PASSED] ttm_tt_populate_populated_ttm
[08:28:00] [PASSED] ttm_tt_unpopulate_basic
[08:28:00] [PASSED] ttm_tt_unpopulate_empty_ttm
[08:28:00] [PASSED] ttm_tt_swapin_basic
[08:28:00] ===================== [PASSED] ttm_tt ======================
[08:28:00] =================== ttm_bo (14 subtests) ===================
[08:28:00] =========== ttm_bo_reserve_optimistic_no_ticket ===========
[08:28:00] [PASSED] Cannot be interrupted and sleeps
[08:28:00] [PASSED] Cannot be interrupted, locks straight away
[08:28:00] [PASSED] Can be interrupted, sleeps
[08:28:00] ======= [PASSED] ttm_bo_reserve_optimistic_no_ticket =======
[08:28:00] [PASSED] ttm_bo_reserve_locked_no_sleep
[08:28:00] [PASSED] ttm_bo_reserve_no_wait_ticket
[08:28:00] [PASSED] ttm_bo_reserve_double_resv
[08:28:00] [PASSED] ttm_bo_reserve_interrupted
[08:28:00] [PASSED] ttm_bo_reserve_deadlock
[08:28:00] [PASSED] ttm_bo_unreserve_basic
[08:28:00] [PASSED] ttm_bo_unreserve_pinned
[08:28:00] [PASSED] ttm_bo_unreserve_bulk
[08:28:00] [PASSED] ttm_bo_put_basic
[08:28:00] [PASSED] ttm_bo_put_shared_resv
[08:28:00] [PASSED] ttm_bo_pin_basic
[08:28:00] [PASSED] ttm_bo_pin_unpin_resource
[08:28:00] [PASSED] ttm_bo_multiple_pin_one_unpin
[08:28:00] ===================== [PASSED] ttm_bo ======================
[08:28:00] ============== ttm_bo_validate (22 subtests) ===============
[08:28:00] ============== ttm_bo_init_reserved_sys_man ===============
[08:28:00] [PASSED] Buffer object for userspace
[08:28:00] [PASSED] Kernel buffer object
[08:28:00] [PASSED] Shared buffer object
[08:28:00] ========== [PASSED] ttm_bo_init_reserved_sys_man ===========
[08:28:00] ============== ttm_bo_init_reserved_mock_man ==============
[08:28:00] [PASSED] Buffer object for userspace
[08:28:00] [PASSED] Kernel buffer object
[08:28:00] [PASSED] Shared buffer object
[08:28:00] ========== [PASSED] ttm_bo_init_reserved_mock_man ==========
[08:28:00] [PASSED] ttm_bo_init_reserved_resv
[08:28:00] ================== ttm_bo_validate_basic ==================
[08:28:00] [PASSED] Buffer object for userspace
[08:28:00] [PASSED] Kernel buffer object
[08:28:00] [PASSED] Shared buffer object
[08:28:00] ============== [PASSED] ttm_bo_validate_basic ==============
[08:28:00] [PASSED] ttm_bo_validate_invalid_placement
[08:28:00] ============= ttm_bo_validate_same_placement ==============
[08:28:00] [PASSED] System manager
[08:28:00] [PASSED] VRAM manager
[08:28:00] ========= [PASSED] ttm_bo_validate_same_placement ==========
[08:28:00] [PASSED] ttm_bo_validate_failed_alloc
[08:28:00] [PASSED] ttm_bo_validate_pinned
[08:28:00] [PASSED] ttm_bo_validate_busy_placement
[08:28:00] ================ ttm_bo_validate_multihop =================
[08:28:00] [PASSED] Buffer object for userspace
[08:28:00] [PASSED] Kernel buffer object
[08:28:00] [PASSED] Shared buffer object
[08:28:00] ============ [PASSED] ttm_bo_validate_multihop =============
[08:28:00] ========== ttm_bo_validate_no_placement_signaled ==========
[08:28:00] [PASSED] Buffer object in system domain, no page vector
[08:28:00] [PASSED] Buffer object in system domain with an existing page vector
[08:28:00] ====== [PASSED] ttm_bo_validate_no_placement_signaled ======
[08:28:00] ======== ttm_bo_validate_no_placement_not_signaled ========
[08:28:00] [PASSED] Buffer object for userspace
[08:28:00] [PASSED] Kernel buffer object
[08:28:00] [PASSED] Shared buffer object
[08:28:00] ==== [PASSED] ttm_bo_validate_no_placement_not_signaled ====
[08:28:00] [PASSED] ttm_bo_validate_move_fence_signaled
[08:28:00] ========= ttm_bo_validate_move_fence_not_signaled =========
[08:28:00] [PASSED] Waits for GPU
[08:28:00] [PASSED] Tries to lock straight away
[08:28:01] ===== [PASSED] ttm_bo_validate_move_fence_not_signaled =====
[08:28:01] [PASSED] ttm_bo_validate_swapout
[08:28:01] [PASSED] ttm_bo_validate_happy_evict
[08:28:01] [PASSED] ttm_bo_validate_all_pinned_evict
[08:28:01] [PASSED] ttm_bo_validate_allowed_only_evict
[08:28:01] [PASSED] ttm_bo_validate_deleted_evict
[08:28:01] [PASSED] ttm_bo_validate_busy_domain_evict
[08:28:01] [PASSED] ttm_bo_validate_evict_gutting
[08:28:01] [PASSED] ttm_bo_validate_recrusive_evict
stty: 'standard input': Inappropriate ioctl for device
[08:28:01] ================= [PASSED] ttm_bo_validate =================
[08:28:01] ============================================================
[08:28:01] Testing complete. Ran 102 tests: passed: 102
[08:28:01] Elapsed time: 9.224s total, 1.427s configuring, 7.080s building, 0.614s running
+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel
^ permalink raw reply [flat|nested] 10+ messages in thread* ✗ CI.Build: failure for gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
2024-09-27 0:01 [PATCH] gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state Alessandro Zanni
` (3 preceding siblings ...)
2024-09-27 8:28 ` ✓ CI.KUnit: success " Patchwork
@ 2024-09-27 8:32 ` Patchwork
4 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2024-09-27 8:32 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-xe
== Series Details ==
Series: gpu: drm: i915: display: Avoid null values intel_plane_atomic_check_with_state
URL : https://patchwork.freedesktop.org/series/139198/
State : failure
== Summary ==
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml2_dc_resource_mgmt.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml2_mall_phantom.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml_display_rq_dlg_calc.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_top/dml_top.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_top/dml_top_mcache.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_top/dml2_top_optimization.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/inc/dml2_debug.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_core/dml2_core_factory.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_dpmm/dml2_dpmm_dcn4.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_dpmm/dml2_dpmm_factory.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_mcg/dml2_mcg_dcn4.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_mcg/dml2_mcg_factory.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_factory.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn4_fams2.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_standalone_libraries/lib_float_math.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/dml21_translation_helper.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/dml21_wrapper.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/dml21_utils.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce120/dce120_timing_generator.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce112/dce112_compressor.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_timing_generator.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_compressor.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_opp_regamma_v.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_opp_csc_v.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_timing_generator_v.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_mem_input_v.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_opp_v.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_transform_v.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce80/dce80_timing_generator.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce60/dce60_timing_generator.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce60/dce60_hw_sequencer.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dce60/dce60_resource.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/hdcp/hdcp_msg.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl_scl_filters.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl_scl_easf_filters.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl_isharp_filters.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl_filters.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/spl/spl_fixpt31_32.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/spl/spl_custom_float.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stat.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_hw_sequencer.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_sink.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_surface.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_debug.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_enc_cfg.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_exports.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_state.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_vm_helper.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dc_helper.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dc_dmub_srv.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dc_edid_parser.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dc/dc_spl_translate.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/freesync/freesync.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_table.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/info_packet/info_packet.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/power/power_helpers.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_srv.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_srv_stat.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_reg.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn20.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn21.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn30.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn301.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn302.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn303.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn31.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn314.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn315.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn316.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn32.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn35.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn351.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn401.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp_ddc.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp_log.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp_psp.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp1_execution.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp1_transition.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp2_execution.o
CC [M] drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp2_transition.o
LD [M] drivers/gpu/drm/amd/amdgpu/amdgpu.o
make[5]: *** [../scripts/Makefile.build:485: drivers/gpu/drm] Error 2
make[4]: *** [../scripts/Makefile.build:485: drivers/gpu] Error 2
make[3]: *** [../scripts/Makefile.build:485: drivers] Error 2
make[2]: *** [/kernel/Makefile:1926: .] Error 2
make[1]: *** [/kernel/Makefile:224: __sub-make] Error 2
make[1]: Leaving directory '/kernel/build64-default'
make: *** [Makefile:224: __sub-make] Error 2
+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel
^ permalink raw reply [flat|nested] 10+ messages in thread