From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Tan, Jianfeng" Subject: Re: [PATCH] vhost_user: protect active rings from async ring changes Date: Fri, 8 Dec 2017 08:51:57 +0000 Message-ID: References: <20171206135329.652-1-vkaplans@redhat.com> <00a522dc-2287-b3ed-7ca4-e666f6217c91@redhat.com> <7352a09d-f90d-5b3a-51f9-bfc82faf744e@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Cc: "stable@dpdk.org" , "jfreiman@redhat.com" To: Maxime Coquelin , Victor Kaplansky , "dev@dpdk.org" , "yliu@fridaylinux.org" , "Bie, Tiwei" Return-path: In-Reply-To: <7352a09d-f90d-5b3a-51f9-bfc82faf744e@redhat.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogTWF4aW1lIENvcXVlbGlu IFttYWlsdG86bWF4aW1lLmNvcXVlbGluQHJlZGhhdC5jb21dDQo+IFNlbnQ6IEZyaWRheSwgRGVj ZW1iZXIgOCwgMjAxNyA0OjM2IFBNDQo+IFRvOiBUYW4sIEppYW5mZW5nOyBWaWN0b3IgS2FwbGFu c2t5OyBkZXZAZHBkay5vcmc7IHlsaXVAZnJpZGF5bGludXgub3JnOyBCaWUsDQo+IFRpd2VpDQo+ IENjOiBzdGFibGVAZHBkay5vcmc7IGpmcmVpbWFuQHJlZGhhdC5jb20NCj4gU3ViamVjdDogUmU6 IFtQQVRDSF0gdmhvc3RfdXNlcjogcHJvdGVjdCBhY3RpdmUgcmluZ3MgZnJvbSBhc3luYyByaW5n DQo+IGNoYW5nZXMNCj4gDQo+IA0KPiANCj4gT24gMTIvMDgvMjAxNyAwMzoxNCBBTSwgVGFuLCBK aWFuZmVuZyB3cm90ZToNCj4gPg0KPiA+DQo+ID4+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0t DQo+ID4+IEZyb206IE1heGltZSBDb3F1ZWxpbiBbbWFpbHRvOm1heGltZS5jb3F1ZWxpbkByZWRo YXQuY29tXQ0KPiA+PiBTZW50OiBUaHVyc2RheSwgRGVjZW1iZXIgNywgMjAxNyA2OjAyIFBNDQo+ ID4+IFRvOiBUYW4sIEppYW5mZW5nOyBWaWN0b3IgS2FwbGFuc2t5OyBkZXZAZHBkay5vcmc7IHls aXVAZnJpZGF5bGludXgub3JnOw0KPiBCaWUsDQo+ID4+IFRpd2VpDQo+ID4+IENjOiBzdGFibGVA ZHBkay5vcmc7IGpmcmVpbWFuQHJlZGhhdC5jb20NCj4gPj4gU3ViamVjdDogUmU6IFtQQVRDSF0g dmhvc3RfdXNlcjogcHJvdGVjdCBhY3RpdmUgcmluZ3MgZnJvbSBhc3luYyByaW5nDQo+ID4+IGNo YW5nZXMNCj4gPj4NCj4gPj4NCj4gPj4NCj4gPj4gT24gMTIvMDcvMjAxNyAxMDozMyBBTSwgVGFu LCBKaWFuZmVuZyB3cm90ZToNCj4gPj4+DQo+ID4+Pg0KPiA+Pj4+IC0tLS0tT3JpZ2luYWwgTWVz c2FnZS0tLS0tDQo+ID4+Pj4gRnJvbTogVmljdG9yIEthcGxhbnNreSBbbWFpbHRvOnZrYXBsYW5z QHJlZGhhdC5jb21dDQo+ID4+Pj4gU2VudDogV2VkbmVzZGF5LCBEZWNlbWJlciA2LCAyMDE3IDk6 NTYgUE0NCj4gPj4+PiBUbzogZGV2QGRwZGsub3JnOyB5bGl1QGZyaWRheWxpbnV4Lm9yZzsgQmll LCBUaXdlaTsgVGFuLCBKaWFuZmVuZzsNCj4gPj4+PiB2a2FwbGFuc0ByZWRoYXQuY29tDQo+ID4+ Pj4gQ2M6IHN0YWJsZUBkcGRrLm9yZzsgamZyZWltYW5AcmVkaGF0LmNvbTsgTWF4aW1lIENvcXVl bGluDQo+ID4+Pj4gU3ViamVjdDogW1BBVENIXSB2aG9zdF91c2VyOiBwcm90ZWN0IGFjdGl2ZSBy aW5ncyBmcm9tIGFzeW5jIHJpbmcNCj4gY2hhbmdlcw0KPiA+Pj4+DQo+ID4+Pj4gV2hlbiBwZXJm b3JtaW5nIGxpdmUgbWlncmF0aW9uIG9yIG1lbW9yeSBob3QtcGx1Z2dpbmcsDQo+ID4+Pj4gdGhl IGNoYW5nZXMgdG8gdGhlIGRldmljZSBhbmQgdnJpbmdzIG1hZGUgYnkgbWVzc2FnZSBoYW5kbGVy DQo+ID4+Pj4gZG9uZSBpbmRlcGVuZGVudGx5IGZyb20gdnJpbmcgdXNhZ2UgYnkgUE1EIHRocmVh ZHMuDQo+ID4+Pj4NCj4gPj4+PiBUaGlzIGNhdXNlcyBmb3IgZXhhbXBsZSBzZWdmYXVscyBkdXJp bmcgbGl2ZS1taWdyYXRpb24NCj4gPj4+DQo+ID4+PiBzZWdmYXVscyAtPnNlZ2ZhdWx0cz8NCj4g Pj4+DQo+ID4+Pj4gd2l0aCBNUSBlbmFibGUsIGJ1dCBpbiBnZW5lcmFsIHZpcnR1YWxseSBhbnkg cmVxdWVzdA0KPiA+Pj4+IHNlbnQgYnkgcWVtdSBjaGFuZ2luZyB0aGUgc3RhdGUgb2YgZGV2aWNl IGNhbiBjYXVzZQ0KPiA+Pj4+IHByb2JsZW1zLg0KPiA+Pj4+DQo+ID4+Pj4gVGhlc2UgcGF0Y2hl cyBmaXhlcyBhbGwgYWJvdmUgaXNzdWVzIGJ5IGFkZGluZyBhIHNwaW5sb2NrDQo+ID4+Pj4gdG8g ZXZlcnkgdnJpbmcgYW5kIHJlcXVpcmluZyBtZXNzYWdlIGhhbmRsZXIgdG8gc3RhcnQgb3BlcmF0 aW9uDQo+ID4+Pj4gb25seSBhZnRlciBlbnN1cmluZyB0aGF0IGFsbCBQTUQgdGhyZWFkcyByZWxh dGVkIHRvIHRoZSBkaXZlY2UNCj4gPj4+DQo+ID4+PiBBbm90aGVyIHR5cG86IGRpdmVjZS4NCj4g Pj4+DQo+ID4+Pj4gYXJlIG91dCBvZiBjcml0aWNhbCBzZWN0aW9uIGFjY2Vzc2luZyB0aGUgdnJp bmcgZGF0YS4NCj4gPj4+Pg0KPiA+Pj4+IEVhY2ggdnJpbmcgaGFzIGl0cyBvd24gbG9jayBpbiBv cmRlciB0byBub3QgY3JlYXRlIGNvbnRlbnRpb24NCj4gPj4+PiBiZXR3ZWVuIFBNRCB0aHJlYWRz IG9mIGRpZmZlcmVudCB2cmluZ3MgYW5kIHRvIHByZXZlbnQNCj4gPj4+PiBwZXJmb3JtYW5jZSBk ZWdyYWRhdGlvbiBieSBzY2FsaW5nIHF1ZXVlIHBhaXIgbnVtYmVyLg0KPiA+Pj4NCj4gPj4+IEFs c28gd29uZGVyIGhvdyBtdWNoIG92ZXJoZWFkIGl0IGJyaW5ncy4NCj4gPj4+DQo+ID4+PiBJbnN0 ZWFkIG9mIGxvY2tpbmcgZWFjaCB2cmluZywgY2FuIHdlIGp1c3QsIHdhaXRpbmcgYSB3aGlsZSAo MTB1cyBmb3INCj4gZXhhbXBsZSkNCj4gPj4gYWZ0ZXIgY2FsbCBkZXN0cm95X2RldmljZSgpIGNh bGxiYWNrIHNvIHRoYXQgZXZlcnkgUE1EIHRocmVhZCBoYXMgZW5vdWdoDQo+ID4+IHRpbWUgdG8g c2tpcCBvdXQgdGhlIGNyaXRlcmlhbCBhcmVhPw0KPiA+Pg0KPiA+PiBObywgYmVjYXVzZSB3ZSBh cmUgbm90IGRlc3Ryb3lpbmcgdGhlIGRldmljZSB3aGVuIGl0IGlzIG5lZWRlZC4NCj4gPj4gQWN0 dWFsbHksIG9uY2UgZGVzdHJveV9kZXZpY2UoKSBpcyBjYWxsZWQsIGl0IGlzIGxpa2VseSB0aGF0 IHRoZQ0KPiA+PiBhcHBsaWNhdGlvbiBoYXMgdGFrZW4gY2FyZSB0aGUgcmluZyBhcmVuJ3QgYmVp bmcgcHJvY2Vzc2VkIGFueW1vcmUNCj4gPj4gYmVmb3JlIHJldHVybmluZyBmcm9tIHRoZSBjYWxs YmFjayAoVGhpcyBpcyBhdCBsZWFzdCB0aGUgY2FzZSB3aXRoIFZob3N0DQo+ID4+IFBNRCkuDQo+ ID4NCj4gPiBPSywgSSBkaWQgbm90IHB1dCBpdCByaWdodCB3YXkgYXMgdGhlcmUgYXJlIG11bHRp cGxlIGNhc2VzIGFib3ZlOiBtaWdyYXRpb24NCj4gYW5kIG1lbW9yeSBob3QgcGx1Zy4gTGV0IG1l IHRyeSBhZ2FpbjoNCj4gPg0KPiA+IFdoZW5ldmVyIGEgdmhvc3QgdGhyZWFkIGhhbmRsZXMgYSBt ZXNzYWdlIGFmZmVjdGluZyBQTUQgdGhyZWFkcywgKGxpa2UNCj4gU0VUX01FTV9UQUJMRSwgR0VU X1ZSSU5HX0JBU0UsIGV0Yykgd2UgY2FuIHJlbW92ZSB0aGUgZGV2IGZsYWcgLQ0KPiBWSVJUSU9f REVWX1JVTk5JTkcsIGFuZCB3YWl0IGZvciBhIHdoaWxlIHNvIHRoYXQgUE1EIHRocmVhZHMgc2tp cCBvdXQgb2YNCj4gdGhvc2UgY3JpdGVyaWFsIGFyZWEuIEFmdGVyIG1lc3NhZ2UgaGFuZGxpbmcs IHJlc2V0IHRoZSBmbGFnIC0NCj4gVklSVElPX0RFVl9SVU5OSU5HLg0KPiANCj4gSSB0aGluayB5 b3UgbWVhbiBjbGVhcmluZyB2cSdzIGVuYWJsZWQgZmxhZywgYmVjYXVzZSBQTUQgdGhyZWFkcyBu ZXZlcg0KPiBjaGVjayB0aGUgVklSVElPX0RFVl9SVU5OSU5HIGZsYWcuDQoNCkFoLCB5ZXMuDQoN Cj4gDQo+ID4gSSBzdXBwb3NlIGl0IGNhbiB3b3JrLCBiYXNpbmcgb24gYW4gYXNzdW1wdGlvbiB0 aGF0IFBNRCB0aHJlYWRzIHdvcmsgaW4NCj4gcG9sbGluZyBtb2RlIGFuZCBjYW4gc2tpcCBjcml0 ZXJpYWwgYXJlYSBxdWlja2x5IGFuZCBpbmV2aXRhYmx5Lg0KPiANCj4gVGhhdCBzb3VuZHMgdmVy eSBmcmFnaWxlLCBiZWNhdXNlIGlmIHRoZSBDUFUgYXJlbid0IHBlcmZlY3RseSBpc29sYXRlZCwN Cj4geW91ciBQTUQgdGhyZWFkIGNhbiBiZSBwcmVlbXB0ZWQgZm9yIGludGVycnVwdCBoYW5kbGlu ZyBmb3IgZXhhbXBsZS4NCj4gDQo+IE9yIHdoYXQgaWYgZm9yIHNvbWUgcmVhc29uIHRoZSBQTUQg dGhyZWFkIENQVSBzdGFsbHMgZm9yIGEgc2hvcnQgd2hpbGU/DQo+IA0KPiBUaGUgbGF0ZXIgaXMg dW5saWtlbHksIGJ1dCBpZiBpdCBoYXBwZW5zLCBpdCB3aWxsIGJlIGhhcmQgdG8gZGVidWcuDQo+ IA0KPiBMZXQncyBzZWUgZmlyc3QgdGhlIHBlcmZvcm1hbmNlIGltcGFjdCBvZiB1c2luZyB0aGUg c3BpbmxvY2suIEl0IG1pZ2h0DQo+IG5vdCBiZSB0aGF0IGltcG9ydGFudCBiZWNhdXNlIDk5Ljk5 OTklIG9mIHRoZSB0aW1lcywgaXQgd2lsbCBub3QgZXZlbg0KPiBzcGluLg0KDQpGYWlyIGVub3Vn aC4NCg0KVGhhbmtzLA0KSmlhbmZlbmcNCg0KPiANCj4gVGhhbmtzLA0KPiBNYXhpbWUNCj4gDQo+ ID4+DQo+ID4+IFRoZSByZWFzb24gd2UgbmVlZCB0aGUgbG9jayBpcyB0byBwcm90ZWN0IFBNRCB0 aHJlYWRzIGZyb20gdGhlIGhhbmRsaW5nDQo+ID4+IG9mIHNvbWUgdmhvc3QtdXNlciBwcm90b2Nv bCByZXF1ZXN0cy4NCj4gPj4gRm9yIGV4YW1wbGUgU0VUX01FTV9UQUJMRSBpbiB0aGUgY2FzZSBv ZiBtZW1vcnkgaG90cGx1Zywgb3INCj4gPj4gU0VUX0xPR19CQVNFDQo+ID4+IGluIGNhc2Ugb2Yg bXVsdGlxdWV1ZSwgd2hpY2ggaXMgc2VudCBmb3IgZXZlcnkgcXVldWUgcGFpciBhbmQgcmVzdWx0 cyBpbg0KPiA+PiB1bm1hcHBpbmcvcmVtYXBwaW5nIHRoZSBsb2dnaW5nIGFyZWEuDQo+ID4NCj4g PiBZZXMsIEkgdW5kZXJzdGFuZCBob3cgaXQgZmFpbHMuDQo+ID4NCj4gPiBUaGFua3MsDQo+ID4g SmlhbmZlbmcNCj4gPg0KPiA+Pg0KPiA+PiBNYXhpbWUNCg==