From: Imre Deak <imre.deak@intel.com>
To: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH 5/9] drm/display/dp_mst: Fix the payload VCPI check in drm_dp_mst_dump_topology()
Date: Fri, 27 Jan 2023 21:55:32 +0200 [thread overview]
Message-ID: <Y9QsNP5LRf23BHbD@ideak-desk.fi.intel.com> (raw)
In-Reply-To: <Y9QpL1Y110IZcrSK@intel.com>
On Fri, Jan 27, 2023 at 09:42:39PM +0200, Ville Syrjälä wrote:
> On Wed, Jan 25, 2023 at 01:48:48PM +0200, Imre Deak wrote:
> > Fix an off-by-one error in the VCPI check in drm_dp_mst_dump_topology().
> >
> > Cc: Lyude Paul <lyude@redhat.com>
> > Cc: dri-devel@lists.freedesktop.org
> > Signed-off-by: Imre Deak <imre.deak@intel.com>
> > ---
> > drivers/gpu/drm/display/drm_dp_mst_topology.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c b/drivers/gpu/drm/display/drm_dp_mst_topology.c
> > index 81cc0c3b1e000..619f616d69e20 100644
> > --- a/drivers/gpu/drm/display/drm_dp_mst_topology.c
> > +++ b/drivers/gpu/drm/display/drm_dp_mst_topology.c
> > @@ -4770,7 +4770,7 @@ void drm_dp_mst_dump_topology(struct seq_file *m,
> > list_for_each_entry(payload, &state->payloads, next) {
> > char name[14];
> >
> > - if (payload->vcpi != i || payload->delete)
> > + if (payload->vcpi != i + 1 || payload->delete)
>
> Why does this code even do that funny nested double loop?
The payload list is not ordered by VCPIs I think, but the printout wants
to list them in VCPI order.
>
> > continue;
> >
> > fetch_monitor_name(mgr, payload->port, name, sizeof(name));
> > --
> > 2.37.1
>
> --
> Ville Syrjälä
> Intel
WARNING: multiple messages have this Message-ID (diff)
From: Imre Deak <imre.deak@intel.com>
To: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 5/9] drm/display/dp_mst: Fix the payload VCPI check in drm_dp_mst_dump_topology()
Date: Fri, 27 Jan 2023 21:55:32 +0200 [thread overview]
Message-ID: <Y9QsNP5LRf23BHbD@ideak-desk.fi.intel.com> (raw)
In-Reply-To: <Y9QpL1Y110IZcrSK@intel.com>
On Fri, Jan 27, 2023 at 09:42:39PM +0200, Ville Syrjälä wrote:
> On Wed, Jan 25, 2023 at 01:48:48PM +0200, Imre Deak wrote:
> > Fix an off-by-one error in the VCPI check in drm_dp_mst_dump_topology().
> >
> > Cc: Lyude Paul <lyude@redhat.com>
> > Cc: dri-devel@lists.freedesktop.org
> > Signed-off-by: Imre Deak <imre.deak@intel.com>
> > ---
> > drivers/gpu/drm/display/drm_dp_mst_topology.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c b/drivers/gpu/drm/display/drm_dp_mst_topology.c
> > index 81cc0c3b1e000..619f616d69e20 100644
> > --- a/drivers/gpu/drm/display/drm_dp_mst_topology.c
> > +++ b/drivers/gpu/drm/display/drm_dp_mst_topology.c
> > @@ -4770,7 +4770,7 @@ void drm_dp_mst_dump_topology(struct seq_file *m,
> > list_for_each_entry(payload, &state->payloads, next) {
> > char name[14];
> >
> > - if (payload->vcpi != i || payload->delete)
> > + if (payload->vcpi != i + 1 || payload->delete)
>
> Why does this code even do that funny nested double loop?
The payload list is not ordered by VCPIs I think, but the printout wants
to list them in VCPI order.
>
> > continue;
> >
> > fetch_monitor_name(mgr, payload->port, name, sizeof(name));
> > --
> > 2.37.1
>
> --
> Ville Syrjälä
> Intel
next prev parent reply other threads:[~2023-01-27 19:55 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-25 11:48 [Intel-gfx] [PATCH 0/9] drm/i915/dp_mst: Fix MST payload removal during output disabling Imre Deak
2023-01-25 11:48 ` [Intel-gfx] [PATCH 1/9] drm/i915/dp_mst: Add the MST topology state for modesetted CRTCs Imre Deak
2023-01-25 11:48 ` Imre Deak
2023-01-26 9:13 ` [Intel-gfx] [PATCH v2 " Imre Deak
2023-01-26 9:13 ` Imre Deak
2023-01-26 18:34 ` [Intel-gfx] " Ville Syrjälä
2023-01-26 20:29 ` Lyude Paul
2023-01-26 20:29 ` Lyude Paul
2023-01-25 11:48 ` [Intel-gfx] [PATCH 2/9] drm/display/dp_mst: Handle old/new payload states in drm_dp_remove_payload() Imre Deak
2023-01-25 11:48 ` Imre Deak
2023-01-25 11:48 ` Imre Deak
2023-01-26 17:37 ` [Intel-gfx] " Ville Syrjälä
2023-01-26 17:37 ` Ville Syrjälä
2023-01-26 17:37 ` Ville Syrjälä
2023-01-26 18:33 ` [Intel-gfx] " Ville Syrjälä
2023-01-26 18:33 ` Ville Syrjälä
2023-01-26 20:21 ` Imre Deak
2023-01-26 20:21 ` Imre Deak
2023-01-25 11:48 ` [Intel-gfx] [PATCH 3/9] drm/display/dp_mst: Add drm_atomic_get_old_mst_topology_state() Imre Deak
2023-01-25 11:48 ` Imre Deak
2023-01-25 11:48 ` Imre Deak
2023-01-26 18:36 ` [Intel-gfx] " Ville Syrjälä
2023-01-26 18:36 ` Ville Syrjälä
2023-01-26 18:36 ` Ville Syrjälä
2023-01-26 20:28 ` [Intel-gfx] " Imre Deak
2023-01-26 20:28 ` Imre Deak
2023-01-26 20:28 ` Imre Deak
2023-01-25 11:48 ` [Intel-gfx] [PATCH 4/9] drm/i915/dp_mst: Fix payload removal during output disabling Imre Deak
2023-01-25 11:48 ` Imre Deak
2023-01-26 18:38 ` [Intel-gfx] " Ville Syrjälä
2023-01-26 20:48 ` Imre Deak
2023-01-25 11:48 ` [Intel-gfx] [PATCH 5/9] drm/display/dp_mst: Fix the payload VCPI check in drm_dp_mst_dump_topology() Imre Deak
2023-01-25 11:48 ` Imre Deak
2023-01-27 19:42 ` [Intel-gfx] " Ville Syrjälä
2023-01-27 19:42 ` Ville Syrjälä
2023-01-27 19:55 ` Imre Deak [this message]
2023-01-27 19:55 ` Imre Deak
2023-01-25 11:48 ` [Intel-gfx] [PATCH 6/9] drm/i915: Factor out helpers for modesetting CRTCs and connectors Imre Deak
2023-01-25 11:48 ` [Intel-gfx] [PATCH 7/9] drm/i915/dp_mst: Move getting the MST topology state earlier to connector check Imre Deak
2023-01-25 11:48 ` [Intel-gfx] [PATCH 8/9] drm/display/dp_mst: Add a helper to verify the MST payload state Imre Deak
2023-01-25 11:48 ` Imre Deak
2023-01-25 11:48 ` [Intel-gfx] [PATCH 9/9] drm/i915/dp_mst: Verify the MST state of modesetted outputs Imre Deak
2023-01-25 23:39 ` [Intel-gfx] ✗ Fi.CI.BUILD: failure for drm/i915/dp_mst: Fix MST payload removal during output disabling Patchwork
2023-01-26 10:04 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/dp_mst: Fix MST payload removal during output disabling (rev2) Patchwork
2023-01-26 10:24 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2023-01-26 16:55 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Y9QsNP5LRf23BHbD@ideak-desk.fi.intel.com \
--to=imre.deak@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=ville.syrjala@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.