From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maarten Lankhorst Subject: Re: [PATCH i-g-t v3 2/7] lib/igt_kms: Fix off-by-one bug on skip of missing pipe Date: Thu, 31 Aug 2017 09:08:56 +0200 Message-ID: References: <20170831061810.11773-1-krisman@collabora.co.uk> <20170831061810.11773-3-krisman@collabora.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTPS id 7EB086E161 for ; Thu, 31 Aug 2017 07:09:00 +0000 (UTC) In-Reply-To: Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Gabriel Krisman Bertazi , intel-gfx@lists.freedesktop.org Cc: daniel.vetter@intel.com, daniels@collabora.com, tomeu.vizoso@collabora.com List-Id: intel-gfx@lists.freedesktop.org T3AgMzEtMDgtMTcgb20gMDk6MDYgc2NocmVlZiBNYWFydGVuIExhbmtob3JzdDoKPiBPcCAzMS0w OC0xNyBvbSAwODoxOCBzY2hyZWVmIEdhYnJpZWwgS3Jpc21hbiBCZXJ0YXppOgo+PiBkaXNwbGF5 LT5uX3BpcGVzIGlzIHplcm8taW5kZXhlZCwgc28gTiByZXR1cm5lZCBpbgo+PiBpZ3RfZGlzcGxh eV9nZXRfbl9waXBlcyBpcyBhbHJlYWR5IG5vdCBhIHZhbGlkIHBpcGUuICBUaGlzIHBhdGNoCj4+ IHByZXZlbnRzIGttc19jY3MgZnJvbSBnb2luZyBudXRzIHdoZW4gdGVzdGluZyB0aGUgZmlyc3Qg dW54ZXN0aW5nIHBpcGUuCj4+Cj4+IFNpZ25lZC1vZmYtYnk6IEdhYnJpZWwgS3Jpc21hbiBCZXJ0 YXppIDxrcmlzbWFuQGNvbGxhYm9yYS5jby51az4KPj4gLS0tCj4+ICBsaWIvaWd0X2ttcy5jIHwg MiArLQo+PiAgMSBmaWxlIGNoYW5nZWQsIDEgaW5zZXJ0aW9uKCspLCAxIGRlbGV0aW9uKC0pCj4+ Cj4+IGRpZmYgLS1naXQgYS9saWIvaWd0X2ttcy5jIGIvbGliL2lndF9rbXMuYwo+PiBpbmRleCAx NGUyNzAxYzNhZmQuLmNlMDdmY2MxZmM3MyAxMDA2NDQKPj4gLS0tIGEvbGliL2lndF9rbXMuYwo+ PiArKysgYi9saWIvaWd0X2ttcy5jCj4+IEBAIC0xODY0LDcgKzE4NjQsNyBAQCB2b2lkIGlndF9k aXNwbGF5X3JlcXVpcmVfb3V0cHV0X29uX3BpcGUoaWd0X2Rpc3BsYXlfdCAqZGlzcGxheSwgZW51 bSBwaXBlIHBpcGUpCj4+ICB7Cj4+ICAJaWd0X291dHB1dF90ICpvdXRwdXQ7Cj4+ICAKPj4gLQlp Z3Rfc2tpcF9vbl9mKGlndF9kaXNwbGF5X2dldF9uX3BpcGVzKGRpc3BsYXkpIDwgcGlwZSwKPj4g KwlpZ3Rfc2tpcF9vbl9mKGlndF9kaXNwbGF5X2dldF9uX3BpcGVzKGRpc3BsYXkpIDw9IHBpcGUs Cj4+ICAJCSAgICAgICJQaXBlICVzIGRvZXMgbm90IGV4aXN0LlxuIiwga21zdGVzdF9waXBlX25h bWUocGlwZSkpOwo+PiAgCj4+ICAJZm9yX2VhY2hfdmFsaWRfb3V0cHV0X29uX3BpcGUoZGlzcGxh eSwgcGlwZSwgb3V0cHV0KQo+IEkgdGhpbmsgdGhlIG9yaWdpbmFsIGNoZWNrIGlzIGNvcnJlY3Q/ Cj4KPiBBbHRob3VnaCBpdCBjb3VsZCBiZSBpbnZlcnRlZCB0byBtYWtlIGl0IG1vcmUgY2xlYXIs IG1hdGNoaW5nIHRoZSBvdGhlciBzaW1pbGFyIGNoZWNrcyBpbiB0ZXN0cy9rbXNfKi5jOgo+Cj4g aWd0X3NraXBfb25fZihwaXBlID49IGlndF9kaXNwbGF5X2dldF9uX3BpcGVzKGRpc3BsYXkpLApP b3BzLCBlYXJseSBtb3JuaW5nIG1hdGggZmFpbC4uCgpKdXN0IHVzZSB0aGUgc3VnZ2VzdGVkIGZv cm0gcGxlYXNlLiA6KQoKUmV2aWV3ZWQtYnk6IE1hYXJ0ZW4gTGFua2hvcnN0IDxtYWFydGVuLmxh bmtob3JzdEBsaW51eC5pbnRlbC5jb20+Cgp+TWFhcnRlbgpfX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdm eEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFp bG1hbi9saXN0aW5mby9pbnRlbC1nZngK