From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [RFC][PATCH 07/11] drm/probe-helper: Avoid iterating the list twice on ww backoff Date: Tue, 6 Mar 2018 13:48:32 +0200 Message-ID: <20180306114832.GB5453@intel.com> References: <20180227125700.6527-1-ville.syrjala@linux.intel.com> <20180227125700.6527-8-ville.syrjala@linux.intel.com> <20180306094943.GV22212@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20180306094943.GV22212@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCBNYXIgMDYsIDIwMTggYXQgMTA6NDk6NDNBTSArMDEwMCwgRGFuaWVsIFZldHRlciB3 cm90ZToKPiBPbiBUdWUsIEZlYiAyNywgMjAxOCBhdCAwMjo1Njo1NlBNICswMjAwLCBWaWxsZSBT eXJqYWxhIHdyb3RlOgo+ID4gRnJvbTogVmlsbGUgU3lyasOkbMOkIDx2aWxsZS5zeXJqYWxhQGxp bnV4LmludGVsLmNvbT4KPiA+IAo+ID4gSWYgd2UgaGF2ZSB0byBiYWNrb2ZmIHRoZXJlJ3Mgbm8g cG9pbnQgaW4gZ29pbmcgb3ZlciB0aGUgbW9kZSBsaXN0IGFnYWluCj4gPiB0byBtYXJrIGFsbCB0 aGUgbW9kZXMgYXMgc3RhbGUuIFdlIGNhbiBkZWZlciB0aGF0IHVudGlsIHdlJ3JlIHJlYWR5IHRv Cj4gPiByZWZyZXNoIHRoZSBtb2RlIGxpc3QuIEF2b2lkcyBtdWx0aXBsZSBsaXN0IHdhbGtzIGlm IHdlIGhhdmUgdG8gZG8gdGhlCj4gPiBsb2NraW5nIGJhY2tvZmYuCj4gPiAKPiA+IENjOiBLZWl0 aCBQYWNrYXJkIDxrZWl0aHBAa2VpdGhwLmNvbT4KPiA+IENjOiBEYW5pZWwgVmV0dGVyIDxkYW5p ZWwudmV0dGVyQGZmd2xsLmNoPgo+ID4gU2lnbmVkLW9mZi1ieTogVmlsbGUgU3lyasOkbMOkIDx2 aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4KPiAKPiBTb21lIGRyaXZlcnMgYWRkIG1vZGVz IGZyb20gdGhlaXIgLT5kZXRlY3QgY2FsbGJhY2tzIEkgdGhpbmssIHdvbid0IHRoaXMKPiBicmVh ayB0aGVtPyBBdCBsZWFzdCBJJ20gbmV2ZXIgc3VyZSBhYm91dCB3aGF0IGRpZmZlcmVudCBkcml2 ZXJzIGFyZQo+IHN1cHBvc2VkIHRvIGRvIGluIGVhY2ggY2FzZS4KPiAKPiBPdG9oIHRoZSBrZXJu ZWxkb2MgaXMgZmFpcmx5IGNsZWFyLCBhbmQgdGhpcyBwYXRjaCBpcyBlYXN5IHRvIHJldmVydCwg YW5kCj4gaXQgbWlnaHQgaGVscCBpbiBmb3JjaW5nIGRyaXZlcnMgdG8gYmUgbW9yZSBjb25zaXN0 ZW50LgoKSG1tLiBZZWFoIEkgZGlkbid0IGFjdHVhbGx5IG1ha2Ugc3VyZSB0aGF0IGRyaXZlcnMg YXJlbid0IGFkZGluZyBzdHVmZgp0byB0aGUgLT5tb2RlcyBsaXN0IGRpcmVjdGx5LiBUaGV5IHNo b3VsZG4ndCBidXQgd2hvIGtub3dzLgoKPiAKPiBJZiB5b3UgYWRkIGEgbm90ZSBhYm91dCBteSBj b25jZXJucyAoYXMgYSBoaW50IGluIGNhc2Ugd2UgYmlzZWN0IGEKPiByZWdyZXNzaW9uIHRvIHRo aXMgcGF0Y2gpLCB0aGVuIHRoaXMgaXM6CgpTdXJlLCBJJ2xsIHRyeSB0byBhZGQgc29tZXRoaW5n LgoKPiAKPiBSZXZpZXdlZC1ieTogRGFuaWVsIFZldHRlciA8ZGFuaWVsLnZldHRlckBmZndsbC5j aD4KPiAKPiAKPiA+IC0tLQo+ID4gIGRyaXZlcnMvZ3B1L2RybS9kcm1fcHJvYmVfaGVscGVyLmMg fCA4ICsrKystLS0tCj4gPiAgMSBmaWxlIGNoYW5nZWQsIDQgaW5zZXJ0aW9ucygrKSwgNCBkZWxl dGlvbnMoLSkKPiA+IAo+ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9kcm1fcHJvYmVf aGVscGVyLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX3Byb2JlX2hlbHBlci5jCj4gPiBpbmRleCA1 Mjc3NDMzOTQxNTAuLjdkYzdlNjM1ZDdlNCAxMDA2NDQKPiA+IC0tLSBhL2RyaXZlcnMvZ3B1L2Ry bS9kcm1fcHJvYmVfaGVscGVyLmMKPiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fcHJvYmVf aGVscGVyLmMKPiA+IEBAIC00MTUsMTAgKzQxNSw2IEBAIGludCBkcm1faGVscGVyX3Byb2JlX3Np bmdsZV9jb25uZWN0b3JfbW9kZXMoc3RydWN0IGRybV9jb25uZWN0b3IgKmNvbm5lY3RvciwKPiA+ ICAJfSBlbHNlCj4gPiAgCQlXQVJOX09OKHJldCA8IDApOwo+ID4gIAo+ID4gLQkvKiBzZXQgYWxs IG9sZCBtb2RlcyB0byB0aGUgc3RhbGUgc3RhdGUgKi8KPiA+IC0JbGlzdF9mb3JfZWFjaF9lbnRy eShtb2RlLCAmY29ubmVjdG9yLT5tb2RlcywgaGVhZCkKPiA+IC0JCW1vZGUtPnN0YXR1cyA9IE1P REVfU1RBTEU7Cj4gPiAtCj4gPiAgCW9sZF9zdGF0dXMgPSBjb25uZWN0b3ItPnN0YXR1czsKPiA+ ICAKPiA+ICAJaWYgKGNvbm5lY3Rvci0+Zm9yY2UpIHsKPiA+IEBAIC00NzIsNiArNDY4LDEwIEBA IGludCBkcm1faGVscGVyX3Byb2JlX3NpbmdsZV9jb25uZWN0b3JfbW9kZXMoc3RydWN0IGRybV9j b25uZWN0b3IgKmNvbm5lY3RvciwKPiA+ICAKPiA+ICAJZGV2LT5tb2RlX2NvbmZpZy5wb2xsX3J1 bm5pbmcgPSBkcm1fa21zX2hlbHBlcl9wb2xsOwo+ID4gIAo+ID4gKwkvKiBzZXQgYWxsIG9sZCBt b2RlcyB0byB0aGUgc3RhbGUgc3RhdGUgKi8KPiA+ICsJbGlzdF9mb3JfZWFjaF9lbnRyeShtb2Rl LCAmY29ubmVjdG9yLT5tb2RlcywgaGVhZCkKPiA+ICsJCW1vZGUtPnN0YXR1cyA9IE1PREVfU1RB TEU7Cj4gPiArCj4gPiAgCWlmIChjb25uZWN0b3ItPnN0YXR1cyA9PSBjb25uZWN0b3Jfc3RhdHVz X2Rpc2Nvbm5lY3RlZCkgewo+ID4gIAkJRFJNX0RFQlVHX0tNUygiW0NPTk5FQ1RPUjolZDolc10g ZGlzY29ubmVjdGVkXG4iLAo+ID4gIAkJCWNvbm5lY3Rvci0+YmFzZS5pZCwgY29ubmVjdG9yLT5u YW1lKTsKPiA+IC0tIAo+ID4gMi4xMy42Cj4gPiAKPiAKPiAtLSAKPiBEYW5pZWwgVmV0dGVyCj4g U29mdHdhcmUgRW5naW5lZXIsIEludGVsIENvcnBvcmF0aW9uCj4gaHR0cDovL2Jsb2cuZmZ3bGwu Y2gKCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4 QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWls bWFuL2xpc3RpbmZvL2ludGVsLWdmeAo=