From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cornelia Huck Subject: Re: [PATCH V3 4/7] mdev: introduce device specific ops Date: Wed, 16 Oct 2019 10:52:32 +0200 Message-ID: <20191016105232.663dd3c9.cohuck@redhat.com> References: <20191011081557.28302-1-jasowang@redhat.com> <20191011081557.28302-5-jasowang@redhat.com> <20191015124137.4f948bd2.cohuck@redhat.com> <20191015112646.3776dc29@x1.home> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Parav Pandit Cc: "christophe.de.dinechin@gmail.com" , "kvm@vger.kernel.org" , "mst@redhat.com" , "airlied@linux.ie" , "joonas.lahtinen@linux.intel.com" , "heiko.carstens@de.ibm.com" , "dri-devel@lists.freedesktop.org" , "virtualization@lists.linux-foundation.org" , "kwankhede@nvidia.com" , "rob.miller@broadcom.com" , "linux-s390@vger.kernel.org" , "sebott@linux.ibm.com" , "lulu@redhat.com" , "eperezma@redhat.com" , "pasic@linux.ibm.com" , "borntraeger@de.ibm.com" , haotian.wang@sifive.c List-Id: virtualization@lists.linuxfoundation.org T24gV2VkLCAxNiBPY3QgMjAxOSAwNTo1MDowOCArMDAwMApQYXJhdiBQYW5kaXQgPHBhcmF2QG1l bGxhbm94LmNvbT4gd3JvdGU6Cgo+IEhpIEFsZXgsCj4gCj4gPiAtLS0tLU9yaWdpbmFsIE1lc3Nh Z2UtLS0tLQo+ID4gRnJvbTogQWxleCBXaWxsaWFtc29uIDxhbGV4LndpbGxpYW1zb25AcmVkaGF0 LmNvbT4KPiA+IFNlbnQ6IFR1ZXNkYXksIE9jdG9iZXIgMTUsIDIwMTkgMTI6MjcgUE0KPiA+IFRv OiBKYXNvbiBXYW5nIDxqYXNvd2FuZ0ByZWRoYXQuY29tPgo+ID4gQ2M6IENvcm5lbGlhIEh1Y2sg PGNvaHVja0ByZWRoYXQuY29tPjsga3ZtQHZnZXIua2VybmVsLm9yZzsgbGludXgtCj4gPiBzMzkw QHZnZXIua2VybmVsLm9yZzsgbGludXgta2VybmVsQHZnZXIua2VybmVsLm9yZzsgZHJpLQo+ID4g ZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnOyBpbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Au b3JnOyBpbnRlbC1ndnQtCj4gPiBkZXZAbGlzdHMuZnJlZWRlc2t0b3Aub3JnOyBrd2Fua2hlZGVA bnZpZGlhLmNvbTsgbXN0QHJlZGhhdC5jb207Cj4gPiB0aXdlaS5iaWVAaW50ZWwuY29tOyB2aXJ0 dWFsaXphdGlvbkBsaXN0cy5saW51eC1mb3VuZGF0aW9uLm9yZzsKPiA+IG5ldGRldkB2Z2VyLmtl cm5lbC5vcmc7IG1heGltZS5jb3F1ZWxpbkByZWRoYXQuY29tOwo+ID4gY3VubWluZy5saWFuZ0Bp bnRlbC5jb207IHpoaWhvbmcud2FuZ0BpbnRlbC5jb207Cj4gPiByb2IubWlsbGVyQGJyb2FkY29t LmNvbTsgeGlhby53LndhbmdAaW50ZWwuY29tOwo+ID4gaGFvdGlhbi53YW5nQHNpZml2ZS5jb207 IHpoZW55dXdAbGludXguaW50ZWwuY29tOyB6aGkuYS53YW5nQGludGVsLmNvbTsKPiA+IGphbmku bmlrdWxhQGxpbnV4LmludGVsLmNvbTsgam9vbmFzLmxhaHRpbmVuQGxpbnV4LmludGVsLmNvbTsK PiA+IHJvZHJpZ28udml2aUBpbnRlbC5jb207IGFpcmxpZWRAbGludXguaWU7IGRhbmllbEBmZnds bC5jaDsKPiA+IGZhcm1hbkBsaW51eC5pYm0uY29tOyBwYXNpY0BsaW51eC5pYm0uY29tOyBzZWJv dHRAbGludXguaWJtLmNvbTsKPiA+IG9iZXJwYXJAbGludXguaWJtLmNvbTsgaGVpa28uY2Fyc3Rl bnNAZGUuaWJtLmNvbTsgZ29yQGxpbnV4LmlibS5jb207Cj4gPiBib3JudHJhZWdlckBkZS5pYm0u Y29tOyBha3Jvd2lha0BsaW51eC5pYm0uY29tOyBmcmV1ZGVAbGludXguaWJtLmNvbTsKPiA+IGxp bmdzaGFuLnpodUBpbnRlbC5jb207IElkbyBTaGFtYXkgPGlkb3NAbWVsbGFub3guY29tPjsKPiA+ IGVwZXJlem1hQHJlZGhhdC5jb207IGx1bHVAcmVkaGF0LmNvbTsgUGFyYXYgUGFuZGl0Cj4gPiA8 cGFyYXZAbWVsbGFub3guY29tPjsgY2hyaXN0b3BoZS5kZS5kaW5lY2hpbkBnbWFpbC5jb207Cj4g PiBrZXZpbi50aWFuQGludGVsLmNvbQo+ID4gU3ViamVjdDogUmU6IFtQQVRDSCBWMyA0LzddIG1k ZXY6IGludHJvZHVjZSBkZXZpY2Ugc3BlY2lmaWMgb3BzCj4gPiAKPiA+IE9uIFR1ZSwgMTUgT2N0 IDIwMTkgMjA6MTc6MDEgKzA4MDAKPiA+IEphc29uIFdhbmcgPGphc293YW5nQHJlZGhhdC5jb20+ IHdyb3RlOgo+ID4gICAKPiA+ID4gT24gMjAxOS8xMC8xNSDkuIvljYg2OjQxLCBDb3JuZWxpYSBI dWNrIHdyb3RlOiAgCj4gPiA+ID4gT24gRnJpLCAxMSBPY3QgMjAxOSAxNjoxNTo1NCArMDgwMAo+ ID4gPiA+IEphc29uIFdhbmcgPGphc293YW5nQHJlZGhhdC5jb20+IHdyb3RlOgoKPiA+ID4gPj4g QEAgLTE2Nyw5ICsxNzYsMTAgQEAgcmVnaXN0ZXIgaXRzZWxmIHdpdGggdGhlIG1kZXYgY29yZSBk cml2ZXI6Ogo+ID4gPiA+PiAgIAlleHRlcm4gaW50ICBtZGV2X3JlZ2lzdGVyX2RldmljZShzdHJ1 Y3QgZGV2aWNlICpkZXYsCj4gPiA+ID4+ICAgCSAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgIGNvbnN0IHN0cnVjdCBtZGV2X3BhcmVudF9vcHMKPiA+ID4gPj4gKm9wcyk7Cj4gPiA+ID4+ Cj4gPiA+ID4+IC1JdCBpcyBhbHNvIHJlcXVpcmVkIHRvIHNwZWNpZnkgdGhlIGNsYXNzX2lkIHRo cm91Z2g6Ogo+ID4gPiA+PiArSXQgaXMgYWxzbyByZXF1aXJlZCB0byBzcGVjaWZ5IHRoZSBjbGFz c19pZCBhbmQgZGV2aWNlIHNwZWNpZmljIG9wcyAgCj4gPiB0aHJvdWdoOjogIAo+ID4gPiA+Pgo+ ID4gPiA+PiAtCWV4dGVybiBpbnQgbWRldl9zZXRfY2xhc3Moc3RydWN0IGRldmljZSAqZGV2LCB1 MTYgaWQpOwo+ID4gPiA+PiArCWV4dGVybiBpbnQgbWRldl9zZXRfY2xhc3Moc3RydWN0IGRldmlj ZSAqZGV2LCB1MTYgaWQsCj4gPiA+ID4+ICsJICAgICAgICAgICAgICAgICAgICAgICAgICBjb25z dCB2b2lkICpvcHMpOyAgCj4gPiA+ID4gQXBvbG9naWVzIGlmIHRoYXQgaGFzIGFscmVhZHkgYmVl biBkaXNjdXNzZWQsIGJ1dCBkbyB3ZSB3YW50IGEgMToxCj4gPiA+ID4gcmVsYXRpb25zaGlwIGJl dHdlZW4gaWQgYW5kIG9wcywgb3IgY2FuIGRpZmZlcmVudCBkZXZpY2VzIHdpdGggdGhlCj4gPiA+ ID4gc2FtZSBpZCByZWdpc3RlciBkaWZmZXJlbnQgb3BzPyAgCj4gPiA+Cj4gPiA+Cj4gPiA+IEkg dGhpbmsgd2UgaGF2ZSBhIE46MSBtYXBwaW5nIGJldHdlZW4gaWQgYW5kIG9wcywgZS5nIHdlIHdh bnQgYm90aAo+ID4gPiB2aXJ0aW8tbWRldiBhbmQgdmhvc3QtbWRldiB1c2UgYSBzaW5nbGUgc2V0 IG9mIGRldmljZSBvcHMuICAKPiA+IAo+ID4gVGhlIGNvbnRlbnRzIG9mIHRoZSBvcHMgc3RydWN0 dXJlIGlzIGVzc2VudGlhbGx5IGRlZmluZWQgYnkgdGhlIGlkLCB3aGljaCBpcwo+ID4gd2h5IEkg d2FzIGxlYW5pbmcgdG93YXJkcyB0aGVtIGJlaW5nIGRlZmluZWQgdG9nZXRoZXIuICBUaGV5IGFy ZSBlZmZlY3RpdmVseQo+ID4gaW50ZXJsb2NrZWQsIHRoZSBpZCBkZWZpbmVzIHdoaWNoIG1kZXYg ImVuZHBvaW50Igo+ID4gZHJpdmVyIGlzIGxvYWRlZCBhbmQgdGhhdCBkcml2ZXIgcmVxdWlyZXMg bWRldl9nZXRfZGV2X29wcygpIHRvIHJldHVybiB0aGUKPiA+IHN0cnVjdHVyZSByZXF1aXJlZCBi eSB0aGUgZHJpdmVyLiAgSSB3aXNoIHRoZXJlIHdhcyBhIHdheSB3ZSBjb3VsZAo+ID4gaW5jb3Jw b3JhdGUgdHlwZSBjaGVja2luZyBoZXJlLiAgV2UgdG95ZWQgd2l0aCB0aGUgaWRlYSBvZiBoYXZp bmcgdGhlIGNsYXNzCj4gPiBpbiB0aGUgc2FtZSBzdHJ1Y3R1cmUgYXMgdGhlIG9wcywgYnV0IEkg dGhpbmsgdGhpcyBhcHByb2FjaCB3YXMgY2hvc2VuIGZvcgo+ID4gc2ltcGxpY2l0eS4gIFdlIGNv dWxkIHN0aWxsIGRvIHNvbWV0aGluZyBsaWtlOgo+ID4gCj4gPiBpbnQgbWRldl9zZXRfY2xhc3Nf c3RydWN0KHN0cnVjdCBkZXZpY2UgKmRldiwgY29uc3Qgc3RydWN0IG1kZXZfY2xhc3Nfc3RydWN0 Cj4gPiAqY2xhc3MpOwo+ID4gCj4gPiBzdHJ1Y3QgbWRldl9jbGFzc19zdHJ1Y3Qgewo+ID4gCXUx NglpZDsKPiA+IAl1bmlvbiB7Cj4gPiAJCXN0cnVjdCB2ZmlvX21kZXZfb3BzIHZmaW9fb3BzOwo+ ID4gCQlzdHJ1Y3QgdmlydGlvX21kZXZfb3BzIHZpcnRpb19vcHM7Cj4gPiAJfTsKPiA+IH07Cj4g PiAKPiA+IE1heWJlIGV2ZW46Cj4gPiAKPiA+IHN0cnVjdCB2ZmlvX21kZXZfb3BzICptZGV2X2dl dF92ZmlvX29wcyhzdHJ1Y3QgbWRldl9kZXZpY2UgKm1kZXYpIHsKPiA+IAlCVUdfT04obWRldi0+ Y2xhc3MuaWQgIT0gTURFVl9JRF9WRklPKTsKPiA+IAlyZXR1cm4gJm1kZXYtPmNsYXNzLnZmaW9f b3BzOwo+ID4gfQo+ID4gCj4gPiBUaGUgbWF0Y2ggY2FsbGJhY2sgd291bGQgb2YgY291cnNlIGp1 c3QgdXNlIHRoZSBtZGV2LT5jbGFzcy5pZCB2YWx1ZS4KPiA+IEZ1bmN0aW9uYWxseSBlcXVpdmFs ZW50LCBidXQgbWF5YmUgYmV0dGVyIHR5cGUgY2hhcmFjdGVyaXN0aWNzLiAgVGhhbmtzLAo+ID4g Cj4gPiBBbGV4ICAKPiAKPiBXZSBoYXZlIDMgdXNlIGNhc2VzIG9mIG1kZXYuCj4gMS4gY3VycmVu dCBtZGV2IGJpbmRpbmcgdG8gdmZpb19tZGV2Cj4gMi4gbWRldiBiaW5kaW5nIHRvIHZpcnRpbwo+ IDMuIG1kZXYgYmluZGluZyB0byBtbHg1X2NvcmUgd2l0aG91dCBkZXZfb3BzCj4gCj4gQWxzbyAK PiAoYSkgYSBnaXZlbiBwYXJlbnQgbWF5IHNlcnZlIG11bHRpcGxlIHR5cGVzIG9mIGNsYXNzZXMg aW4gZnV0dXJlLgo+IChiKSBudW1iZXIgb2YgY2xhc3NlcyBtYXkgbm90IGxpa2VseSBleHBsb2Rl LCB0aGV5IHdpbGwgYmUgaGFuZGZ1bCBvZiB0aGVtLiAodmZpb19tZGV2LCB2aXJ0aW8pCj4gCj4g U28sIGluc3RlYWQgb2YgbWFraW5nIGNvcGllcyBvZiB0aGlzIGRldl9vcHMgcG9pbnRlciBpbiBl YWNoIG1kZXYsIGl0IGlzIGJldHRlciB0byBrZWVwIGNvbnN0IG11bHRpcGxlIG9wcyBpbiB0aGVp ciBwYXJlbnQgZGV2aWNlLgo+IFNvbWV0aGluZyBsaWtlIGJlbG93LAo+IAo+IHN0cnVjdCBtZGV2 X3BhcmVudCB7Cj4gCVsuLl0KPiAJc3RydWN0IG1kZXZfcGFyZW50X29wcyAqcGFyZW50X29wczsg LyogY3JlYXRlLCByZW1vdmUgKi8KPiAJc3RydWN0IHZmaW9fbWRldl9vcHMgKnZmaW9fb3BzOyAv KiByZWFkLHdyaXRlLCBpb2N0bCBldGMgKi8KPiAJc3RydWN0IHZpcnRpb19tZGV2X29wcyAqdmly dGlvX29wczsgLyogdmlydGlvIG9wcyAqLwo+IH07CgpUaGF0IGZlZWxzIGEgYml0IG9kZC4gV2h5 IHNob3VsZCB0aGUgcGFyZW50IGNhcnJ5IHBvaW50ZXJzIHRvIGV2ZXJ5CnBvc3NpYmxlIHZlcnNp b24gb2Ygb3BzPwoKPiAKPiBjb25zdCBzdHJ1Y3QgdmZpb19tZGV2X29wcyAqbWRldl9nZXRfdmZp b19vcHMoc3RydWN0IG1kZXZfcGFyZW50ICpwYXJlbnQpOwo+IGNvbnN0IHN0cnVjdCB2aXJ0aW9f bWRldl9vcHMgKm1kZXZfZ2V0X3ZpcnRpb19vcHMoc3RydWN0IG1kZXZfcGFyZW50ICpwYXJlbnQp Owo+IAo+IFRoaXMgd2F5LCAKPiAoYSkgd2UgaGF2ZSBzdHJvbmcgdHlwZSBjaGVjayBzdXBwb3J0 Cj4gKGIpIG9wcyBwb2ludGVyIGlzIG5vdCBkdXBsaWNhdGVkIGFjcm9zcyBzZXZlcmFsIGh1bmRy ZWQgbWRldiBkZXZpY2VzLCBhbmQgZG9uJ3QgaGF2ZSB0byBzZXQgb24gZXZlcnkgbWRldiBjcmVh dGlvbgo+IChjKSBhbGwgMyBjbGFzc2VzIG9mIG1kZXYgYXJlIHN1cHBvcnRlZAo+IChkKSBvbmUg ZXh0cmEgc3ltYm9sIHRhYmxlIGVudHJ5IHVzZWQgcGVyIG9wcyB0eXBlLCBidXQgdGhlcmUgYXJl IG5vdCBleHBlY3RlZCB0byBncm93IGEgbG90Lgo+IChlKSBtdWx0aXBsZSBjbGFzc2VzIHBlciBz aW5nbGUgcGFyZW50IGlzIHN0aWxsIHN1cHBvcnRlZAo+IChmKSBzdGlsbCBleHRlbmRpYmxlIGZv ciBtdWx0aXBsZSBjbGFzc2VzICh3ZWxsIGRlZmluZWQgY2xhc3NlcyA9IHZmaW8sIHZpcnRpbywg YW5kIHZlbmRvciBjbGFzcykKCllldCBhbm90aGVyIHN1Z2dlc3Rpb246IGhhdmUgdGhlIGNsYXNz IGlkIGRlcml2ZSBmcm9tIHRoZSBmdW5jdGlvbiB5b3UKdXNlIHRvIHNldCB1cCB0aGUgb3BzLgoK dm9pZCBtZGV2X3NldF92ZmlvX29wcyhzdHJ1Y3QgbWRldl9kZXZpY2UgKm1kZXYsIGNvbnN0IHN0 cnVjdCB2ZmlvX21kZXZfb3BzICp2ZmlvX29wcykKewoJbWRldi0+ZGV2aWNlX29wcyA9IHZmaW9f b3BzOwoJbWRldi0+Y2xhc3NfaWQgPSBNREVWX0lEX1ZGSU87Cn0KCnZvaWQgbWRldl9zZXRfdmly dGlvX29wcyhzdHJ1Y3QgbWRldl9kZXZpY2UgKm1kZXYsIGNvbnN0IHN0cnVjdCB2aXJ0aW9fbWRl dl9vcHMgKnZpcnRpb19vcHMpCnsKCW1kZXYtPmRldmljZV9vcHMgPSB2aXJ0aW9fb3BzOwoJbWRl di0+Y2xhc3NfaWQgPSBNREVWX0lEX1ZJUlRJTzsKfQoKdm9pZCBtZGV2X3NldF92aG9zdF9vcHMo c3RydWN0IG1kZXZfZGV2aWNlICptZGV2LCBjb25zdCBzdHJ1Y3QgdmlydGlvX21kZXZfb3BzICp2 aXJ0aW9fb3BzKQp7CgltZGV2LT5kZXZpY2Vfb3BzID0gdmlydGlvX29wczsKCW1kZXYtPmNsYXNz X2lkID0gTURFVl9JRF9WSE9TVDsKfQoKdm9pZCBtZGV2X3NldF92ZW5kb3Jfb3BzKHN0cnVjdCBt ZGV2X2RldmljZSAqbWRldikgLyogbm8gb3BzICovCnsKCW1kZXYtPmNsYXNzX2lkID0gTURFVl9J RF9WRU5ET1I7Cn0KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18KVmlydHVhbGl6YXRpb24gbWFpbGluZyBsaXN0ClZpcnR1YWxpemF0aW9uQGxpc3RzLmxpbnV4 LWZvdW5kYXRpb24ub3JnCmh0dHBzOi8vbGlzdHMubGludXhmb3VuZGF0aW9uLm9yZy9tYWlsbWFu L2xpc3RpbmZvL3ZpcnR1YWxpemF0aW9u