From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by gabe.freedesktop.org (Postfix) with ESMTPS id 22A126E07A for ; Wed, 13 Mar 2019 09:20:06 +0000 (UTC) Date: Wed, 13 Mar 2019 11:20:00 +0200 From: Petri Latvala Message-ID: <20190313092000.GN4038@platvala-desk.ger.corp.intel.com> References: <20190312112114.1979-1-petri.latvala@intel.com> <155239047742.30003.3351260138826147792@skylake-alporthouse-com> <20190312130004.GL4038@platvala-desk.ger.corp.intel.com> <155239645672.30003.10269825860038059931@skylake-alporthouse-com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <155239645672.30003.10269825860038059931@skylake-alporthouse-com> Subject: Re: [igt-dev] [PATCH i-g-t] lib/ioctl_wrappers: Assert that prime_handle_to_fd returns a valid fd 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: Chris Wilson Cc: igt-dev@lists.freedesktop.org List-ID: T24gVHVlLCBNYXIgMTIsIDIwMTkgYXQgMDE6MTQ6MTdQTSArMDAwMCwgQ2hyaXMgV2lsc29uIHdy b3RlOgo+IFF1b3RpbmcgUGV0cmkgTGF0dmFsYSAoMjAxOS0wMy0xMiAxMzowMDowNCkKPiA+IE9u IFR1ZSwgTWFyIDEyLCAyMDE5IGF0IDExOjM0OjM3QU0gKzAwMDAsIENocmlzIFdpbHNvbiB3cm90 ZToKPiA+ID4gUXVvdGluZyBQZXRyaSBMYXR2YWxhICgyMDE5LTAzLTEyIDExOjIxOjE0KQo+ID4g PiA+IElmIHRoZSBpb2N0bCBpcyBzdWNjZXNzZnVsLCB0aGUgcmV0dXJuZWQgZmQgc2hvdWxkIGJl IHZhbGlkLiBDaGVjawo+ID4gPiA+IHRoYXQgaXQgaXMsIHRodXMgYWxzbyBoZWxwaW5nIHN0YXRp YyBhbmFseXNpcyBpbiBhbG1vc3QgNzAgY2FsbCBzaXRlcy4KPiA+ID4gPiAKPiA+ID4gPiBTaWdu ZWQtb2ZmLWJ5OiBQZXRyaSBMYXR2YWxhIDxwZXRyaS5sYXR2YWxhQGludGVsLmNvbT4KPiA+ID4g PiAtLS0KPiA+ID4gPiAgbGliL2lvY3RsX3dyYXBwZXJzLmMgfCAxICsKPiA+ID4gPiAgMSBmaWxl IGNoYW5nZWQsIDEgaW5zZXJ0aW9uKCspCj4gPiA+ID4gCj4gPiA+ID4gZGlmZiAtLWdpdCBhL2xp Yi9pb2N0bF93cmFwcGVycy5jIGIvbGliL2lvY3RsX3dyYXBwZXJzLmMKPiA+ID4gPiBpbmRleCAz OTkyMGY4Ny4uMzE5NjllNzcgMTAwNjQ0Cj4gPiA+ID4gLS0tIGEvbGliL2lvY3RsX3dyYXBwZXJz LmMKPiA+ID4gPiArKysgYi9saWIvaW9jdGxfd3JhcHBlcnMuYwo+ID4gPiA+IEBAIC0xMzMyLDYg KzEzMzIsNyBAQCBpbnQgcHJpbWVfaGFuZGxlX3RvX2ZkKGludCBmZCwgdWludDMyX3QgaGFuZGxl KQo+ID4gPiA+ICAgICAgICAgYXJncy5mZCA9IC0xOwo+ID4gPiA+ICAKPiA+ID4gPiAgICAgICAg IGRvX2lvY3RsKGZkLCBEUk1fSU9DVExfUFJJTUVfSEFORExFX1RPX0ZELCAmYXJncyk7Cj4gPiA+ ID4gKyAgICAgICBpZ3RfYXNzZXJ0X2ZkKGFyZ3MuZmQpOwo+ID4gPiAKPiA+ID4gV2h5PyBXaHkg d291bGQgd2UgYmUgdXNpbmcgYSBzdGF0aWMgYW5hbHlzaXMgdG9vbCB0aGF0IGNvbXBsYWlucyBh Ym91dAo+ID4gPiBmZWVkaW5nIGludmFsaWQgZGF0YSB0byB0aGUga2VybmVsLCBiZWNhdXNlIHRo YXQgaXMgZXhhY3RseSB0aGUgZ29hbCBvZgo+ID4gPiBpZ3QuCj4gPiAKPiA+IEJlY2F1c2UgYWZ0 ZXIgdGVhY2hpbmcgaXQgd2hhdCBkYXRhIHNob3VsZCBiZSBpbnZhbGlkIGFuZCB3aGF0IHNob3Vs ZAo+ID4gYmUgdmFsaWQsIGl0IGhhcyBhbHJlYWR5IHJldmVhbGVkIGEgY291cGxlIG9mIGJ1Z3M/ Cj4gCj4gRGlkIHRoZSBrZXJuZWwgaGFuZGxlIHRoZSBpbnZhbGlkIGZkIGNvcnJlY3RseT8gVGhh dCBpcyB3aGF0IGlzCj4gaW1wb3J0YW50LiBJdCdzIG5vdCBhIGJ1ZyBmb3IgaWd0IHRvIGNhbGwg Y2xvc2UoLTEpIG9yIHdoYXRldmVyLgoKQnV0IGl0IGlzIGEgYnVnIGluIElHVCBpZiB5b3UgY2Fs bCBhIGZ1bmN0aW9uIGZvbygpIGFuZCBleHBlY3QgaXQgdG8KZG8gZXJyb3IgaGFuZGxpbmcgZm9y IHlvdSwgYW5kIGl0IGdpdmluZyB5b3Ugc3R1ZmYgeW91J2Qgc3RpbGwgbmVlZCB0bwpjaGVjayB5 b3Vyc2VsZi4KCkltYWdpbmUgUFJJTUVfSEFORExFX1RPX0ZEIGlvY3RsIGJlaW5nIGJyb2tlbiBh bmQgZ2l2aW5nIG91dCBmZCAtNS4gSXQKd291bGQgc2hvdyB1cCBpbiB0ZXN0IHJlc3VsdHMsIGJ1 dCBwb2ludGluZyB0byBmdXJ0aGVyIGlvY3RscyBmYWlsaW5nCndpdGggRUlOVkFMLgoKPiA+IEFy ZSB5b3Ugb2JqZWN0aW5nIHRvIGNoZWNraW5nIHRoYXQgdGhlIGlvY3RsIGdpdmVzIGFuIGZkIHRo YXQgaXMgPj0gMD8KPiA+IFdoZXJlJ3MgdGhlIGxpbmUgaGVyZSwgd2hhdCBwYXJ0cyBvZiB0aGUg dWFwaSBzZW1hbnRpY3MgbXVzdCBhbHdheXMgYmUKPiA+IGxlZnQgdW5jaGVja2VkIGluIGEgdGVz dCBzdWl0ZSBmb3IgdWFwaT8KPiAKPiBObywgbXkgb3BpbmlvbiBpcyBodGF0IHRoaXMgYXNzZXJ0 aW9uIGlzIG9ubHkgdmFsaWQgaW4gYSB0ZXN0IG5vdCBsaWJyYXJ5Cj4gY29kZSwgYXMgdGhlIGxp YnJhcnkgY29kZSBzaG91bGQganVzdCBiZSBhIGNvbmR1aXQgYW5kIGV2ZW4gdGhlIGRvX2lvY3Rs Cj4gaXMgYSBtaXN0YWtlIGFzIHRoZXJlIGlzIG5vIGhpZ2hlciBsZXZlbCB3cmFwcGVyIGhlcmUu CgpJcyB0aGF0IHRlc3RlZCBpbiBhIHRlc3QgY3VycmVudGx5LCBJIGRpZG4ndCBzZWFyY2ggdGhv cm91Z2hseT8gQWxsCmNhbGwgc2l0ZXMgSSBjaGVja2VkIGp1c3QgdXNlIHRoZSByZXR1cm5lZCBm ZCBhcyBpdCB3YXMgdmFsaWQuCgpMaWJyYXJ5IGNvZGUgc2hvdWxkIGp1c3QgYmUgYSBjb25kdWl0 LCBzdXJlLiBCdXQgaXQncyBhbHNvIGEgdmFsaWQKZXhwZWN0YXRpb24gdGhhdCBmdW5jdGlvbnMg dGhhdCBhcmUgc3VwcG9zZWQgdG8gYXV0b21hdGljYWxseQpmYWlsL3NraXAgZm9yIHlvdSBvbiBl cnJvcnMgZG8gdGhlbSBmb3IgYWxsIG9jY2FzaW9ucyB0aGF0IGNhbiBjYXVzZQp0aGUgbGlicmFy eSBjb2RlIHRvIGZhaWwgdGhlaXIgcG9zdGNvbmRpdGlvbiBvZiB2YWxpZCBkYXRhLiBJdCdzIHRo ZQp3aG9sZSByZWFzb24gd2UgaGF2ZSBmdW5jdGlvbnMgdHlwaWNhbGx5IGluIHBhaXJzIG9mIGZv bygpIGFuZApfX2ZvbygpLgoKSG93IGRvIHdlIGdvIGZvcndhcmQ/CgoxKSBTcGxhc2ggaW4gYW4g X19wcmltZV9oYW5kbGVfdG9fZmQoKSB0aGF0IHdpbGwgcmV0dXJuIHRoZSBmZApkaXJlY3RseSwg bWFrZSBwcmltZV9oYW5kbGVfdG9fZmQgY2FsbCB0aGF0IGFuZCBhc3NlcnQgdGhhdCB0aGUgaW9j dGwKc3VjY2VlZGVkIGFuZCB0aGUgZmQgaXMgPj0gMAoKMikgVXNlIGlndF9hc3N1bWUoKSBpbnN0 ZWFkIGZvciB0aGUgZmQsIHBvaW50IHRvIGEgdGVzdCB0aGF0IGNoZWNrcwp0aGUgdmFsaWRuZXNz IG9mIHRoZSBmZCBvbiBhIHN1Y2Nlc3NmdWwgaW9jdGwgaW4gdGhlIGNvbW1pdCBtZXNzYWdlCgoK LS0gClBldHJpIExhdHZhbGEKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX18KaWd0LWRldiBtYWlsaW5nIGxpc3QKaWd0LWRldkBsaXN0cy5mcmVlZGVza3RvcC5v cmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pZ3QtZGV2