From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Durrant Subject: Re: [PATCH v8 01/11] x86/hvm/ioreq: maintain an array of ioreq servers rather than a list Date: Wed, 4 Oct 2017 10:20:20 +0000 Message-ID: <950c561c37c541dbba5ffce6f0066a41@AMSPEX02CL03.citrite.net> References: <20170929145157.30180-1-paul.durrant@citrix.com> <20170929145157.30180-2-paul.durrant@citrix.com> <05a104de-8f87-8904-66cd-4bc8fa29bd64@citrix.com> <67f0b421bac34414ab40ca1ea4a64719@AMSPEX02CL03.citrite.net> <59D4D10A0200007800182016@prv-mh.provo.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dzgnM-00027P-LL for xen-devel@lists.xenproject.org; Wed, 04 Oct 2017 10:20:52 +0000 In-Reply-To: <59D4D10A0200007800182016@prv-mh.provo.novell.com> Content-Language: en-US List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: 'Jan Beulich' , Andrew Cooper Cc: "xen-devel@lists.xenproject.org" List-Id: xen-devel@lists.xenproject.org PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBKYW4gQmV1bGljaCBbbWFpbHRv OkpCZXVsaWNoQHN1c2UuY29tXQ0KPiBTZW50OiAwNCBPY3RvYmVyIDIwMTcgMTE6MTYNCj4gVG86 IEFuZHJldyBDb29wZXIgPEFuZHJldy5Db29wZXIzQGNpdHJpeC5jb20+OyBQYXVsIER1cnJhbnQN Cj4gPFBhdWwuRHVycmFudEBjaXRyaXguY29tPg0KPiBDYzogeGVuLWRldmVsQGxpc3RzLnhlbnBy b2plY3Qub3JnDQo+IFN1YmplY3Q6IFJFOiBbUEFUQ0ggdjggMDEvMTFdIHg4Ni9odm0vaW9yZXE6 IG1haW50YWluIGFuIGFycmF5IG9mIGlvcmVxDQo+IHNlcnZlcnMgcmF0aGVyIHRoYW4gYSBsaXN0 DQo+IA0KPiA+Pj4gT24gMjkuMDkuMTcgYXQgMTc6MzgsIDxQYXVsLkR1cnJhbnRAY2l0cml4LmNv bT4gd3JvdGU6DQo+ID4+ICAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiA+PiBGcm9tOiBB bmRyZXcgQ29vcGVyDQo+ID4+IFNlbnQ6IDI5IFNlcHRlbWJlciAyMDE3IDE2OjM1DQo+ID4+IFRv OiBQYXVsIER1cnJhbnQgPFBhdWwuRHVycmFudEBjaXRyaXguY29tPjsgeGVuLQ0KPiBkZXZlbEBs aXN0cy54ZW5wcm9qZWN0Lm9yZw0KPiA+PiBDYzogSmFuIEJldWxpY2ggPGpiZXVsaWNoQHN1c2Uu Y29tPg0KPiA+PiBTdWJqZWN0OiBSZTogW1BBVENIIHY4IDAxLzExXSB4ODYvaHZtL2lvcmVxOiBt YWludGFpbiBhbiBhcnJheSBvZiBpb3JlcQ0KPiA+PiBzZXJ2ZXJzIHJhdGhlciB0aGFuIGEgbGlz dA0KPiA+Pg0KPiA+PiBPbiAyOS8wOS8xNyAxNTo1MSwgUGF1bCBEdXJyYW50IHdyb3RlOg0KPiA+ PiA+IEEgc3Vic2VxdWVudCBwYXRjaCB3aWxsIHJlbW92ZSB0aGUgY3VycmVudCBpbXBsaWNpdCBs aW1pdGF0aW9uIG9uDQo+IGNyZWF0aW9uDQo+ID4+ID4gb2YgaW9yZXEgc2VydmVycyB3aGljaCBp cyBkdWUgdG8gdGhlIGFsbG9jYXRpb24gb2YgZ2ZucyBmb3IgdGhlIGlvcmVxDQo+ID4+ID4gc3Ry dWN0dXJlcyBhbmQgYnVmZmVyZWQgaW9yZXEgcmluZy4NCj4gPj4gPg0KPiA+PiA+IEl0IHdpbGwg dGhlcmVmb3JlIGJlIG5lY2Vzc2FyeSB0byBpbnRyb2R1Y2UgYW4gZXhwbGljaXQgbGltaXQgYW5k LCBzaW5jZQ0KPiA+PiA+IHRoaXMgbGltaXQgc2hvdWxkIGJlIHNtYWxsLCBpdCBzaW1wbGlmaWVz IHRoZSBjb2RlIHRvIG1haW50YWluIGFuIGFycmF5IG9mDQo+ID4+ID4gdGhhdCBzaXplIHJhdGhl ciB0aGFuIHVzaW5nIGEgbGlzdC4NCj4gPj4gPg0KPiA+PiA+IEFsc28sIGJ5IHJlc2VydmluZyBh biBhcnJheSBzbG90IGZvciB0aGUgZGVmYXVsdCBzZXJ2ZXIgYW5kIHBvcHVsYXRpbmcNCj4gPj4g PiBhcnJheSBzbG90cyBlYXJseSBpbiBjcmVhdGUsIHRoZSBuZWVkIHRvIHBhc3MgYW4gJ2lzX2Rl ZmF1bHQnIGJvb2xlYW4NCj4gPj4gPiB0byBzdWItZnVuY3Rpb25zIGNhbiBiZSBhdm9pZGVkLg0K PiA+PiA+DQo+ID4+ID4gU29tZSBmdW5jdGlvbiByZXR1cm4gdmFsdWVzIGFyZSBjaGFuZ2VkIGJ5 IHRoaXMgcGF0Y2g6IFNwZWNpZmljYWxseSwgaW4NCj4gPj4gPiB0aGUgY2FzZSB3aGVyZSB0aGUg aWQgb2YgdGhlIGRlZmF1bHQgaW9yZXEgc2VydmVyIGlzIHBhc3NlZCBpbiwgLQ0KPiA+PiBFT1BO T1RTVVBQDQo+ID4+ID4gaXMgbm93IHJldHVybmVkIHJhdGhlciB0aGFuIC1FTk9FTlQuDQo+ID4+ ID4NCj4gPj4gPiBTaWduZWQtb2ZmLWJ5OiBQYXVsIER1cnJhbnQgPHBhdWwuZHVycmFudEBjaXRy aXguY29tPg0KPiA+PiA+IFJldmlld2VkLWJ5OiBSb2dlciBQYXUgTW9ubsOpIDxyb2dlci5wYXVA Y2l0cml4LmNvbT4NCj4gPj4gPiAtLS0NCj4gPj4gPiBDYzogSmFuIEJldWxpY2ggPGpiZXVsaWNo QHN1c2UuY29tPg0KPiA+PiA+IENjOiBBbmRyZXcgQ29vcGVyIDxhbmRyZXcuY29vcGVyM0BjaXRy aXguY29tPg0KPiA+PiA+DQo+ID4+ID4gdjg6DQo+ID4+ID4gIC0gQWRkcmVzc2VkIHZhcmlvdXMg Y29tbWVudHMgZnJvbSBKYW4uDQo+ID4+ID4NCj4gPj4gPiB2NzoNCj4gPj4gPiAgLSBGaXhlZCBh c3NlcnRpb24gZmFpbHVyZSBmb3VuZCBpbiB0ZXN0aW5nLg0KPiA+PiA+DQo+ID4+ID4gdjY6DQo+ ID4+ID4gIC0gVXBkYXRlZCBhY2NvcmRpbmcgdG8gY29tbWVudHMgbWFkZSBieSBSb2dlciBvbiB2 NCB0aGF0IEknZCBtaXNzZWQuDQo+ID4+ID4NCj4gPj4gPiB2NToNCj4gPj4gPiAgLSBTd2l0Y2hl ZCBHRVQvU0VUX0lPUkVRX1NFUlZFUigpIG1hY3JvcyB0byBnZXQvc2V0X2lvcmVxX3NlcnZlcigp DQo+ID4+ID4gICAgZnVuY3Rpb25zIHRvIGF2b2lkIHBvc3NpYmxlIGRvdWJsZS1ldmFsdWF0aW9u IGlzc3Vlcy4NCj4gPj4gPg0KPiA+PiA+IHY0Og0KPiA+PiA+ICAtIEludHJvZHVjZWQgbW9yZSBo ZWxwZXIgbWFjcm9zIGFuZCByZWxvY2F0ZWQgdGhlbSB0byB0aGUgdG9wIG9mIHRoZQ0KPiA+PiA+ ICAgIGNvZGUuDQo+ID4+ID4NCj4gPj4gPiB2MzoNCj4gPj4gPiAgLSBOZXcgcGF0Y2ggKHJlcGxh Y2luZyAibW92ZSBpc19kZWZhdWx0IGludG8gc3RydWN0DQo+IGh2bV9pb3JlcV9zZXJ2ZXIiKSBp bg0KPiA+PiA+ICAgIHJlc3BvbnNlIHRvIHJldmlldyBjb21tZW50cy4NCj4gPj4gPiAtLS0NCj4g Pj4gPiAgeGVuL2FyY2gveDg2L2h2bS9pb3JlcS5jICAgICAgICAgfCA1MjUgKysrKysrKysrKysr KysrKysrKystLS0tLS0tLS0tDQo+IC0tLS0NCj4gPj4gLS0tLS0NCj4gPj4gPiAgeGVuL2luY2x1 ZGUvYXNtLXg4Ni9odm0vZG9tYWluLmggfCAgMTAgKy0NCj4gPj4gPiAgMiBmaWxlcyBjaGFuZ2Vk LCAyNzAgaW5zZXJ0aW9ucygrKSwgMjY1IGRlbGV0aW9ucygtKQ0KPiA+PiA+DQo+ID4+ID4gZGlm ZiAtLWdpdCBhL3hlbi9hcmNoL3g4Ni9odm0vaW9yZXEuYyBiL3hlbi9hcmNoL3g4Ni9odm0vaW9y ZXEuYw0KPiA+PiA+IGluZGV4IGYyZTBiM2Y3NGEuLmU2NTVkMmVhYjMgMTAwNjQ0DQo+ID4+ID4g LS0tIGEveGVuL2FyY2gveDg2L2h2bS9pb3JlcS5jDQo+ID4+ID4gKysrIGIveGVuL2FyY2gveDg2 L2h2bS9pb3JlcS5jDQo+ID4+ID4gQEAgLTMzLDYgKzMzLDQxIEBADQo+ID4+ID4NCj4gPj4gPiAg I2luY2x1ZGUgPHB1YmxpYy9odm0vaW9yZXEuaD4NCj4gPj4gPg0KPiA+PiA+ICtzdGF0aWMgdm9p ZCBzZXRfaW9yZXFfc2VydmVyKHN0cnVjdCBkb21haW4gKmQsIHVuc2lnbmVkIGludCBpZCwNCj4g Pj4gPiArICAgICAgICAgICAgICAgICAgICAgICAgICAgICBzdHJ1Y3QgaHZtX2lvcmVxX3NlcnZl ciAqcykNCj4gPj4gPiArew0KPiA+PiA+ICsgICAgQVNTRVJUKGlkIDwgTUFYX05SX0lPUkVRX1NF UlZFUlMpOw0KPiA+PiA+ICsgICAgQVNTRVJUKCFzIHx8ICFkLT5hcmNoLmh2bV9kb21haW4uaW9y ZXFfc2VydmVyLnNlcnZlcltpZF0pOw0KPiA+PiA+ICsNCj4gPj4gPiArICAgIGQtPmFyY2guaHZt X2RvbWFpbi5pb3JlcV9zZXJ2ZXIuc2VydmVyW2lkXSA9IHM7DQo+ID4+ID4gK30NCj4gPj4gPiAr DQo+ID4+ID4gKyNkZWZpbmUgR0VUX0lPUkVRX1NFUlZFUihkLCBpZCkgXA0KPiA+PiA+ICsgICAg KGQpLT5hcmNoLmh2bV9kb21haW4uaW9yZXFfc2VydmVyLnNlcnZlcltpZF0NCj4gPj4gPiArDQo+ ID4+ID4gK3N0YXRpYyBzdHJ1Y3QgaHZtX2lvcmVxX3NlcnZlciAqZ2V0X2lvcmVxX3NlcnZlcihj b25zdCBzdHJ1Y3QgZG9tYWluDQo+ID4+ICpkLA0KPiA+PiA+ICsgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgdW5zaWduZWQgaW50IGlkKQ0KPiA+PiA+ICt7 DQo+ID4+ID4gKyAgICBpZiAoIGlkID49IE1BWF9OUl9JT1JFUV9TRVJWRVJTICkNCj4gPj4gPiAr ICAgICAgICByZXR1cm4gTlVMTDsNCj4gPj4gPiArDQo+ID4+ID4gKyAgICByZXR1cm4gR0VUX0lP UkVRX1NFUlZFUihkLCBpZCk7DQo+ID4+ID4gK30NCj4gPj4gPiArDQo+ID4+ID4gKyNkZWZpbmUg SVNfREVGQVVMVChzKSBcDQo+ID4+ID4gKyAgICAoKHMpID09IGdldF9pb3JlcV9zZXJ2ZXIoKHMp LT5kb21haW4sIERFRkFVTFRfSU9TRVJWSUQpKQ0KPiA+PiA+ICsNCj4gPj4gPiArLyoNCj4gPj4g PiArICogSXRlcmF0ZSBvdmVyIGFsbCBwb3NzaWJsZSBpb3JlcSBzZXJ2ZXJzLiBUaGUgdXNlIG9m IGlubGluZSBmdW5jdGlvbg0KPiA+PiA+ICsgKiBnZXRfaW9yZXFfc2VydmVyKCkgaW4gdGhlIGlu Y3JlbWVudCBpcyBkZWxpYmVyYXRlIGFzIHVzZSBvZiB0aGUNCj4gPj4gPiArICogR0VUX0lPUkVR X1NFUlZFUigpIG1hY3JvIHdpbGwgY2F1c2UgZ2NjIHRvIGNvbXBsYWluIGFib3V0IGFuDQo+IGFy cmF5DQo+ID4+ID4gKyAqIG92ZXJmbG93Lg0KPiA+PiA+ICsgKi8NCj4gPj4gPiArI2RlZmluZSBG T1JfRUFDSF9JT1JFUV9TRVJWRVIoZCwgaWQsIHMpIFwNCj4gPj4gPiArICAgIGZvciAoIChpZCkg PSAwLCAocykgPSBHRVRfSU9SRVFfU0VSVkVSKGQsIDApOyBcDQo+ID4+ID4gKyAgICAgICAgICAo aWQpIDwgTUFYX05SX0lPUkVRX1NFUlZFUlM7IFwNCj4gPj4gPiArICAgICAgICAgIChzKSA9IGdl dF9pb3JlcV9zZXJ2ZXIoZCwgKysoaWQpKSApDQo+ID4+DQo+ID4+IEknbSBndWVzc2luZyBmcm9t IHRoZSB2YXJpb3VzIGNvbnN0cnVjdHMsIHRoZSBsaXN0IG9mIGlvcmVxIHNlcnZlcnMNCj4gPj4g bWlnaHQgaGF2ZSBlbWJlZGRlZCBOVUxMcyBpbiB0aGUgbWlkZGxlPw0KPiA+Pg0KPiA+PiBJZiBz bywgaG93IGFib3V0IHRoaXM/DQo+ID4+DQo+ID4+ICNkZWZpbmUgRk9SX0VBQ0hfSU9SRVFfU0VS VkVSKGQsIGlkLCBzKSBcDQo+ID4+ICAgICBmb3IgKCAoaWQpID0gMCwgKHMpID0gR0VUX0lPUkVR X1NFUlZFUihkLCAwKTsgXA0KPiA+PiAgICAgICAgICAgKGlkKSA8IE1BWF9OUl9JT1JFUV9TRVJW RVJTOyBcDQo+ID4+ICAgICAgICAgICAocykgPSBnZXRfaW9yZXFfc2VydmVyKGQsICsrKGlkKSkg KSBcDQo+ID4+ICAgICBpZiAoICFzICkgXA0KPiA+PiAgICAgICAgIGNvbnRpbnVlOyBcDQo+ID4+ ICAgICBlbHNlDQo+ID4NCj4gPiBJJ20gb2sgd2l0aCBpdCBidXQgSSdsbCB3YWl0IGZvciBvdGhl cnMgb3BpbmlvbiBvbiB3aGV0aGVyIHRoaXMgaXMgdGFraW5nDQo+ID4gdGhlIG1hY3JvIG1hZ2lj IHRvbyBmYXIuDQo+IA0KPiBJJ20gZmluZSB3aXRoIEFuZHJldydzIHN1Z2dlc3Rpb247IEknbSBz dXJwcmlzZWQgdGhvdWdoIHRyaWNrZXJ5DQo+IGxpa2UgdGhpcyBpcyBiZWluZyBzdWdnZXN0ZWQg YXQgYWxsLCBhcyBjb21tb25seSBnYW1lcyBJIGhhcHBlbg0KPiB0byBiZSB0cnlpbmcgdG8gcGxh eSBvbmNlIGluIGEgd2hpbGUgZG9uJ3Qgc2VlbSB0byBiZSByZWFsbHkgbGlrZWQuDQo+IA0KDQpZ ZWFoLCBJJ20gYWx3YXlzIG5lcnZvdXMgb2YgbWFjcm9zIGdldHRpbmcgdG9vIGxhcmdlLi4uIG1h aW5seSBiZWNhdXNlIHRoZXkgZW5kIHVwIGJlaW5nIGEgcGFpbiB0byBkZWJ1ZyBpbiBteSBleHBl cmllbmNlLCBidXQgSSdsbCByZS1kbyB0aGUgbWFjcm8gYXMgQW5kcmV3IHN1Z2dlc3RzLg0KDQog IFBhdWwNCg0KPiBKYW4NCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fClhlbi1kZXZlbCBtYWlsaW5nIGxpc3QKWGVuLWRldmVsQGxpc3RzLnhlbi5vcmcKaHR0 cHM6Ly9saXN0cy54ZW4ub3JnL3hlbi1kZXZlbAo=