Re: [PATCH v3] Return only active connectors for get_resources ioctl

From: Greg KH <greg@kroah.com>
To: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>
Cc: dri-devel@lists.freedesktop.org, martin.peres@intel.com,
	ville.syrjala@intel.com, chris@chris-wilson.co.uk,
	stable@vger.kernel.org, intel-gfx@lists.freedesktop.org,
	daniel@ffwll.ch, lyude@redhat.com
Subject: Re: [PATCH v3] Return only active connectors for get_resources ioctl
Date: Thu, 29 Nov 2018 12:21:04 +0100	[thread overview]
Message-ID: <20181129112104.GA8983@kroah.com> (raw)
In-Reply-To: <20181129110921.5832-1-stanislav.lisovskiy@intel.com>

On Thu, Nov 29, 2018 at 01:09:21PM +0200, Stanislav Lisovskiy wrote:
> Currently kernel might allocate different connector ids
> for the same outputs in case of DP MST, which seems to
> confuse userspace. There are can be different connector
> ids in the list, which could be assigned to the same
> output, while being in different states.
> This results in issues, like external displays staying
> blank after quick unplugging and plugging back(bug #106250).
> Returning only active DP connectors fixes the issue.
> 
> v2: Removed caps from the title
> 
> v3: Refactored initial condition to look more compact.
>     Thanks to Chris Wilson for giving a hint.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106250
> Signed-off-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>
> ---
>  drivers/gpu/drm/drm_mode_config.c | 6 ++++++
>  1 file changed, 6 insertions(+)

<formletter>

This is not the correct way to submit patches for inclusion in the
stable kernel tree.  Please read:
    https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html
for how to do this properly.

</formletter>