From mboxrd@z Thu Jan 1 00:00:00 1970 From: Punit Agrawal Subject: Re: [For Xen-4.10 RFC PATCH 3/3] Prevent redundant icache flushes in populate_physmap() Date: Fri, 31 Mar 2017 14:53:55 +0100 Message-ID: <87zig1blho.fsf@e105922-lin.cambridge.arm.com> References: <20170331102424.11869-1-punit.agrawal@arm.com> <20170331102424.11869-4-punit.agrawal@arm.com> <20170331113430.zdqlsr6a2zbxdfbm@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20170331113430.zdqlsr6a2zbxdfbm@citrix.com> (Wei Liu's message of "Fri, 31 Mar 2017 12:34:30 +0100") List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Wei Liu Cc: sstabellini@kernel.org, George.Dunlap@eu.citrix.com, andrew.cooper3@citrix.com, tim@xen.org, xen-devel@lists.xen.org, julien.grall@arm.com, jbeulich@suse.com, ian.jackson@eu.citrix.com List-Id: xen-devel@lists.xenproject.org SGkgV2VpLAoKVGhhbmtzIGZvciB0YWtpbmcgYSBsb29rIGF0IHRoaXMgUkZDLiBSZXNwb25zZXMv cXVlc3Rpb25zIGJlbG93Li4uCgpXZWkgTGl1IDx3ZWkubGl1MkBjaXRyaXguY29tPiB3cml0ZXM6 Cgo+IE9uIEZyaSwgTWFyIDMxLCAyMDE3IGF0IDExOjI0OjI0QU0gKzAxMDAsIFB1bml0IEFncmF3 YWwgd3JvdGU6Cj4+IHBvcHVsYXRlX3BoeXNtYXAoKSBjYWxscyBhbGxvY19oZWFwX3BhZ2VzKCkg cGVyIHJlcXVlc3RlZCBleHRlbnQuIEFzCj4+IGFsbG9jX2hlYXBfcGFnZXMoKSBwZXJmb3JtcyBp Y2FjaGUgbWFpbnRlbmFuY2Ugb3BlcmF0aW9ucyBhZmZlY3RpbmcgdGhlCj4+IGVudGlyZSBpbnN0 cnVjdGlvbiBjYWNoZSwgdGhpcyBsZWFkcyB0byByZWR1bmRhbnQgY2FjaGUgZmx1c2hlcyB3aGVu Cj4+IGFsbG9jYXRpbmcgbXVsdGlwbGUgZXh0ZW50cyBpbiBwb3B1bGF0ZV9waHlzbWFwKCkuCj4+ IAo+PiBUbyBhbGxldmlhdGUgdGhpcyBwcm9ibGVtLCBpbnRyb2R1Y2UgYSBuZXcgZmxhZyAiTUVN Rl9ub19pY2FjaGVfZmx1c2giCj4+IHdoaWNoIGNhbiBiZSB1c2VkIHByZXZlbnQgYWxsb2NfaGVh cF9wYWdlcygpIHRvIHBlcmZvcm0gdW5uZWNlc3NhcnkKPj4gaWNhY2hlIG1haW50ZW5hbmNlIG9w ZXJhdGlvbnMuIFVzZSB0aGUgZmxhZyBpbiBwb3B1bGF0ZV9waHlzbWFwKCkgYW5kCj4+IHBlcmZv cm0gdGhlIHJlcXVpcmVkIGljYWNoZSBtYWludGVuYW5jZSBmdW5jdGlvbiBhdCB0aGUgZW5kIG9m IHRoZQo+PiBvcGVyYXRpb24uCj4+IAo+PiBTaWduZWQtb2ZmLWJ5OiBQdW5pdCBBZ3Jhd2FsIDxw dW5pdC5hZ3Jhd2FsQGFybS5jb20+Cj4+IC0tLQo+PiAgeGVuL2NvbW1vbi9tZW1vcnkuYyAgICAg ICAgfCA2ICsrKysrKwo+PiAgeGVuL2NvbW1vbi9wYWdlX2FsbG9jLmMgICAgfCAyICstCj4+ICB4 ZW4vaW5jbHVkZS9hc20teDg2L3BhZ2UuaCB8IDQgKysrKwo+PiAgeGVuL2luY2x1ZGUveGVuL21t LmggICAgICAgfCAyICsrCj4+ICA0IGZpbGVzIGNoYW5nZWQsIDEzIGluc2VydGlvbnMoKyksIDEg ZGVsZXRpb24oLSkKPj4gCj4+IGRpZmYgLS1naXQgYS94ZW4vY29tbW9uL21lbW9yeS5jIGIveGVu L2NvbW1vbi9tZW1vcnkuYwo+PiBpbmRleCBhZDBiMzNjZWI2Li41MDdmMzYzOTI0IDEwMDY0NAo+ PiAtLS0gYS94ZW4vY29tbW9uL21lbW9yeS5jCj4+ICsrKyBiL3hlbi9jb21tb24vbWVtb3J5LmMK Pj4gQEAgLTE2Miw2ICsxNjIsOCBAQCBzdGF0aWMgdm9pZCBwb3B1bGF0ZV9waHlzbWFwKHN0cnVj dCBtZW1vcF9hcmdzICphKQo+PiAgICAgIGlmICggdW5saWtlbHkoIWQtPmNyZWF0aW9uX2Zpbmlz aGVkKSApCj4+ICAgICAgICAgIGEtPm1lbWZsYWdzIHw9IE1FTUZfbm9fdGxiZmx1c2g7Cj4+ICAK Pj4gKyAgICBhLT5tZW1mbGFncyB8PSBNRU1GX25vX2ljYWNoZV9mbHVzaDsKPj4gKwo+Cj4gTm90 IGEgZ29vZCBpZGVhLgo+Cj4gSSB0aGluayB3aGF0IHlvdSBuZWVkIGlzIHByb2JhYmx5IHRvIGdy b3VwIHRoaXMgdW5kZXIKPiAhZC0+Y3JlYXRpb25fZmluaXNoZWQuCgpDYW4geW91IGV4cGxhaW4g d2h5IHlvdSB0aGluayB0aGUgYWJvdmUgaXMgbm90IGEgZ29vZCBpZGVhPwoKV2l0aG91dCBhZGRp dGlvbmFsIHN5bmNocm9uaXNhdGlvbiB0aGVyZSBpcyBhIHJhY2UgYmV0d2VlbgpkLT5jcmVhdGlv bl9maW5pc2hlZCBiZWluZyBzZXQgYW5kIHVzZWQgKENvbnNpZGVyIHRoZSBjYXNlIHdoZXJlCnBv cHVsYXRlX3BoeXNtYXAoKSBiZWluZyBjYWxsZWQgb24gYSBkaWZmZXJlbnQgY3B1IHRvIHdoZXJl CmQtPmNyZWF0aW9uX2ZpbmlzaGVkIGlzIHNldCB0byB0cnVlKS4KCj4KPj4gICAgICBmb3IgKCBp ID0gYS0+bnJfZG9uZTsgaSA8IGEtPm5yX2V4dGVudHM7IGkrKyApCj4+ICAgICAgewo+PiAgICAg ICAgICBpZiAoIGkgIT0gYS0+bnJfZG9uZSAmJiBoeXBlcmNhbGxfcHJlZW1wdF9jaGVjaygpICkK Pj4gQEAgLTI1Myw2ICsyNTUsMTAgQEAgc3RhdGljIHZvaWQgcG9wdWxhdGVfcGh5c21hcChzdHJ1 Y3QgbWVtb3BfYXJncyAqYSkKPj4gIG91dDoKPj4gICAgICBpZiAoIG5lZWRfdGxiZmx1c2ggKQo+ PiAgICAgICAgICBmaWx0ZXJlZF9mbHVzaF90bGJfbWFzayh0bGJmbHVzaF90aW1lc3RhbXApOwo+ PiArCj4+ICsgICAgaWYgKCBhLT5tZW1mbGFncyAmIE1FTUZfbm9faWNhY2hlX2ZsdXNoICkKPgo+ IFRoaXMgc2hvdWxkIGJlIGludmVydGVkLCByaWdodD8KClRoZSBNRU1GX25vX2ljYWNoZV9mbHVz aCBmbGFnIGlzIHVzZWQgdG8gaW5kaWNhdGUgdG8gYWxsb2NfaGVhcF9wYWdlcygpCihjYWxsZWQg dmlhIGFsbG9jX2RvbWhlYXBfcGFnZXMoKSBoZXJlKSB0aGF0IHRoZSBjYWxsZXIgaXMgZ29pbmcg dG8gdGFrZQpjYXJlIG9mIHN5bmNocm9uaXNpbmcgdGhlIGljYWNoZSB3aXRoIHRoZSBkY2FjaGUu IEFzIHN1Y2gsIHdoZW4gdGhlIGZsYWcKaXMgc2V0LCB3ZSB3YW50IHRvIHBlcmZvcm0gaWNhY2hl IG1haW50ZW5hbmNlIG9wZXJhdGlvbnMgaGVyZS4KCj4KPiBUaGUgbW9kaWZpY2F0aW9uIHRvIHRo aXMgZnVuY3Rpb24gaXMgZGVmaW5pdGVseSB3cm9uZyBhcy1pcy4gWW91IGVuZCB1cAo+IGFsd2F5 cyBzZXR0aW5nIHRoZSBubyBmbHVzaCBmbGFnIGFuZCBsYXRlciBhbHdheXMgZmx1c2hpbmcgdGhl IGNhY2hlLgoKQ29ycmVjdCEKCmludmFsaWRhdGVfaWNhY2hlKCkgZmx1c2hlcyB0aGUgZW50aXJl IGluc3RydWN0aW9uIGNhY2hlIHdoaWNoIGVuZHMgdXAKYmVpbmcgY2FsbGVkIGVhY2ggdGltZSBm bHVzaF9wYWdlX3RvX3JhbSgpIGlzIGludm9rZWQgZnJvbQphbGxvY19oZWFwX3BhZ2VzKCkuIFRo ZSBwYXRjaCBwcmV2ZW50cyByZXBlYXRlZCBjYWxscyB0bwppbnZhbGlkYXRlX2ljYWNoZSgpIGFu ZCBtb3ZlcyB0aGUgcmVzcG9uc2liaWxpdHkgdG8gcG9wdWxhdGVfcGh5c21hcCgpCndoaWNoIHdh cyB0aGUgaW50ZW50aW9uLgoKSXQgd291bGQgaGVscCBpZiB5b3UgY2FuIGV4cGxhaW4gd2h5IHlv dSB0aGluayB0aGUgY2hhbmdlcyBhcmUKd3JvbmcuCgpUaGFua3MsClB1bml0Cgo+Cj4KPiBXZWku CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpYZW4tZGV2 ZWwgbWFpbGluZyBsaXN0Clhlbi1kZXZlbEBsaXN0cy54ZW4ub3JnCmh0dHBzOi8vbGlzdHMueGVu Lm9yZy94ZW4tZGV2ZWwK