From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wei Chen Subject: Re: [for-4.7 2/2] xen/arm: p2m: Release the p2m lock before undoing the mappings Date: Tue, 17 May 2016 14:21:42 +0800 Message-ID: References: <1463407735-7409-1-git-send-email-julien.grall@arm.com> <1463407735-7409-3-git-send-email-julien.grall@arm.com> <45f975ea-e916-4066-03b7-8fac4225bda7@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <45f975ea-e916-4066-03b7-8fac4225bda7@arm.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Julien Grall , xen-devel@lists.xen.org Cc: sstabellini@kernel.org, wei.liu2@citrix.com List-Id: xen-devel@lists.xenproject.org SGkgSnVsaWVuLAoKSSBoYXZlIHNvbWUgY29uY2VybiBhYm91dCB0aGlzIHBhdGNoLiBCZWNhdXNl IHdlIHJlbGVhc2VkIHRoZSBzcGlubG9jawpiZWZvcmUgcmVtb3ZlIHRoZSBtYXBwZWQgbWVtb3J5 LiBJZiBzb21lYm9keSBhY3F1aXJlcyB0aGUgc3BpbmxvY2sKYmVmb3JlIHdlIHJlbW92ZSB0aGUg bWFwcGVkIG1lbW9yeSwgdGhpcyBtYXBwZWQgbWVtb3J5IHJlZ2lvbiBjYW4gYmUKYWNjZXNzZWQg YnkgZ3Vlc3QuCgpUaGUgYXBwbHlfcDJtX2NoYW5nZXMgaXMgbm8gbG9uZ2VyIGF0b21pYy4gSXMg aXQgYSBzZWN1cml0eSByaXNrPwoKT24gMjAxNi81LzE3IDEzOjQ1LCBXZWkgQ2hlbiB3cm90ZToK PiBIaSBKdWxpZW4sCj4KPiBUaGlzIGNvZGUgbG9va3MgZ29vZCB0byBtZSwgYW5kIEkgaGF2ZSB0 ZXN0ZWQgdGhhdAo+IHRoZSBkZWFkbG9jayBpcyBmaXhlZCBieSB0aGlzIHBhdGNoLgo+Cj4gUmV2 aWV3ZWQtYW5kLVRlc3RlZC1ieTogV2VpIENoZW4gPFdlaS5DaGVuQGFybS5jb20+Cj4KPiBPcmln aW5hbDoKPiAoWEVOKSBzbW11OiAvc21iL3NtbXVAZTA4MDAwMDA6IFAyTSBJUEEgc2l6ZSBub3Qg c3VwcG9ydGVkIChQMk09NDQKPiBTTU1VPTQwKSEKPiAoWEVOKSBJL08gdmlydHVhbGlzYXRpb24g ZGlzYWJsZWQKPiAoWEVOKSBSZXF1ZXN0IHAybSBTcGlubG9jay4uLgo+IChYRU4pIFJlcXVlc3Qg cDJtIFNwaW5sb2NrLi4uT0sKPiAoWEVOKSBhcHBseV9wMm1fY2hhbmdlcyBmYWlsZWQhIHJvbGxi YWNrISBjYWxsIGFwcGx5X3AybV9jaGFuZ2VzCj4gcmVjdXJzaXZlbHkhCj4gKFhFTikgUmVxdWVz dCBwMm0gU3BpbmxvY2suLi4KPgo+IFBhdGNoZWQ6Cj4gKFhFTikgc21tdTogL3NtYi9zbW11QGUw ODAwMDAwOiBQMk0gSVBBIHNpemUgbm90IHN1cHBvcnRlZCAoUDJNPTQ0Cj4gU01NVT00MCkhCj4g KFhFTikgSS9PIHZpcnR1YWxpc2F0aW9uIGRpc2FibGVkCj4gKFhFTikgUmVxdWVzdCBwMm0gU3Bp bmxvY2suLi4KPiAoWEVOKSBSZXF1ZXN0IHAybSBTcGlubG9jay4uLk9LCj4gKFhFTikgYXBwbHlf cDJtX2NoYW5nZXMgZmFpbGVkISByb2xsYmFjayEgY2FsbCBhcHBseV9wMm1fY2hhbmdlcwo+IHJl Y3Vyc2l2ZWx5IQo+IChYRU4pIFJlcXVlc3QgcDJtIFNwaW5sb2NrLi4uCj4gKFhFTikgUmVxdWVz dCBwMm0gU3BpbmxvY2suLi5PSwo+Cj4KPiBPbiAyMDE2LzUvMTYgMjI6MDgsIEp1bGllbiBHcmFs bCB3cm90ZToKPj4gU2luY2UgY29tbWl0IDRiMjU0MjNhICJhcmNoL2FybTogdW5tYXAgcGFydGlh bGx5LW1hcHBlZCBtZW1vcnkgcmVnaW9ucyIsCj4+IFhlbiBoYXMgYmVlbiB1bmRvaW5nIHRoZSBQ Mk0gbWFwcGluZ3Mgd2hlbiBhbiBlcnJvciBvY2N1cnJlZCBkdXJpbmcKPj4gaW5zZXJ0aW9uIG9y IG1lbW9yeSBhbGxvY2F0aW9uLgo+Pgo+PiBUaGlzIGlzIGRvbmUgYnkgY2FsbGluZyByZWN1cnNp dmVseSBhcHBseV9wMm1fY2hhbmdlcywgaG93ZXZlciB0aGUKPj4gc2Vjb25kIGNhbGwgaXMgZG9u ZSB3aXRoIHRoZSBwMm0gbG9jayB0YWtlbiB3aGljaCB3aWxsIHJlc3VsdCBpbiBhCj4+IGRlYWRs b2NrIGZvciB0aGUgY3VycmVudCBwcm9jZXNzb3IuCj4+Cj4+IEZpeCBpdCBieSBtb3ZpbmcgdGhl IHJlY3Vyc2l2ZSBjYWxsIGFmdGVyIHRoZSBsb2NrIGhhcyBiZWVuIHJlbGVhc2VkLgo+Pgo+PiBT aWduZWQtb2ZmLWJ5OiBKdWxpZW4gR3JhbGwgPGp1bGllbi5ncmFsbEBhcm0uY29tPgo+Pgo+PiAt LS0KPj4gICAgIEkgdGhpbmsgd2UgY291bGQgdW5sb2NrIHRoZSBwMm0gbG9jayBiZWZvcmUgZnJl ZWluZyB0aGUgdGVtcG9yYXJ5Cj4+ICAgICBtYXBwaW5nLiBBbHRob3VnaCwgSSBwbGF5ZWQgc2Fm ZSBhcyB0aGlzIGlzIGEgYnVnIGZpeCBmb3IgWGVuIDQuNwo+PiAgICAgYW5kIHRvIGJlIGJhY2tw b3J0ZWQgdXAgdG8gWGVuIDQuNS4KPj4gLS0tCj4+ICB4ZW4vYXJjaC9hcm0vcDJtLmMgfCAxNiAr KysrKysrKy0tLS0tLS0tCj4+ICAxIGZpbGUgY2hhbmdlZCwgOCBpbnNlcnRpb25zKCspLCA4IGRl bGV0aW9ucygtKQo+Pgo+PiBkaWZmIC0tZ2l0IGEveGVuL2FyY2gvYXJtL3AybS5jIGIveGVuL2Fy Y2gvYXJtL3AybS5jCj4+IGluZGV4IDY4YzY3YjAuLjgzOGQwMDQgMTAwNjQ0Cj4+IC0tLSBhL3hl bi9hcmNoL2FybS9wMm0uYwo+PiArKysgYi94ZW4vYXJjaC9hcm0vcDJtLmMKPj4gQEAgLTExODQs NiArMTE4NCwxNCBAQCBvdXQ6Cj4+ICAgICAgd2hpbGUgKCAocGcgPSBwYWdlX2xpc3RfcmVtb3Zl X2hlYWQoJmZyZWVfcGFnZXMpKSApCj4+ICAgICAgICAgIGZyZWVfZG9taGVhcF9wYWdlKHBnKTsK Pj4KPj4gKyAgICBmb3IgKCBsZXZlbCA9IFAyTV9ST09UX0xFVkVMOyBsZXZlbCA8IDQ7IGxldmVs ICsrICkKPj4gKyAgICB7Cj4+ICsgICAgICAgIGlmICggbWFwcGluZ3NbbGV2ZWxdICkKPj4gKyAg ICAgICAgICAgIHVubWFwX2RvbWFpbl9wYWdlKG1hcHBpbmdzW2xldmVsXSk7Cj4+ICsgICAgfQo+ PiArCj4+ICsgICAgc3Bpbl91bmxvY2soJnAybS0+bG9jayk7Cj4+ICsKPj4gICAgICBpZiAoIHJj IDwgMCAmJiAoIG9wID09IElOU0VSVCB8fCBvcCA9PSBBTExPQ0FURSApICYmCj4+ICAgICAgICAg ICBhZGRyICE9IHN0YXJ0X2dwYWRkciApCj4+ICAgICAgewo+PiBAQCAtMTE5NiwxNCArMTIwNCw2 IEBAIG91dDoKPj4gICAgICAgICAgICAgICAgICAgICAgICAgICAgbWF0dHIsIDAsIHAybV9pbnZh bGlkLAo+PiBkLT5hcmNoLnAybS5kZWZhdWx0X2FjY2Vzcyk7Cj4+ICAgICAgfQo+Pgo+PiAtICAg IGZvciAoIGxldmVsID0gUDJNX1JPT1RfTEVWRUw7IGxldmVsIDwgNDsgbGV2ZWwgKysgKQo+PiAt ICAgIHsKPj4gLSAgICAgICAgaWYgKCBtYXBwaW5nc1tsZXZlbF0gKQo+PiAtICAgICAgICAgICAg dW5tYXBfZG9tYWluX3BhZ2UobWFwcGluZ3NbbGV2ZWxdKTsKPj4gLSAgICB9Cj4+IC0KPj4gLSAg ICBzcGluX3VubG9jaygmcDJtLT5sb2NrKTsKPj4gLQo+PiAgICAgIHJldHVybiByYzsKPj4gIH0K Pj4KPj4KSU1QT1JUQU5UIE5PVElDRTogVGhlIGNvbnRlbnRzIG9mIHRoaXMgZW1haWwgYW5kIGFu eSBhdHRhY2htZW50cyBhcmUgY29uZmlkZW50aWFsIGFuZCBtYXkgYWxzbyBiZSBwcml2aWxlZ2Vk LiBJZiB5b3UgYXJlIG5vdCB0aGUgaW50ZW5kZWQgcmVjaXBpZW50LCBwbGVhc2Ugbm90aWZ5IHRo ZSBzZW5kZXIgaW1tZWRpYXRlbHkgYW5kIGRvIG5vdCBkaXNjbG9zZSB0aGUgY29udGVudHMgdG8g YW55IG90aGVyIHBlcnNvbiwgdXNlIGl0IGZvciBhbnkgcHVycG9zZSwgb3Igc3RvcmUgb3IgY29w eSB0aGUgaW5mb3JtYXRpb24gaW4gYW55IG1lZGl1bS4gVGhhbmsgeW91LgoKCl9fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fClhlbi1kZXZlbCBtYWlsaW5nIGxp c3QKWGVuLWRldmVsQGxpc3RzLnhlbi5vcmcKaHR0cDovL2xpc3RzLnhlbi5vcmcveGVuLWRldmVs Cg==