From mboxrd@z Thu Jan  1 00:00:00 1970
From: Roger Pau =?utf-8?B?TW9ubsOp?= <roger.pau@citrix.com>
Subject: Re: [PATCH v2] xen/pt: fix some pass-thru devices don't
 work across reboot
Date: Fri, 16 Nov 2018 10:35:29 +0100
Message-ID: <20181116093529.4ucjehnm75tulsdq@mac>
References: <1542244226-35003-1-git-send-email-chao.gao@intel.com>
 <20181115104039.geyiusafhsq4q7x4@mac>
 <20181116075348.GA6684@gao-cwp>
Mime-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
Return-path: <xen-devel-bounces@lists.xenproject.org>
Received: from us1-rack-dfw2.inumbo.com ([104.130.134.6])
 by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from
 <srs0=f3fv=n3=citrix.com=prvs=8510c96a3=roger.pau@srs-us1.protection.inumbo.net>)
 id 1gNaXN-0004P4-Py
 for xen-devel@lists.xenproject.org; Fri, 16 Nov 2018 09:35:41 +0000
Content-Disposition: inline
In-Reply-To: <20181116075348.GA6684@gao-cwp>
List-Unsubscribe: <https://lists.xenproject.org/mailman/options/xen-devel>,
 <mailto:xen-devel-request@lists.xenproject.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xenproject.org>
List-Help: <mailto:xen-devel-request@lists.xenproject.org?subject=help>
List-Subscribe: <https://lists.xenproject.org/mailman/listinfo/xen-devel>,
 <mailto:xen-devel-request@lists.xenproject.org?subject=subscribe>
Errors-To: xen-devel-bounces@lists.xenproject.org
Sender: "Xen-devel" <xen-devel-bounces@lists.xenproject.org>
To: Chao Gao <chao.gao@intel.com>
Cc: xen-devel@lists.xenproject.org, Wei Liu <wei.liu2@citrix.com>, Jan Beulich <jbeulich@suse.com>, Andrew Cooper <andrew.cooper3@citrix.com>
List-Id: xen-devel@lists.xenproject.org

T24gRnJpLCBOb3YgMTYsIDIwMTggYXQgMDM6NTM6NTBQTSArMDgwMCwgQ2hhbyBHYW8gd3JvdGU6
Cj4gT24gVGh1LCBOb3YgMTUsIDIwMTggYXQgMTE6NDA6MzlBTSArMDEwMCwgUm9nZXIgUGF1IE1v
bm7DqSB3cm90ZToKPiA+T24gVGh1LCBOb3YgMTUsIDIwMTggYXQgMDk6MTA6MjZBTSArMDgwMCwg
Q2hhbyBHYW8gd3JvdGU6Cj4gPj4gSSBmaW5kIHNvbWUgcGFzcy10aHJ1IGRldmljZXMgZG9uJ3Qg
d29yayBhbnkgbW9yZSBhY3Jvc3MgZ3Vlc3QKPiA+PiByZWJvb3QuIEFzc2lnbmluZyBpdCB0byBh
bm90aGVyIGRvbWFpbiBhbHNvIG1lZXRzIHRoZSBzYW1lIGlzc3VlLiBBbmQKPiA+PiB0aGUgb25s
eSB3YXkgdG8gbWFrZSBpdCB3b3JrIGFnYWluIGlzIHVuLWJpbmRpbmcgYW5kIGJpbmRpbmcgaXQg
dG8KPiA+PiBwY2liYWNrLiBTb21lb25lIHJlcG9ydGVkIHRoaXMgaXNzdWUgb25lIHllYXIgYWdv
IFsxXS4KPiA+Cj4gPkhvdyBkb2VzIHVuYmluZGluZyBhbmQgYmluZGluZyB0byBwY2liYWNrIGZp
eCB0aGUgaXNzdWU/IElzIHRoaXMgZHVlCj4gPnRvIERvbTAgdXNpbmcgc29tZSBQViBvciBEb20w
IG9ubHkgaHlwZXJjYWxsIHRoYXQgY2FuIHJlc2V0IHRoZQo+ID5ob3N0X21hc2thbGwgc3RhdGU/
Cj4gCj4gSSB0aGluayBpdCBpcyBtc2l4X2NhcGFiaWxpdHlfaW5pdCgpIHRoYXQgY2xlYXIgaG9z
dF9tYXNrYWxsLiBBbmQgdGhpcwo+IGZ1bmN0aW9uIGlzIGNhbGxlZCBieSBQSFlTREVWT1BfcHJl
cGFyZV9tc2l4IHN1Yi1oeXBlcmNhbGwuCj4gCj4gPgo+ID4+IAo+ID4+IElmIHRoZSBkZXZpY2Un
cyBkcml2ZXIgZG9lc24ndCBkaXNhYmxlIE1TSS1YIGR1cmluZyBzaHV0ZG93biBvciBxZW11IGlz
Cj4gPj4ga2lsbGVkL2NyYXNoZWQgYmVmb3JlIHRoZSBkb21haW4gc2h1dGRvd24sIHRoaXMgZG9t
YWluJ3MgcGlycSB3b24ndCBiZQo+ID4+IHVubWFwcGVkLiBUaGVuIHhlbiB3aWxsIHVubWFwIGFs
bCBwaXJxLiBCdXQgcGNpYmFjayBoYXMgYWxyZWFkeSBkaXNhYmxlZAo+ID4+IG1lb3J5IGRlY29k
aW5nIGJlZm9yZSB4ZW4gdW5tYXBwaW5nIHBpcnEuIFRoZW4gd2hlbiBYZW4gaXMgZGlzYWJsaW5n
IGEKPiA+PiBNU0kgb2YgdGhlIGRldmljZSwgaXQgaGFzIHRvIHNldHMgdGhlIGhvc3RfbWFza2Fs
bCBmbGFnIGFuZCBtYXNrYWxsIGJpdAo+ID4+IHRvIG1hc2sgYSBNU0kgcmF0aGVyIHRoYW4gc2V0
cyBtYXNrYml0IGluIE1TSS14IHRhYmxlLiBUaGUgY2FsbCB0cmFjZSBvZgo+ID4+IHRoaXMgcHJv
Y2VzcyBpczoKPiA+PiAtPmFyY2hfZG9tYWluX2Rlc3Ryb3kKPiA+PiAgICAgLT5mcmVlX2RvbWFp
bl9waXJxcwo+ID4+ICAgICAgICAgLT51bm1hcF9kb21haW5fcGlycSAoaWYgcGlycSBpc24ndCB1
bm1hcCBieSBxZW11KQo+ID4+ICAgICAgICAgICAgIC0+cGlycV9ndWVzdF9mb3JjZV91bmJpbmQK
PiA+PiAgICAgICAgICAgICAgICAgLT5fX3BpcnFfZ3Vlc3RfdW5iaW5kCj4gPj4gICAgICAgICAg
ICAgICAgICAgICAtPm1hc2tfbXNpX2lycSg9ZGVzYy0+aGFuZGxlci0+ZGlzYWJsZSgpKQo+ID4+
ICAgICAgICAgICAgICAgICAgICAgICAgIC0+dGhlIHdhcm5pbmcgaW4gbXNpX3NldF9tYXNrX2Jp
dCgpCj4gPj4gCj4gPj4gVGhlIGhvc3RfbWFza2FsbCBiaXQgd2lsbCBwcmV2ZW50IGd1ZXN0cyBm
cm9tIGNsZWFyaW5nIHRoZSBtYXNrYWxsIGJpdAo+ID4+IGV2ZW4gdGhlIGRldmljZSBpcyBhc3Np
Z25lZCB0byBhbm90aGVyIGd1ZXN0IGxhdGVyLiBHdWVzdHMgY2Fubm90Cj4gPj4gcmVjZWl2ZSBp
bnRlcnJ1cHRzIGZyb20gdGhpcyBkZXZpY2UuCj4gPj4KPiA+PiAKPiA+PiBUbyBmaXggdGhpcywg
aG9zdF9tYXNrYWxsIGZsYWcgaXMgY2xlYXJlZCB3aGVuIGFsbCBNU0lzIG9mIGEgZGV2aWNlIGFy
ZSBmcmVlZC4KPiA+PiBJdCBpcyBkZWZpbml0ZWx5IHNhZmVseSB0byBjbGVhciBpdCBiZWNhdXNl
IG5vIG1zaSBpcyBhY3R1YWxseSBzZXQgdXAKPiA+PiBmb3IgdGhpcyBkZXZpY2UuIEFsc28sICdt
c2l4LT53YXJuZWQnIGlzIGluaXRpYWxpemVkIHRvIERPTUlEX0lOVkFMSUQKPiA+PiByYXRoZXIg
dGhhbiAwIHRvIGF2b2lkIHdhcm5pbmdzIG1pc3NpbmcgZm9yIERvbTAuCj4gPj4gCj4gPj4gWzFd
OiBodHRwczovL2xpc3RzLnhlbnByb2plY3Qub3JnL2FyY2hpdmVzL2h0bWwveGVuLWRldmVsLzIw
MTctMDkvbXNnMDI1MjAuaHRtbAo+ID4+IAo+ID4+IFNpZ25lZC1vZmYtYnk6IENoYW8gR2FvIDxj
aGFvLmdhb0BpbnRlbC5jb20+Cj4gPj4gLS0tCj4gPj4gIHhlbi9hcmNoL3g4Ni9tc2kuYyAgICAg
ICAgICAgIHwgMTggKysrKysrKysrKysrKysrKysrCj4gPj4gIHhlbi9kcml2ZXJzL3Bhc3N0aHJv
dWdoL3BjaS5jIHwgIDEgKwo+ID4+ICAyIGZpbGVzIGNoYW5nZWQsIDE5IGluc2VydGlvbnMoKykK
PiA+PiAKPiA+PiBkaWZmIC0tZ2l0IGEveGVuL2FyY2gveDg2L21zaS5jIGIveGVuL2FyY2gveDg2
L21zaS5jCj4gPj4gaW5kZXggNTU2Nzk5MC4uY2Q1NzBiYyAxMDA2NDQKPiA+PiAtLS0gYS94ZW4v
YXJjaC94ODYvbXNpLmMKPiA+PiArKysgYi94ZW4vYXJjaC94ODYvbXNpLmMKPiA+PiBAQCAtNjM3
LDYgKzYzNyw3IEBAIGludCBtc2lfZnJlZV9pcnEoc3RydWN0IG1zaV9kZXNjICplbnRyeSkKPiA+
PiAgewo+ID4+ICAgICAgdW5zaWduZWQgaW50IG5yID0gZW50cnktPm1zaV9hdHRyaWIudHlwZSAh
PSBQQ0lfQ0FQX0lEX01TSVgKPiA+PiAgICAgICAgICAgICAgICAgICAgICAgID8gZW50cnktPm1z
aS5udmVjIDogMTsKPiA+PiArICAgIHN0cnVjdCBwY2lfZGV2ICpwZGV2ID0gZW50cnktPmRldjsK
PiA+PiAgCj4gPj4gICAgICB3aGlsZSAoIG5yLS0gKQo+ID4+ICAgICAgewo+ID4+IEBAIC02NTQs
NiArNjU1LDIzIEBAIGludCBtc2lfZnJlZV9pcnEoc3RydWN0IG1zaV9kZXNjICplbnRyeSkKPiA+
PiAgCj4gPj4gICAgICBsaXN0X2RlbCgmZW50cnktPmxpc3QpOwo+ID4+ICAgICAgeGZyZWUoZW50
cnkpOwo+ID4+ICsKPiA+PiArICAgIC8qCj4gPj4gKyAgICAgKiBJbiBzb21lIGNhc2VzLCB0aGUg
J2hvc3RfbWFza2FsbCcgaXMgc2V0IGZvciBzYWZldHkuIEhlcmUgY2xlYXIKPiA+PiArICAgICAq
ICdob3N0X21hc2thbGwnIGlmIGFsbCBNU0lzIGFyZSBmcmVlZCBmb3IgYSBtc2kteCBjYXBhYmxl
IGRldmljZS4KPiA+PiArICAgICAqIFdpdGhvdXQgaXQsIHRoZSBkZXZpY2UncyBtc2l4IGtlZXBz
IGRpc2FibGVkIGV2ZW4gYmVlbiByZWFzc2lnbmVkCj4gPgo+ID4iLi4uIGV2ZW4gYWZ0ZXIgYmVp
bmcgcmVhc3NpZ25lZCAuLi4iCj4gPgo+ID5JIHRoaW5rIGl0J3MgY2xlYXJlci4KPiA+Cj4gPj4g
KyAgICAgKiB0byBhbm90aGVyIGRvbWFpbi4KPiA+PiArICAgICAqLwo+ID4+ICsgICAgaWYgKCBw
ZGV2ICYmIGxpc3RfZW1wdHkoJnBkZXYtPm1zaV9saXN0KSAmJiBwZGV2LT5tc2l4ICkKPiA+PiAr
ICAgIHsKPiA+PiArICAgICAgICBpZiAoIHBkZXYtPm1zaXgtPmhvc3RfbWFza2FsbCApCj4gPj4g
KyAgICAgICAgICAgIHByaW50ayhYRU5MT0dfR19XQVJOSU5HCj4gPj4gKyAgICAgICAgICAgICAg
ICAgICAiUmVzZXR0aW5nIG1zaXggc3RhdHVzIG9mICUwNHg6JTAyeDolMDJ4LiV1XG4iLAo+ID4+
ICsgICAgICAgICAgICAgICAgICAgcGRldi0+c2VnLCBwZGV2LT5idXMsIFBDSV9TTE9UKHBkZXYt
PmRldmZuKSwKPiA+PiArICAgICAgICAgICAgICAgICAgIFBDSV9GVU5DKHBkZXYtPmRldmZuKSk7
Cj4gPj4gKyAgICAgICAgcGRldi0+bXNpeC0+aG9zdF9tYXNrYWxsID0gZmFsc2U7Cj4gPj4gKyAg
ICAgICAgcGRldi0+bXNpeC0+d2FybmVkID0gRE9NSURfSU5WQUxJRDsKCkFGQUlDVCBhIGd1ZXN0
IGNvdWxkIHRyaWdnZXIgdGhpcyBtZXNzYWdlIG11bHRpcGxlIHRpbWVzIGJ5IGZvcmNpbmcgYQpQ
SVJRIG1hcC91bm1hcCBvZiBhbGwgdGhlIHZlY3RvcnMgaW4gTVNJWCwgdGh1cyBsaWtlbHkgZmxv
b2RpbmcgdGhlCmNvbnNvbGUgc2luY2UgdGhpcyBpcyBub3QgcmF0ZSBsaW1pdGVkLiBTaW5jZSBJ
IHRoaW5rIGEgZ3Vlc3QgY2FuCm1hbmFnZSB0byByZWFjaCB0aGlzIGNvZGUgcGF0aCB3aGlsZSBy
dW5uaW5nLCBjbGVhcmluZyB3YXJuZWQgaXMgbm90CmNvcnJlY3QuCgpBbHNvLCBpZiBhIGd1ZXN0
IGNhbiBtYW5hZ2UgdG8gdHJpZ2dlciB0aGlzIHBhdGggZHVyaW5nIGl0J3MgcnVudGltZSwKY291
bGQgaXQgYWxzbyBoaXQgdGhlIGlzc3VlIG9mIGdldHRpbmcgaG9zdF9tYXNrYWxsIHNldCBhbmQg
bm90IGJlaW5nCmFibGUgdG8gY2xlYXIgaXQ/Cgo+ID4+ICsgICAgfQo+ID4+ICAgICAgcmV0dXJu
IDA7Cj4gPgo+ID5JTU8gdGhpcyBsb29rcyBxdWl0ZSBzaW1pbGFyIHRvIGEgbXNpX3Jlc2V0X3N0
YXRlIGZ1bmN0aW9uIG9yIGF0IGxlYXN0Cj4gPnRoZSBzdGFydCBvZiBpdC4KPiA+Cj4gPk1heWJl
IGl0IHNob3VsZCBiZSBpbiBhIHNlcGFyYXRlIGhlbHBlciB0aGF0IHNldHMgYWxsIHRoZSBmaWVs
ZHMgdG8KPiA+dGhlaXIgaW5pdGlhbCB2YWx1ZXMsCj4gCj4gV2lsbCBkby4KPiAKPiA+d2l0aCB0
aGUgZXhwZWN0YXRpb24gdGhhdCBEb20wIHdpbGwgcGVyZm9ybSB0aGUKPiA+ZGV2aWNlIHJlc2V0
Pwo+IAo+IERvbTAgcmVzZXRzIGRldmljZXMgd2hlbiAoZGUtKWFzc2lnbm1lbnQgYnkgZWNobyAx
IHRvCj4gL3N5cy9idXMvcGNpL2RldmljZXMvPHNiZGY+L3Jlc2V0Lgo+IAo+ID4KPiA+VGhlIHVu
ZGVybHlpbmcgcHJvYmxlbSBoZXJlIEFGQUlDVCBpcyB0aGF0IHRoZSBYZW4gaW50ZXJuYWwgZGV2
aWNlCj4gPnN0YXRlIGlzIG5vdCB0aGUgc2FtZSBiZXR3ZWVuIGRldmljZSBhc3NpZ25tZW50cy4K
PiAKPiBZZXMsIGl0IGlzIGFjY3VyYXRlLgo+IAo+ID4KPiA+SW4gYW55IGNhc2UgdGhlcmUgc2hv
dWxkIGJlIGF0IGxlYXN0IGEgbm90ZSBoZXJlIHBvaW50aW5nIG91dCB0aGF0IFhlbgo+ID5leHBl
Y3RzIHRoZSBoYXJkd2FyZSBkb21haW4gdG8gcGVyZm9ybSBhIGRldmljZSByZXNldCwgc28gdGhl
IFhlbgo+ID5pbnRlcm5hbCBzdGF0ZSBhY3R1YWxseSBtYXRjaGVzIHRoZSBkZXZpY2Ugc3RhdGUg
YmVmb3JlIHRyeWluZyB0bwo+ID5hc3NpZ24gdGhlIGRldmljZSB0byBhbm90aGVyIGd1ZXN0Lgo+
IAo+IFNvdW5kcyBnb29kLiBUaGlzIGlzc3VlIGlzIHRoYXQgWGVuIHRyaWVzIHRvIG1hc2sgbXNp
ICh3aGVuIHVubWFwcGluZyBwaXJxKQo+IGFmdGVyIG1lbW9yeSBkZWNvZGluZyBpcyBkaXNhYmxl
ZCBieSBwY2liYWNrLiBJZiBwY2liYWNrIGNhbiB1bm1hcCBhbGwgdGhlCj4gcGlycS1zIHJlbGF0
ZWQgYSBnaXZlbiBkZXZpY2UgYmVmb3JlIGRpc2FibGluZyBtZW1vcnkgZGVjb2RpbmcsIFhlbiB3
b24ndCBtZWV0Cj4gdGhpcyBpc3N1ZS4gQ3VycmVudGx5LCBwY2liYWNrIGRvZXNuJ3QgbWFpbnRh
aW4gdGhlIHBpcnEgaW5mb3JtYXRpb247IGl0Cj4gaXNuJ3QgYWJsZSB0byBkbyB0aGlzLgoKSSB3
b3VsZCBsaWtlIHRvIGhlYXIgSmFuJ3Mgb3BpbmlvbiBvbiB0aGlzLCBidXQgSSB0aGluayBpdCBt
aWdodCBiZQpoZWxwZnVsIHRvIGludHJvZHVjZSBhIG5ldyBoeXBlcmNhbGwgRG9tMCAoaWU6IHRv
b2xzdGFjaykgY2FuIHVzZSB0bwpzaWduYWwgWGVuIGEgUENJIGRldmljZSBoYXMgYmVlbiByZXNl
dCwgc28gdGhhdCBYZW4gY2FuIHNhZmVseSByZXNldAp0aGUgZGV2aWNlIHN0YXRlIHRvIHRoZSBp
bml0aWFsIG9uZS4gVGhpcyB3b3VsZCBiZSBzaW1wbGVyIGlmIFhlbiB3YXMKdGhlIG9uZSBwZXJm
b3JtaW5nIHRoZSBkZXZpY2UgcmVzZXQuCgpPbmNlIGEgZGV2aWNlIGhhcyBiZWVuIGFzc2lnbmVk
IGl0IHdvdWxkIGJlY29tZSAnZGlydHknIGFuZCB3b3VsZApyZXF1aXJlIHN1Y2ggcmVzZXQgYmVm
b3JlIGl0IGNvdWxkIGJlIGFzc2lnbmVkIGFnYWluIHRvIGEgZG9tYWluLgoKVGhhbmtzLCBSb2dl
ci4KCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fClhlbi1k
ZXZlbCBtYWlsaW5nIGxpc3QKWGVuLWRldmVsQGxpc3RzLnhlbnByb2plY3Qub3JnCmh0dHBzOi8v
bGlzdHMueGVucHJvamVjdC5vcmcvbWFpbG1hbi9saXN0aW5mby94ZW4tZGV2ZWw=