From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by gabe.freedesktop.org (Postfix) with ESMTPS id 754BC6E581 for ; Thu, 4 Apr 2019 08:35:41 +0000 (UTC) From: "Lisovskiy, Stanislav" Date: Thu, 4 Apr 2019 08:35:38 +0000 Message-ID: <04b35e9b8230119a08f520f7c9aa431640cb56af.camel@intel.com> References: <20190219093800.4312-1-stanislav.lisovskiy@intel.com> <690c1112b0fd97f4810cced16d442d16ca75809b.camel@intel.com> <20190403142311.GL2665@phenom.ffwll.local> <126e6a048495814781d60526fc7fc813158d821c.camel@intel.com> <20190404081046.GQ2665@phenom.ffwll.local> In-Reply-To: <20190404081046.GQ2665@phenom.ffwll.local> Content-Language: en-US Content-ID: MIME-Version: 1.0 Subject: Re: [igt-dev] [PATCH i-g-t v4] igt/tests: Fix error checking in kms_atomic_transition List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: "daniel@ffwll.ch" Cc: "norkernel@freedesktop.org" , "Syrjala, Ville" , "igt-dev@lists.freedesktop.org" , "Peres, Martin" , "Heikkila, Juha-pekka" List-ID: T24gVGh1LCAyMDE5LTA0LTA0IGF0IDEwOjEwICswMjAwLCBEYW5pZWwgVmV0dGVyIHdyb3RlOg0K PiBPbiBUaHUsIEFwciAwNCwgMjAxOSBhdCAwNzozMzozMUFNICswMDAwLCBMaXNvdnNraXksIFN0 YW5pc2xhdiB3cm90ZToNCj4gPiBPbiBXZWQsIDIwMTktMDQtMDMgYXQgMTY6MjMgKzAyMDAsIERh bmllbCBWZXR0ZXIgd3JvdGU6DQo+ID4gPiBPbiBUdWUsIEZlYiAxOSwgMjAxOSBhdCAwOTowNTox MlBNICswMDAwLCBTdW1tZXJzLCBTdHVhcnQgd3JvdGU6DQo+ID4gPiA+IE9uIFR1ZSwgMjAxOS0w Mi0xOSBhdCAxMTozOCArMDIwMCwgU3RhbmlzbGF2IExpc292c2tpeSB3cm90ZToNCj4gPiA+ID4g PiBGcm9tOiBTdGFuaXNsYXYgTGlzb3Zza2l5IDxzdGFuaXNsYXYubGlzb3Zza2l5QGdtYWlsLmNv bT4NCj4gPiA+ID4gPiANCj4gPiA+ID4gPiBUaGVyZSBpcyBubyBndWFyYW50ZWUgdGhhdCBlcnJv ciByZXR1cm4gdmFsdWUgd2lsbCBiZQ0KPiA+ID4gPiA+IGFsd2F5cyBFSU5WQUwsIG1hZGUgYSBj aGVjayBtb3JlIGdlbmVyYWwgYXMgaXQgY2FuIGJlDQo+ID4gPiA+ID4gRVJBTkdFLCBFTk9TUEMs IEVJTlZBTCBhbmQgcHJvYmFibHkgb3RoZXJzLCB3aGljaCBhbGwNCj4gPiA+ID4gPiBtZWFuIHRo ZSBzYW1lIGluIGNvbnRleHQgb2YgdGhpcyB0ZXN0IGNhc2U6IGkuZSB0aGlzIHNwcml0ZQ0KPiA+ ID4gPiA+IHNpemUgaXMgbm90IHZhbGlkLg0KPiA+ID4gPiA+IA0KPiA+ID4gPiA+IHYyOiBBZGRl ZCBtYWNybyB0byBtYWtlIGNoZWNrIGxvb2sgYSBiaXQgbmljZXIuDQo+ID4gPiA+ID4gdjM6IFJl bW92ZWQgcmVkdW5kYW50IGRlYnVnIGxpbmUuDQo+ID4gPiA+ID4gdjQ6IEFkZGVkIGFzc2VydGlv biBpZiBlcnJvciBpcyBub3QgRUlOVkFMIGFzIGV4cGVjdGVkLA0KPiA+ID4gPiA+ICAgICBvdGhl ciBlcnJvcnMgZXhjZXB0IEVJTlZBTCBhcmUgY29uc2lkZXJlZCBub3cgYSBmYWlsdXJlcy4NCj4g PiA+IA0KPiA+ID4gVWgsIGNvbW1pdCBtZXNzYWdlIHNheXMgdG8gbWFrZSB0aGUgZXJyb3IgbWVz c2FnZSBtb3JlIGdlbmVyYWwsDQo+ID4gPiBhbmQNCj4gPiA+IHRoZW4NCj4gPiA+IHByb2NlZWRz IHRvIG1ha2UgaXQgbm90IG1vcmUgZ2VuZXJhbC4gVGhpcyBraW5kIG9mDQo+ID4gPiBpbmNvbnNp c3RlbmNpZXMNCj4gPiA+IHNob3VsZA0KPiA+ID4gYmUgY2F1Z2h0IGluIHJldmlldy4NCj4gPiAN Cj4gPiBIaSBEYW5pZWwsDQo+ID4gDQo+ID4gQXMgdGhlIGNvbW1pdCBtZXNzYWdlIHNheXMsIHBy ZXZpc291bHkgd2Ugd2VyZSBleHBlY3Rpbmcgb25seQ0KPiA+IEVJTlZBTCh3aGljaCBpbmRpY2F0 ZXMgZm9yDQo+ID4gdXMgdGhhdCB3ZSBoYWQgcmVhY2hlZCBtYXggc2l6ZSBvZiBhIHBsYW5lKSBv ciAwLg0KPiA+IA0KPiA+IFByb2JsZW0gd2FzIHRoYXQgaW4gc29tZSBjaXJjdW1zdGFuY2VzLCBp ZiB3ZSBnZXQgc29tZXRoaW5nIGVsc2UNCj4gPiB0ZXN0IGNhc2UgZGlkIG5vdCBjYXRjaCB0aGF0 IGFuZCBwcm9jZWVkZWQgYXMgaWYgZXZlcnl0aGluZyBpcw0KPiA+IGdyZWF0LCBmYWlsaW5nIGFm dGVyd2FyZHMsIHdoZW4gdHJ5aW5nIGFscmVhZHkgdG8gY29tbWl0IG9yIG90aGVyDQo+ID4gcGxh Y2UuDQo+ID4gVGhlIHRoaW5nIHdoaWNoIHRoaXMgcGF0Y2ggZG9lcyBpcyBhcyBzYWlkICJBZGRl ZCBhc3NlcnRpb24gaWYNCj4gPiBlcnJvciBpcw0KPiA+IG5vdCBFSU5WQUwgYXMgZXhwZWN0ZWQs IG90aGVyIGVycm9ycyBleGNlcHQgRUlOVkFMIGFyZSBjb25zaWRlcmVkDQo+ID4gbm93IGENCj4g PiBmYWlsdXJlcyIuIE90aGVyd2lzZSBpZiB3ZSBnb3QgRUlOVkFMLCB3ZSBwcm9jZWVkIG5vcm1h bGx5Lg0KPiA+IA0KPiA+IFRoZXJlIHdhcyBhY3R1YWxseSBubyBwdXJwb3NlIHRvICJmaXgiIHNv bWV0aGluZyBieSB0aGF0LCBidXQNCj4gPiByYXRoZXINCj4gPiBtYWtlIGl0IGZhaWwgaW4gYSBw cm9wZXIgcGxhY2UgYW5kIHByb3BlciB3YXksIG90aGVyd2lzZSBlcnJvciBqdXN0DQo+ID4gd2Vu dCBmdXJ0aGVyLiANCj4gPiANCj4gPiBNeSBpbml0aWFsIGlkZWEgYWN0dWFsbHkgd2FzIHRvIGNh dGNoIGFsbCBraW5kIG9mIHByb2JsZW1zIGhlcmUsIHNvDQo+ID4gdGhhdCB3ZSBhbHdheXMgZ2V0 IHNvbWUga2luZCBvZiB3b3JraW5nIHBsYW5lIGNvbmZpZ3VyYXRpb24gaGVyZS4NCj4gPiBIb3dl dmVyIGl0IHdhcyBub3QgYWNjZXB0ZWQgYW5kIGN1cnJlbnRseSBvbmx5IEVJTlZBTCBpcyB0cmVh dGVkIGFzDQo+ID4gIm5vcm1hbCIgZXJyb3IuDQo+ID4gDQo+ID4gSWYgeW91IGhhdmUgYW55IHBy b3Bvc2FscywgaG93IHRoaXMgY2FuIGJlIGltcHJvdmVkIHN1cmUgSSBjYW4gZG8NCj4gPiB0aGlz Lg0KPiANCj4gWWVhaCBJIHRoaW5rIHRoZSBwYXRjaCBpcyBmaW5lLCBidXQgdGhlIGNvbW1pdCBt ZXNzYWdlIGlzbid0IHRoZQ0KPiBjbGVhcmVzdCwNCj4gc2luY2UgaXQgZXhwbGFpbnMgd2hhdCBl eGFjdGx5IGFuIGludGVyaW0gdmVyc2lvbiBkaWQsIGFuZCB0aGVuDQo+IGFtZW5kcw0KPiB0aGF0 LiBXaGljaCBpcyBjb25mdXNpbmcuDQo+IA0KPiBZb3UncmUgZXhwbGFuYXRpb24gYWJvdmUgd291 bGQgaGF2ZSBtYWRlIGEgbXVjaCBiZXR0ZXIgY29tbWl0DQo+IG1lc3NhZ2UsIGJ1dA0KPiBvaCB3 ZWxsIHRoZSBwYXRjaCBpcyBjb21taXR0ZWQgc28gY2FuJ3QgZml4IHRoYXQgdXAgYW55bW9yZS4g QnV0IGdvb2QNCj4gdG8NCj4gaGVlZCBmb3IgbmV4dCB0aW1lIGFyb3VuZCwgcmV2aWV3IHNob3Vs ZCBhbHNvIG1ha2Ugc3VyZSB0aGUgY29tbWl0DQo+IG1lc3NhZ2UNCj4gY2FwdHVyZXMgZXZlcnl0 aGluZyBkaXNjdXNzZWQgYXMgd2VsbCBhcyBwb3NzaWJsZS4NCj4gDQo+IENoZWVycywgRGFuaWVs DQoNCldlbGwsIHllYWggdjQgcmV2aXNpb24gc2F5cyBzb21ldGhpbmcgZGlmZmVyZW50IGZyb20g aW5pdGlhbCBjb21taXQNCm1lc3NhZ2UgOikgSSBqdXN0IHRob3VnaHQgdGhhdCBJIG5lZWQgdG8g a2VlcCBpdCBmb3IgaGlzdG9yeS4NCg0KVG8gbWUgaXQgaXMgc3RpbGwgYXQgbGVhc3QgYXJndWFi bGUsIGlmIHdlIHJlYWxseSBuZWVkIHRvIGZhaWwgaW4NCnNldHVwX3Bhcm1zLiBBbnl3YXksIGlm IHRoZSBmdW5jdGlvbiBwdXJwb3NlIHdhcyB0byBmaW5kIGEgbWF4aW11bSBzaXplDQpmb3Igc3By aXRlIHBsYW5lLCBzaG91bGQgaXQgcmVhbGx5IG1hdHRlciB3aGF0IHdhcyB0aGUgDQplcnJvcigg RU5PU1BDLCBFUkFOR0UsIEVJTlZBTCkgdGhlbiBpZiB3ZSBhcmUgc3RpbGwgYWJsZSB0byBmaW5k IG1heA0KcGxhbmUgc2l6ZSB3aGljaCB3b3Jrcz8NCg0KPiA+IA0KPiA+ID4gDQo+ID4gPiA+ID4g QnVnemlsbGE6IGh0dHBzOi8vYnVncy5mcmVlZGVza3RvcC5vcmcvc2hvd19idWcuY2dpP2lkPTEw OTIyNQ0KPiA+ID4gPiA+IFNpZ25lZC1vZmYtYnk6IFN0YW5pc2xhdiBMaXNvdnNraXkgPA0KPiA+ ID4gPiA+IHN0YW5pc2xhdi5saXNvdnNraXlAaW50ZWwuY29tDQo+ID4gPiA+ID4gPiANCj4gPiA+ IA0KPiA+ID4gQW5kIEkgYWxzbyBkb24ndCB1bmRlcnN0YW5kIGhvdyB0aGlzIHBhdGNoIGZpeGVk IGFueXRoaW5nIHJlYWxseSwNCj4gPiA+IHdlDQo+ID4gPiBzdGlsbA0KPiA+ID4ganVzdCBjaGVj ayBmb3IgRUlOVkFMLg0KPiA+ID4gLURhbmllbA0KPiA+ID4gDQo+ID4gPiA+ID4gLS0tDQo+ID4g PiA+ID4gIHRlc3RzL2ttc19hdG9taWNfdHJhbnNpdGlvbi5jIHwgMTMgKysrKysrKysrKystLQ0K PiA+ID4gPiA+ICAxIGZpbGUgY2hhbmdlZCwgMTEgaW5zZXJ0aW9ucygrKSwgMiBkZWxldGlvbnMo LSkNCj4gPiA+ID4gPiANCj4gPiA+ID4gPiBkaWZmIC0tZ2l0IGEvdGVzdHMva21zX2F0b21pY190 cmFuc2l0aW9uLmMNCj4gPiA+ID4gPiBiL3Rlc3RzL2ttc19hdG9taWNfdHJhbnNpdGlvbi5jDQo+ ID4gPiA+ID4gaW5kZXggMTJiYWZlODcuLmUxYTJkNGEwIDEwMDY0NA0KPiA+ID4gPiA+IC0tLSBh L3Rlc3RzL2ttc19hdG9taWNfdHJhbnNpdGlvbi5jDQo+ID4gPiA+ID4gKysrIGIvdGVzdHMva21z X2F0b21pY190cmFuc2l0aW9uLmMNCj4gPiA+ID4gPiBAQCAtMTg3LDYgKzE4NywxMiBAQCBzdGF0 aWMgdm9pZCBzZXRfc3ByaXRlX3doKGlndF9kaXNwbGF5X3QNCj4gPiA+ID4gPiAqZGlzcGxheSwg ZW51bSBwaXBlIHBpcGUsDQo+ID4gPiA+ID4gIAkJICAgICAgTE9DQUxfRFJNX0ZPUk1BVF9NT0Rf Tk9ORSwgc3ByaXRlX2ZiKTsNCj4gPiA+ID4gPiAgfQ0KPiA+ID4gPiA+ICANCj4gPiA+ID4gPiAr I2RlZmluZSBpc19hdG9taWNfY2hlY2tfZmFpbHVyZV9lcnJubyhlcnJubykgXA0KPiA+ID4gPiA+ ICsJCShlcnJubyAhPSAtRUlOVkFMICYmIGVycm5vICE9IDApDQo+ID4gPiA+ID4gKw0KPiA+ID4g PiA+ICsjZGVmaW5lIGlzX2F0b21pY19jaGVja19wbGFuZV9zaXplX2Vycm5vKGVycm5vKSBcDQo+ ID4gPiA+ID4gKwkJKGVycm5vID09IC1FSU5WQUwpDQo+ID4gPiA+ID4gKw0KPiA+ID4gPiA+ICBz dGF0aWMgdm9pZCBzZXR1cF9wYXJtcyhpZ3RfZGlzcGxheV90ICpkaXNwbGF5LCBlbnVtIHBpcGUN Cj4gPiA+ID4gPiBwaXBlLA0KPiA+ID4gPiA+ICAJCQljb25zdCBkcm1Nb2RlTW9kZUluZm8gKm1v ZGUsDQo+ID4gPiA+ID4gIAkJCXN0cnVjdCBpZ3RfZmIgKnByaW1hcnlfZmIsDQo+ID4gPiA+ID4g QEAgLTI1MSw4ICsyNTcsOSBAQCByZXRyeToNCj4gPiA+ID4gPiAgDQo+ID4gPiA+ID4gIAkJd21f c2V0dXBfcGxhbmUoZGlzcGxheSwgcGlwZSwgKDEgPDwgbl9wbGFuZXMpIC0NCj4gPiA+ID4gPiAx LA0KPiA+ID4gPiA+IHBhcm1zLCBmYWxzZSk7DQo+ID4gPiA+ID4gIAkJcmV0ID0gaWd0X2Rpc3Bs YXlfdHJ5X2NvbW1pdF9hdG9taWMoZGlzcGxheSwNCj4gPiA+ID4gPiBEUk1fTU9ERV9BVE9NSUNf VEVTVF9PTkxZIHwgRFJNX01PREVfQVRPTUlDX0FMTE9XX01PREVTRVQsDQo+ID4gPiA+ID4gTlVM TCk7DQo+ID4gPiA+ID4gKwkJaWd0X2Fzc2VydCghaXNfYXRvbWljX2NoZWNrX2ZhaWx1cmVfZXJy bm8ocmV0KSkNCj4gPiA+ID4gPiA7DQo+ID4gPiA+ID4gIA0KPiA+ID4gPiA+IC0JCWlmIChyZXQg PT0gLUVJTlZBTCkgew0KPiA+ID4gPiA+ICsJCWlmIChpc19hdG9taWNfY2hlY2tfcGxhbmVfc2l6 ZV9lcnJubyhyZXQpKSB7DQo+ID4gPiA+ID4gIAkJCWlmIChjdXJzb3Jfd2lkdGggPT0gc3ByaXRl X3dpZHRoICYmDQo+ID4gPiA+ID4gIAkJCSAgICBjdXJzb3JfaGVpZ2h0ID09IHNwcml0ZV9oZWln aHQpIHsNCj4gPiA+ID4gPiAgCQkJCWlndF9hc3NlcnRfZihhbHBoYSwNCj4gPiA+ID4gPiBAQCAt NDQyLDcgKzQ0OSw5IEBAIHJ1bl90cmFuc2l0aW9uX3Rlc3QoaWd0X2Rpc3BsYXlfdA0KPiA+ID4g PiA+ICpkaXNwbGF5LA0KPiA+ID4gPiA+IGVudW0NCj4gPiA+ID4gPiBwaXBlIHBpcGUsIGlndF9v dXRwdXRfdCAqb3V0cHV0DQo+ID4gPiA+ID4gIAkJCWlndF9waXBlX3JlcXVlc3Rfb3V0X2ZlbmNl KHBpcGVfb2JqKTsNCj4gPiA+ID4gPiAgDQo+ID4gPiA+ID4gIAkJcmV0ID0gaWd0X2Rpc3BsYXlf dHJ5X2NvbW1pdF9hdG9taWMoZGlzcGxheSwNCj4gPiA+ID4gPiBEUk1fTU9ERV9BVE9NSUNfVEVT VF9PTkxZIHwgRFJNX01PREVfQVRPTUlDX0FMTE9XX01PREVTRVQsDQo+ID4gPiA+ID4gTlVMTCk7 DQo+ID4gPiA+ID4gLQkJaWYgKHJldCAhPSAtRUlOVkFMIHx8IHBpcGVfb2JqLT5uX3BsYW5lcyA8 IDMpDQo+ID4gPiA+ID4gKwkJaWd0X2Fzc2VydCghaXNfYXRvbWljX2NoZWNrX2ZhaWx1cmVfZXJy bm8ocmV0KSkNCj4gPiA+ID4gPiA7DQo+ID4gPiA+ID4gKw0KPiA+ID4gPiA+ICsJCWlmICghaXNf YXRvbWljX2NoZWNrX3BsYW5lX3NpemVfZXJybm8ocmV0KSB8fA0KPiA+ID4gPiA+IHBpcGVfb2Jq LQ0KPiA+ID4gPiA+ID4gbl9wbGFuZXMgPCAzKQ0KPiA+ID4gPiA+IA0KPiA+ID4gPiA+ICAJCQli cmVhazsNCj4gPiA+ID4gPiAgDQo+ID4gPiA+ID4gIAkJcmV0ID0gMDsNCj4gPiA+ID4gDQo+ID4g PiA+IExHVE0NCj4gPiA+ID4gDQo+ID4gPiA+IFJldmlld2VkLWJ5OiBTdHVhcnQgU3VtbWVycyA8 c3R1YXJ0LnN1bW1lcnNAaW50ZWwuY29tPg0KPiA+ID4gDQo+ID4gPiANCj4gPiA+IA0KPiA+ID4g PiBfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXw0KPiA+ID4g PiBpZ3QtZGV2IG1haWxpbmcgbGlzdA0KPiA+ID4gPiBpZ3QtZGV2QGxpc3RzLmZyZWVkZXNrdG9w Lm9yZw0KPiA+ID4gPiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3Rp bmZvL2lndC1kZXYNCj4gPiA+IA0KPiA+ID4gDQo+IA0KPiANCl9fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fCmlndC1kZXYgbWFpbGluZyBsaXN0CmlndC1kZXZA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vaWd0LWRldg==