From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com ([192.55.52.115]:31105 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751909AbbLDMXd (ORCPT ); Fri, 4 Dec 2015 07:23:33 -0500 Date: Fri, 4 Dec 2015 14:23:30 +0200 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Daniel Vetter Cc: dri-devel@lists.freedesktop.org, stable@vger.kernel.org Subject: Re: [PATCH 1/7] drm: Don't overwrite UNVERFIED mode status to OK Message-ID: <20151204122330.GV4437@intel.com> References: <1449177255-9515-1-git-send-email-ville.syrjala@linux.intel.com> <1449177255-9515-2-git-send-email-ville.syrjala@linux.intel.com> <20151204081728.GX10243@phenom.ffwll.local> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20151204081728.GX10243@phenom.ffwll.local> Sender: stable-owner@vger.kernel.org List-ID: On Fri, Dec 04, 2015 at 09:17:28AM +0100, Daniel Vetter wrote: > On Thu, Dec 03, 2015 at 11:14:09PM +0200, ville.syrjala@linux.intel.com wrote: > > From: Ville Syrj�l� > > > > The way the mode probing works is this: > > 1. All modes currently on the mode list are marked as UNVERIFIED > > 2. New modes are on the probed_modes list (they start with > > status OK) > > 3. Modes are moved from the probed_modes list to the actual > > mode list. If a mode already on the mode list is deemed > > to match one of the probed modes, the duplicate is dropped > > and the mode status updated to OK. After this the > > probed_modes list will be empty. > > 4. All modes on the mode list are verified to not violate any > > constraints. Any that do are marked as such. > > 5. Any mode left with a non-OK status is pruned from the list, > > with an appropriate debug message. > > This would look really pretty as a kerneldoc addition to > probe_single_connector(). And with asciidoc we can even do pretty ordered > lists like these. Can you please follow-up with a patch for that? I can try. > > > > > What all this means is that any mode on the original list that > > didn't have a duplicate on the probed_modes list, should be left > > with status UNVERFIED (or previously could have been left with > > some other status, but never OK). > > > > I broke that in > > commit 05acaec334fc ("drm: Reorganize probed mode validation") > > by always assigning something to the mode->status during the validation > > step. So any mode from the old list that still passed the validation > > would be left on the list with status OK in the end. > > > > Fix this by not doing the basic mode validation unless the mode > > already has status OK (meaning it came from the probed_modes list, > > or at least a duplicate of it was on that list). This way we will > > correctly prune away any mode from the old mode list that didn't > > appear on the probed_modes list. > > > > Cc: stable@vger.kernel.org > > Cc: Adam Jackson > > Fixes: 05acaec334fc ("drm: Reorganize probed mode validation") > > Signed-off-by: Ville Syrj�l� > > Reviewed-by: Daniel Vetter > > > --- > > drivers/gpu/drm/drm_probe_helper.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/drm_probe_helper.c b/drivers/gpu/drm/drm_probe_helper.c > > index 94ba39e34299..b9b3bd9349ff 100644 > > --- a/drivers/gpu/drm/drm_probe_helper.c > > +++ b/drivers/gpu/drm/drm_probe_helper.c > > @@ -229,7 +229,8 @@ static int drm_helper_probe_single_connector_modes_merge_bits(struct drm_connect > > mode_flags |= DRM_MODE_FLAG_3D_MASK; > > > > list_for_each_entry(mode, &connector->modes, head) { > > - mode->status = drm_mode_validate_basic(mode); > > + if (mode->status == MODE_OK) > > + mode->status = drm_mode_validate_basic(mode); > > > > if (mode->status == MODE_OK) > > mode->status = drm_mode_validate_size(mode, maxX, maxY); > > -- > > 2.4.10 > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/dri-devel > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch -- Ville Syrj�l� Intel OTC From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/7] drm: Don't overwrite UNVERFIED mode status to OK Date: Fri, 4 Dec 2015 14:23:30 +0200 Message-ID: <20151204122330.GV4437@intel.com> References: <1449177255-9515-1-git-send-email-ville.syrjala@linux.intel.com> <1449177255-9515-2-git-send-email-ville.syrjala@linux.intel.com> <20151204081728.GX10243@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by gabe.freedesktop.org (Postfix) with ESMTP id 2BD696E01D for ; Fri, 4 Dec 2015 04:23:33 -0800 (PST) Content-Disposition: inline In-Reply-To: <20151204081728.GX10243@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: stable@vger.kernel.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCBEZWMgMDQsIDIwMTUgYXQgMDk6MTc6MjhBTSArMDEwMCwgRGFuaWVsIFZldHRlciB3 cm90ZToKPiBPbiBUaHUsIERlYyAwMywgMjAxNSBhdCAxMToxNDowOVBNICswMjAwLCB2aWxsZS5z eXJqYWxhQGxpbnV4LmludGVsLmNvbSB3cm90ZToKPiA+IEZyb206IFZpbGxlIFN5cmrDpGzDpCA8 dmlsbGUuc3lyamFsYUBsaW51eC5pbnRlbC5jb20+Cj4gPiAKPiA+IFRoZSB3YXkgdGhlIG1vZGUg cHJvYmluZyB3b3JrcyBpcyB0aGlzOgo+ID4gMS4gQWxsIG1vZGVzIGN1cnJlbnRseSBvbiB0aGUg bW9kZSBsaXN0IGFyZSBtYXJrZWQgYXMgVU5WRVJJRklFRAo+ID4gMi4gTmV3IG1vZGVzIGFyZSBv biB0aGUgcHJvYmVkX21vZGVzIGxpc3QgKHRoZXkgc3RhcnQgd2l0aAo+ID4gICAgc3RhdHVzIE9L KQo+ID4gMy4gTW9kZXMgYXJlIG1vdmVkIGZyb20gdGhlIHByb2JlZF9tb2RlcyBsaXN0IHRvIHRo ZSBhY3R1YWwKPiA+ICAgIG1vZGUgbGlzdC4gSWYgYSBtb2RlIGFscmVhZHkgb24gdGhlIG1vZGUg bGlzdCBpcyBkZWVtZWQKPiA+ICAgIHRvIG1hdGNoIG9uZSBvZiB0aGUgcHJvYmVkIG1vZGVzLCB0 aGUgZHVwbGljYXRlIGlzIGRyb3BwZWQKPiA+ICAgIGFuZCB0aGUgbW9kZSBzdGF0dXMgdXBkYXRl ZCB0byBPSy4gQWZ0ZXIgdGhpcyB0aGUKPiA+ICAgIHByb2JlZF9tb2RlcyBsaXN0IHdpbGwgYmUg ZW1wdHkuCj4gPiA0LiBBbGwgbW9kZXMgb24gdGhlIG1vZGUgbGlzdCBhcmUgdmVyaWZpZWQgdG8g bm90IHZpb2xhdGUgYW55Cj4gPiAgICBjb25zdHJhaW50cy4gQW55IHRoYXQgZG8gYXJlIG1hcmtl ZCBhcyBzdWNoLgo+ID4gNS4gQW55IG1vZGUgbGVmdCB3aXRoIGEgbm9uLU9LIHN0YXR1cyBpcyBw cnVuZWQgZnJvbSB0aGUgbGlzdCwKPiA+ICAgIHdpdGggYW4gYXBwcm9wcmlhdGUgZGVidWcgbWVz c2FnZS4KPiAKPiBUaGlzIHdvdWxkIGxvb2sgcmVhbGx5IHByZXR0eSBhcyBhIGtlcm5lbGRvYyBh ZGRpdGlvbiB0bwo+IHByb2JlX3NpbmdsZV9jb25uZWN0b3IoKS4gQW5kIHdpdGggYXNjaWlkb2Mg d2UgY2FuIGV2ZW4gZG8gcHJldHR5IG9yZGVyZWQKPiBsaXN0cyBsaWtlIHRoZXNlLiBDYW4geW91 IHBsZWFzZSBmb2xsb3ctdXAgd2l0aCBhIHBhdGNoIGZvciB0aGF0PwoKSSBjYW4gdHJ5LgoKPiAK PiA+IAo+ID4gV2hhdCBhbGwgdGhpcyBtZWFucyBpcyB0aGF0IGFueSBtb2RlIG9uIHRoZSBvcmln aW5hbCBsaXN0IHRoYXQKPiA+IGRpZG4ndCBoYXZlIGEgZHVwbGljYXRlIG9uIHRoZSBwcm9iZWRf bW9kZXMgbGlzdCwgc2hvdWxkIGJlIGxlZnQKPiA+IHdpdGggc3RhdHVzIFVOVkVSRklFRCAob3Ig cHJldmlvdXNseSBjb3VsZCBoYXZlIGJlZW4gbGVmdCB3aXRoCj4gPiBzb21lIG90aGVyIHN0YXR1 cywgYnV0IG5ldmVyIE9LKS4KPiA+IAo+ID4gSSBicm9rZSB0aGF0IGluCj4gPiBjb21taXQgMDVh Y2FlYzMzNGZjICgiZHJtOiBSZW9yZ2FuaXplIHByb2JlZCBtb2RlIHZhbGlkYXRpb24iKQo+ID4g YnkgYWx3YXlzIGFzc2lnbmluZyBzb21ldGhpbmcgdG8gdGhlIG1vZGUtPnN0YXR1cyBkdXJpbmcg dGhlIHZhbGlkYXRpb24KPiA+IHN0ZXAuIFNvIGFueSBtb2RlIGZyb20gdGhlIG9sZCBsaXN0IHRo YXQgc3RpbGwgcGFzc2VkIHRoZSB2YWxpZGF0aW9uCj4gPiB3b3VsZCBiZSBsZWZ0IG9uIHRoZSBs aXN0IHdpdGggc3RhdHVzIE9LIGluIHRoZSBlbmQuCj4gPiAKPiA+IEZpeCB0aGlzIGJ5IG5vdCBk b2luZyB0aGUgYmFzaWMgbW9kZSB2YWxpZGF0aW9uIHVubGVzcyB0aGUgbW9kZQo+ID4gYWxyZWFk eSBoYXMgc3RhdHVzIE9LIChtZWFuaW5nIGl0IGNhbWUgZnJvbSB0aGUgcHJvYmVkX21vZGVzIGxp c3QsCj4gPiBvciBhdCBsZWFzdCBhIGR1cGxpY2F0ZSBvZiBpdCB3YXMgb24gdGhhdCBsaXN0KS4g VGhpcyB3YXkgd2Ugd2lsbAo+ID4gY29ycmVjdGx5IHBydW5lIGF3YXkgYW55IG1vZGUgZnJvbSB0 aGUgb2xkIG1vZGUgbGlzdCB0aGF0IGRpZG4ndAo+ID4gYXBwZWFyIG9uIHRoZSBwcm9iZWRfbW9k ZXMgbGlzdC4KPiA+IAo+ID4gQ2M6IHN0YWJsZUB2Z2VyLmtlcm5lbC5vcmcKPiA+IENjOiBBZGFt IEphY2tzb24gPGFqYXhAcmVkaGF0LmNvbT4KPiA+IEZpeGVzOiAwNWFjYWVjMzM0ZmMgKCJkcm06 IFJlb3JnYW5pemUgcHJvYmVkIG1vZGUgdmFsaWRhdGlvbiIpCj4gPiBTaWduZWQtb2ZmLWJ5OiBW aWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+IAo+IFJldmll d2VkLWJ5OiBEYW5pZWwgVmV0dGVyIDxkYW5pZWwudmV0dGVyQGZmd2xsLmNoPgo+IAo+ID4gLS0t Cj4gPiAgZHJpdmVycy9ncHUvZHJtL2RybV9wcm9iZV9oZWxwZXIuYyB8IDMgKystCj4gPiAgMSBm aWxlIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQo+ID4gCj4gPiBkaWZm IC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2RybV9wcm9iZV9oZWxwZXIuYyBiL2RyaXZlcnMvZ3B1 L2RybS9kcm1fcHJvYmVfaGVscGVyLmMKPiA+IGluZGV4IDk0YmEzOWUzNDI5OS4uYjliM2JkOTM0 OWZmIDEwMDY0NAo+ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2RybV9wcm9iZV9oZWxwZXIuYwo+ ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2RybV9wcm9iZV9oZWxwZXIuYwo+ID4gQEAgLTIyOSw3 ICsyMjksOCBAQCBzdGF0aWMgaW50IGRybV9oZWxwZXJfcHJvYmVfc2luZ2xlX2Nvbm5lY3Rvcl9t b2Rlc19tZXJnZV9iaXRzKHN0cnVjdCBkcm1fY29ubmVjdAo+ID4gIAkJbW9kZV9mbGFncyB8PSBE Uk1fTU9ERV9GTEFHXzNEX01BU0s7Cj4gPiAgCj4gPiAgCWxpc3RfZm9yX2VhY2hfZW50cnkobW9k ZSwgJmNvbm5lY3Rvci0+bW9kZXMsIGhlYWQpIHsKPiA+IC0JCW1vZGUtPnN0YXR1cyA9IGRybV9t b2RlX3ZhbGlkYXRlX2Jhc2ljKG1vZGUpOwo+ID4gKwkJaWYgKG1vZGUtPnN0YXR1cyA9PSBNT0RF X09LKQo+ID4gKwkJCW1vZGUtPnN0YXR1cyA9IGRybV9tb2RlX3ZhbGlkYXRlX2Jhc2ljKG1vZGUp Owo+ID4gIAo+ID4gIAkJaWYgKG1vZGUtPnN0YXR1cyA9PSBNT0RFX09LKQo+ID4gIAkJCW1vZGUt PnN0YXR1cyA9IGRybV9tb2RlX3ZhbGlkYXRlX3NpemUobW9kZSwgbWF4WCwgbWF4WSk7Cj4gPiAt LSAKPiA+IDIuNC4xMAo+ID4gCj4gPiBfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fXwo+ID4gZHJpLWRldmVsIG1haWxpbmcgbGlzdAo+ID4gZHJpLWRldmVsQGxp c3RzLmZyZWVkZXNrdG9wLm9yZwo+ID4gaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWls bWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo+IAo+IC0tIAo+IERhbmllbCBWZXR0ZXIKPiBTb2Z0d2Fy ZSBFbmdpbmVlciwgSW50ZWwgQ29ycG9yYXRpb24KPiBodHRwOi8vYmxvZy5mZndsbC5jaAoKLS0g ClZpbGxlIFN5cmrDpGzDpApJbnRlbCBPVEMKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMu ZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0 aW5mby9kcmktZGV2ZWwK