From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andre Przywara Subject: Re: [PATCH RFC] ARM: vPL011: use receive timeout interrupt Date: Mon, 23 Oct 2017 17:01:06 +0100 Message-ID: <7eb21b22-b702-d18b-aaa3-6d6b409f73b9@linaro.org> References: <8dc5e662-4cbd-908d-4f3e-af372d6d0917@arm.com> <20171018134158.32633-1-andre.przywara@arm.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 1e6f5n-0007af-Lu for xen-devel@lists.xenproject.org; Mon, 23 Oct 2017 15:56:43 +0000 Received: by mail-wr0-f196.google.com with SMTP id o44so17886435wrf.11 for ; Mon, 23 Oct 2017 08:56:42 -0700 (PDT) In-Reply-To: Content-Language: en-GB List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Bhupinder Thakur Cc: Xen-devel , Julien Grall , Stefano Stabellini , Dave Martin List-Id: xen-devel@lists.xenproject.org SGksCgpPbiAxOC8xMC8xNyAxNzozMiwgQmh1cGluZGVyIFRoYWt1ciB3cm90ZToKPiBIaSBBbmRy ZSwKPiAKPiBJIHZlcmlmaWVkIHRoaXMgcGF0Y2ggb24gcXVhbGNvbW0gcGxhdGZvcm0uIEl0IGlz IHdvcmtpbmcgZmluZS4KPiAKPiBPbiAxOCBPY3RvYmVyIDIwMTcgYXQgMTk6MTEsIEFuZHJlIFBy enl3YXJhIDxhbmRyZS5wcnp5d2FyYUBhcm0uY29tPiB3cm90ZToKPj4gSW5zdGVhZCBvZiBhc3Nl cnRpbmcgdGhlIHJlY2VpdmUgaW50ZXJydXB0IChSWEkpIG9uIHRoZSBmaXJzdCBjaGFyYWN0ZXIK Pj4gaW4gdGhlIEZJRk8sIGxldHMgKGFiKXVzZSB0aGUgcmVjZWl2ZSB0aW1lb3V0IGludGVycnVw dCAoUlRJKSBmb3IgdGhhdAo+PiBwdXJwb3NlLiBUaGF0IHNlZW1zIHRvIGJlIGNsb3NlciB0byB0 aGUgc3BlYyBhbmQgd2hhdCBoYXJkd2FyZSBkb2VzLgo+PiBJbXByb3ZlIHRoZSByZWFkYWJpbGl0 eSBvZiB2cGwwMTFfZGF0YV9hdmFpbCgpIG9uIHRoZSB3YXkuCj4+Cj4+IFNpZ25lZC1vZmYtYnk6 IEFuZHJlIFByenl3YXJhIDxhbmRyZS5wcnp5d2FyYUBhcm0uY29tPgo+PiAtLS0KPj4gSGksCj4+ Cj4+IHRoaXMgb25lIGlzIHRoZSBhcHByb2FjaCBJIG1lbnRpb25lZCBpbiB0aGUgZW1haWwgZWFy bGllciB0b2RheS4KPj4gSXQgZ29lcyBvbiB0b3Agb2YgQmh1cGluZGVycyB2MTIgMjcvMjcsIGJ1 dCBzaG91bGQgZXZlbnR1YWxseSBiZSBtZXJnZWQKPj4gaW50byB0aGlzIG9uZSBvbmNlIHdlIGFn cmVlZCBvbiB0aGUgc3ViamVjdC4gSSBqdXN0IGNhcnZlZCBpdCBvdXQgaGVyZQo+PiBmb3IgY2xh cml0eSB0byBtYWtlIGl0IGNsZWFyZXIgd2hhdCBoYXMgYmVlbiBjaGFuZ2VkLgo+PiBXb3VsZCBi ZSBnb29kIGlmIHNvbWVvbmUgY291bGQgdGVzdCBpdC4KPj4KPj4gQ2hlZXJzLAo+PiBBbmRyZS4K Pj4gIHhlbi9hcmNoL2FybS92cGwwMTEuYyB8IDYxICsrKysrKysrKysrKysrKysrKysrKysrKy0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQo+PiAgMSBmaWxlIGNoYW5nZWQsIDI5IGluc2VydGlv bnMoKyksIDMyIGRlbGV0aW9ucygtKQo+Pgo+PiBkaWZmIC0tZ2l0IGEveGVuL2FyY2gvYXJtL3Zw bDAxMS5jIGIveGVuL2FyY2gvYXJtL3ZwbDAxMS5jCj4+IGluZGV4IGFkZjE3MTE1NzEuLmFlMThi ZGRkODEgMTAwNjQ0Cj4+IC0tLSBhL3hlbi9hcmNoL2FybS92cGwwMTEuYwo+PiArKysgYi94ZW4v YXJjaC9hcm0vdnBsMDExLmMKPj4gQEAgLTEwNSw5ICsxMDUsMTMgQEAgc3RhdGljIHVpbnQ4X3Qg dnBsMDExX3JlYWRfZGF0YShzdHJ1Y3QgZG9tYWluICpkKQo+PiAgICAgICAgICBpZiAoIGZpZm9f bGV2ZWwgPT0gMCApCj4+ICAgICAgICAgIHsKPj4gICAgICAgICAgICAgIHZwbDAxMS0+dWFydGZy IHw9IFJYRkU7Cj4+IC0gICAgICAgICAgICB2cGwwMTEtPnVhcnRyaXMgJj0gflJYSTsKPj4gLSAg ICAgICAgICAgIHZwbDAxMV91cGRhdGVfaW50ZXJydXB0X3N0YXR1cyhkKTsKPj4gKyAgICAgICAg ICAgIHZwbDAxMS0+dWFydHJpcyAmPSB+UlRJOwo+PiAgICAgICAgICB9Cj4+ICsKPj4gKyAgICAg ICAgaWYgKCBmaWZvX2xldmVsIDwgc2l6ZW9mKGludGYtPmluKSAtIFNCU0FfVUFSVF9GSUZPX1NJ WkUgLyAyICkKPj4gKyAgICAgICAgICAgIHZwbDAxMS0+dWFydHJpcyAmPSB+UlhJOwo+PiArCj4+ ICsgICAgICAgIHZwbDAxMV91cGRhdGVfaW50ZXJydXB0X3N0YXR1cyhkKTsKPiBJIHRoaW5rIHdl IGNoZWNrIGlmICggZmlmb19sZXZlbCA8IFNCU0FfVUFSVF9GSUZPX1NJWkUgLyAyICkgd2hpY2gK PiBzaG91bGQgYmUgYSB2YWxpZCBjb25kaXRpb24gdG8gY2xlYXIgdGhlIFJYIGludGVycnVwdC4K CkFyZSB5b3Ugc3VyZT8gTXkgdW5kZXJzdGFuZGluZyBpcyB0aGF0IHRoZSBzZW1hbnRpY3Mgb2Yg dGhlIHJldHVybiB2YWx1ZQpvZiB4ZW5jb25zX3F1ZXVlZCgpIGRpZmZlcnMgYmV0d2VlbiBpbnRm IGFuZCBvdXRmOgotIEZvciBpbnRmLCBYZW4gZmlsbHMgdGhhdCBidWZmZXIgd2l0aCBpbmNvbWlu ZyBjaGFyYWN0ZXJzLiBUaGUKd2F0ZXJtYXJrIGlzIGFzc3VtZWQgdG8gYmUgKEZJRk8gLyAyKSwg d2hpY2ggdHJhbnNsYXRlcyBpbnRvIDE2CmNoYXJhY3RlcnMuIE5vdyBmb3IgdGhlIFNCU0EgdlVB UlQgUlggc2lkZSB0aGF0IG1lYW5zOiAiQXNzZXJ0IHRoZSBSWAppbnRlcnJ1cHQgaWYgdGhlcmUg aXMgb25seSByb29tIGZvciAxNiAob3IgbGVzcykgY2hhcmFjdGVycyBpbiB0aGUgRklGTwoocmVh ZDogaW50ZiBidWZmZXIgaW4gb3VyIGNhc2UpLiBTaW5jZSB3ZSAoYWIpdXNlIHRoZSBYZW4gYnVm ZmVyIGZvciB0aGUKRklGTywgdGhpcyBtZWFucyB3ZSB3YXJuIGlmIHRoZSBudW1iZXIgb2YgcXVl dWVkIGNoYXJhY3RlcnMgZXhjZWVkcwooYnVmZmVyc2l6ZSAtIDE2KS4KLSBGb3Igb3V0ZiwgdGhl IFVBUlQgZW11bGF0aW9uIGZpbGxzIHRoZSBidWZmZXIuIFRoZSBTQlNBIHZVQVJUIFRYIHNpZGUK ZGVtYW5kcyB0aGF0IHRoZSBUWCBpbnRlcnJ1cHQgaXMgYXNzZXJ0ZWQgaWYgdGhlIGZpbGwgbGV2 ZWwgb2YgdGhlCnRyYW5zbWl0IEZJRk8gaXMgbGVzcyB0aGFuIG9yIGVxdWFsIHRvIHRoZSAxNiBj aGFyYWN0ZXJzLCB3aGljaCBtZWFuczoKbnVtYmVyIG9mIHF1ZXVlZCBjaGFyYWN0ZXJzIGlzIGxl c3MgdGhhbiAxNi4KCkkgdGhpbmsgdGhlIGtleSBwb2ludCBpcyB0aGF0IG91ciB0cmlnZ2VyIGxl dmVsIGlzbid0IHN5bW1ldHJpY2FsIGhlcmUsCnNpbmNlIHdlIGhhdmUgdG8gZW11bGF0ZSB0aGUg YXJjaGl0ZWN0ZWQgMzItYnl0ZSBGSUZPIHNlbWFudGljcyBmb3IgdGhlCmRyaXZlciwgYnV0IGhh dmUgYSAoc2VjcmV0bHkpIG11Y2ggbGFyZ2VyICJGSUZPIiBpbnRlcm5hbGx5LgoKRG8geW91IGFn cmVlIHdpdGggdGhpcyByZWFzb25pbmcgYW5kIGRvIEkgaGF2ZSBhIHRoaW5rbyBoZXJlPyBDb3Vs ZCB3ZWxsCmJlIEkgYW0gc2VyaW91c2x5IG1pc2d1aWRlZCBoZXJlLgoKQ2hlZXJzLApBbmRyZQoK Pj4gICAgICB9Cj4+ICAgICAgZWxzZQo+PiAgICAgICAgICBncHJpbnRrKFhFTkxPR19FUlIsICJ2 cGwwMTE6IFVuZXhwZWN0ZWQgSU4gcmluZyBidWZmZXIgZW1wdHlcbiIpOwo+PiBAQCAtMTI5LDcg KzEzMyw3IEBAIHN0YXRpYyB2b2lkIHZwbDAxMV91cGRhdGVfdHhfZmlmb19zdGF0dXMoc3RydWN0 IHZwbDAxMSAqdnBsMDExLAo+PiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICB1bnNpZ25lZCBpbnQgZmlmb19sZXZlbCkKPj4gIHsKPj4gICAgICBzdHJ1Y3QgeGVuY29u c19pbnRlcmZhY2UgKmludGYgPSB2cGwwMTEtPnJpbmdfYnVmOwo+PiAtICAgIHVuc2lnbmVkIGlu dCBmaWZvX3RocmVzaG9sZDsKPj4gKyAgICB1bnNpZ25lZCBpbnQgZmlmb190aHJlc2hvbGQgPSBz aXplb2YoaW50Zi0+b3V0KSAtIFNCU0FfVUFSVF9GSUZPX1NJWkUvMjsKPj4KPj4gICAgICBCVUlM RF9CVUdfT04oc2l6ZW9mIChpbnRmLT5vdXQpIDwgU0JTQV9VQVJUX0ZJRk9fU0laRSk7Cj4+Cj4+ IEBAIC0xMzcsOCArMTQxLDYgQEAgc3RhdGljIHZvaWQgdnBsMDExX3VwZGF0ZV90eF9maWZvX3N0 YXR1cyhzdHJ1Y3QgdnBsMDExICp2cGwwMTEsCj4+ICAgICAgICogU2V0IHRoZSBUWEkgYml0IG9u bHkgd2hlbiB0aGVyZSBpcyBzcGFjZSBmb3IgZmlmb19zaXplLzIgYnl0ZXMgd2hpY2gKPj4gICAg ICAgKiBpcyB0aGUgdHJpZ2dlciBsZXZlbCBmb3IgYXNzZXJ0aW5nL2RlLWFzc3RlcnRpbmcgdGhl IFRYIGludGVycnVwdC4KPj4gICAgICAgKi8KPj4gLSAgICBmaWZvX3RocmVzaG9sZCA9IHNpemVv ZihpbnRmLT5vdXQpIC0gU0JTQV9VQVJUX0ZJRk9fU0laRS8yOwo+PiAtCj4+ICAgICAgaWYgKCBm aWZvX2xldmVsIDw9IGZpZm9fdGhyZXNob2xkICkKPj4gICAgICAgICAgdnBsMDExLT51YXJ0cmlz IHw9IFRYSTsKPj4gICAgICBlbHNlCj4+IEBAIC0zOTAsMzUgKzM5MiwzMCBAQCBzdGF0aWMgdm9p ZCB2cGwwMTFfZGF0YV9hdmFpbChzdHJ1Y3QgZG9tYWluICpkKQo+PiAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgb3V0X2NvbnMsCj4+ICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICBzaXplb2YoaW50Zi0+b3V0KSk7Cj4+Cj4+IC0gICAgLyogVXBkYXRlIHRo ZSB1YXJ0IHJ4IHN0YXRlIGlmIHRoZSBidWZmZXIgaXMgbm90IGVtcHR5LiAqLwo+PiAtICAgIGlm ICggaW5fZmlmb19sZXZlbCAhPSAwICkKPj4gLSAgICB7Cj4+ICsgICAgLyoqKiogVXBkYXRlIHRo ZSBVQVJUIFJYIHN0YXRlICoqKiovCj4+ICsKPj4gKyAgICAvKiBDbGVhciB0aGUgRklGT19FTVBU WSBiaXQgaWYgdGhlIEZJRk8gaG9sZHMgYXQgbGVhc3Qgb25lIGNoYXJhY3Rlci4gKi8KPj4gKyAg ICBpZiAoIGluX2ZpZm9fbGV2ZWwgPiAwICkKPj4gICAgICAgICAgdnBsMDExLT51YXJ0ZnIgJj0g flJYRkU7Cj4+Cj4+IC0gICAgICAgIGlmICggaW5fZmlmb19sZXZlbCA9PSBzaXplb2YoaW50Zi0+ aW4pICkKPj4gLSAgICAgICAgICAgIHZwbDAxMS0+dWFydGZyIHw9IFJYRkY7Cj4+ICsgICAgLyog U2V0IHRoZSBGSUZPX0ZVTEwgYml0IGlmIHRoZSByaW5nIGJ1ZmZlciBpcyBmdWxsLiAqLwo+PiAr ICAgIGlmICggaW5fZmlmb19sZXZlbCA9PSBzaXplb2YoaW50Zi0+aW4pICkKPj4gKyAgICAgICAg dnBsMDExLT51YXJ0ZnIgfD0gUlhGRjsKPj4KPj4gLSAgICAgICAgLyoKPj4gLSAgICAgICAgICog Q3VycmVudGx5LCB0aGUgUlhJIGJpdCBpcyBnZXR0aW5nIHNldCBldmVuIGlmIHRoZXJlIGlzIGEg c2luZ2xlCj4+IC0gICAgICAgICAqIGJ5dGUgb2YgZGF0YSBpbiB0aGUgcnggZmlmby4gSWRlYWxs eSwgdGhlIFJYSSBiaXQgc2hvdWxkIGJlIHNldAo+PiAtICAgICAgICAgKiBvbmx5IGlmIHRoZSBy eCBmaWZvIGxldmVsIHJlYWNoZXMgdGhlIHRocmVzaG9sZC4KPj4gLSAgICAgICAgICoKPj4gLSAg ICAgICAgICogSG93ZXZlciwgc2luY2UgY3VycmVudGx5IFJYIHRpbWVvdXQgaW50ZXJydXB0IGlz IG5vdAo+PiAtICAgICAgICAgKiBpbXBsZW1lbnRlZCBhcyB0aGVyZSBpcyBub3QgZW5vdWdoIGNs YXJpdHkgaW4gdGhlIFNCU0Egc3BlYywKPj4gLSAgICAgICAgICogdGhlIGd1ZXN0IG1heSBrZWVw IHdhaXRpbmcgZm9yIGFuIGludGVycnVwdCB0byByZWFkIG1vcmUKPj4gLSAgICAgICAgICogZGF0 YS4gVG8gZW5zdXJlIHRoYXQgZ3Vlc3QgcmVhZHMgYWxsIHRoZSBkYXRhIHdpdGhvdXQKPj4gLSAg ICAgICAgICogYW55IGRlbGF5LCB0aGUgUlhJIGludGVycnVwdCBpcyByYWlzZWQgaWYgdGhlcmUg aXMgUlggZGF0YQo+PiAtICAgICAgICAgKiBhdmFpbGFibGUgd2l0aG91dCBjaGVja2luZyB3aGV0 aGVyIGZpZm8gbGV2ZWwgaGFzIHJlYWNoZWQKPj4gLSAgICAgICAgICogdGhlIHRocmVzaG9sZC4K Pj4gLSAgICAgICAgICoKPj4gLSAgICAgICAgICogVEJEOiBPbmNlIHRoZXJlIGlzIG1vcmUgY2xh cml0eSBpbiB0aGUgU0JTQSBzcGVjIG9uIHdoZXRoZXIgUlgKPj4gLSAgICAgICAgICogdGltZW91 dCBpbnRlcnJ1cHQgbmVlZHMgdG8gYmUgaW1wbGVtZW50ZWQsIHRoZSBSWEkgaW50ZXJydXB0Cj4+ IC0gICAgICAgICAqIHdpbGwgYmUgcmFpc2VkIG9ubHkgd2hlbiByeCBmaWZvIGxldmVsIHJlYWNo ZXMgdGhlIHRocmVzaG9sZC4KPj4gLSAgICAgICAgICovCj4+ICsgICAgLyogVGhlIEZJRk8gdHJp Z2dlciBsZXZlbCBpcyBmaXhlZCB0byBoYWxmIG9mIHRoZSBGSUZPLiAqLwo+PiArICAgIGlmICgg aW5fZmlmb19sZXZlbCA+PSBzaXplb2YoaW50Zi0+aW4pIC0gU0JTQV9VQVJUX0ZJRk9fU0laRSAv IDIgKQo+PiAgICAgICAgICB2cGwwMTEtPnVhcnRyaXMgfD0gUlhJOwo+IEhlcmUgYWxzbyBzaG91 bGQgbm90IHdlIGNoZWNrIGlmICggaW5fZmlmb19sZXZlbCA+PQo+IFNCU0FfVUFSVF9GSUZPX1NJ WkUgLyAyICkgc2luY2UgaXQgaXMgYSB2YWxpZCBjb25kaXRpb24gdG8gcmFpc2UgdGhlCj4gUlgg aW50ZXJydXB0Pwo+IAo+PiAtICAgIH0KPj4KPj4gLSAgICAvKiBVcGRhdGUgdGhlIHVhcnQgdHgg c3RhdGUgaWYgdGhlIGJ1ZmZlciBpcyBub3QgZnVsbC4gKi8KPj4gKyAgICAvKgo+PiArICAgICAq IElmIHRoZSBpbnB1dCBxdWV1ZSBpcyBub3QgZW1wdHksIHdlIGFzc2VydCB0aGUgcmVjZWl2ZSB0 aW1lb3V0IGludGVycnVwdC4KPj4gKyAgICAgKiBBcyB3ZSBkb24ndCBlbXVsYXRlIGFueSB0aW1p bmcgaGVyZSwgd2UgaWdub3JlIHRoZSBhY3R1YWwgdGltZW91dAo+PiArICAgICAqIG9mIDMyIGJp dCBwZXJpb2RzLgo+PiArICAgICAqLwo+PiArICAgIGlmICggaW5fZmlmb19sZXZlbCA+IDAgKQo+ PiArICAgICAgICB2cGwwMTEtPnVhcnRyaXMgfD0gUlRJOwo+PiArCj4+ICsgICAgLyoqKiogVXBk YXRlIHRoZSBVQVJUIFRYIHN0YXRlICoqKiovCj4+ICsKPj4gICAgICBpZiAoIG91dF9maWZvX2xl dmVsICE9IHNpemVvZihpbnRmLT5vdXQpICkKPj4gICAgICB7Cj4+ICAgICAgICAgIHZwbDAxMS0+ dWFydGZyICY9IH5UWEZGOwo+PiBAQCAtNDMxLDEzICs0MjgsMTMgQEAgc3RhdGljIHZvaWQgdnBs MDExX2RhdGFfYXZhaWwoc3RydWN0IGRvbWFpbiAqZCkKPj4gICAgICAgICAgdnBsMDExLT51YXJ0 ZnIgJj0gfkJVU1k7Cj4+Cj4+ICAgICAgICAgIHZwbDAxMV91cGRhdGVfdHhfZmlmb19zdGF0dXMo dnBsMDExLCBvdXRfZmlmb19sZXZlbCk7Cj4+IC0KPj4gLSAgICAgICAgaWYgKCBvdXRfZmlmb19s ZXZlbCA9PSAwICkKPj4gLSAgICAgICAgICAgIHZwbDAxMS0+dWFydGZyIHw9IFRYRkU7Cj4+ICAg ICAgfQo+Pgo+PiAgICAgIHZwbDAxMV91cGRhdGVfaW50ZXJydXB0X3N0YXR1cyhkKTsKPj4KPj4g KyAgICBpZiAoIG91dF9maWZvX2xldmVsID09IDAgKQo+PiArICAgICAgICB2cGwwMTEtPnVhcnRm ciB8PSBUWEZFOwo+PiArCj4+ICAgICAgVlBMMDExX1VOTE9DSyhkLCBmbGFncyk7Cj4+ICB9Cj4+ Cj4+IC0tCj4+IDIuMTQuMQo+Pgo+IAo+IFJlZ2FyZHMsCj4gQmh1cGluZGVyCj4gCgpfX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpYZW4tZGV2ZWwgbWFpbGlu ZyBsaXN0Clhlbi1kZXZlbEBsaXN0cy54ZW4ub3JnCmh0dHBzOi8vbGlzdHMueGVuLm9yZy94ZW4t ZGV2ZWwK