From mboxrd@z Thu Jan 1 00:00:00 1970 From: majianpeng Subject: Re: Re: [PATCH] ceph: fix sync read eof check deadlock Date: Tue, 24 Sep 2013 10:43:26 +0800 Message-ID: <201309241043226589572@gmail.com> References: <201309230939212660680@gmail.com>, Reply-To: majianpeng Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: base64 Return-path: Received: from mail-pb0-f46.google.com ([209.85.160.46]:38945 "EHLO mail-pb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752706Ab3IXCna (ORCPT ); Mon, 23 Sep 2013 22:43:30 -0400 Received: by mail-pb0-f46.google.com with SMTP id rq2so3977392pbb.33 for ; Mon, 23 Sep 2013 19:43:29 -0700 (PDT) Sender: ceph-devel-owner@vger.kernel.org List-ID: To: =?iso-8859-1?B?WWFuLCBaaGVuZw==?= Cc: sage , =?iso-8859-1?B?WWFuLCBaaGVuZw==?= , ceph-devel Pk9uIE1vbiwgU2VwIDIzLCAyMDEzIGF0IDk6MzkgQU0sIG1hamlhbnBlbmcgPG1hamlhbnBlbmdA Z21haWwuY29tPiB3cm90ZToNCj4+IEFzIFlhbixaaGVuZyBzYWlkLCBjb21taXQgMDkxMzQ0NDIw OGRiIGludHJ1b2R1Y2UgYSBidWc6ImdldGF0dHIgbmVlZCB0bw0KPj4gInJlYWQgbG9jayIgaW5v ZGUncyBmaWxlbG9jay4gQnV0IHRoZSBsb2NrIGNhbiBiZSBpbiB1bnN0YWJsZSBzdGF0ZS4NCj4+ IHRoZSBnZXRhdHRyIHJlcXVlc3Qgd2FpdHMgZm9yIGxvY2sncyBzdGF0ZSB0byBiZWNvbWUgc3Rh YmxlLCB0aGUgbG9jaw0KPj4gd2FpdHMgZm9yIGNsaWVudCB0byByZWxlYXNlIEZyIGNhcC4iDQo+ Pg0KPj4gQ29tbWl0IDZhMDI2NTg5YmEzMzMxODVjNDY2YzkwIHJlc29sdmVkIHRoZSBzYW1lIGJ1 ZyBhbHNvLg0KPj4gQmVmb3JlIGRvaW5nIGdldGF0dHIsIGl0IG11c3QgcHV0IHRoZSBjYXBzIHdo aWNoIGFscmVhZHkgaG9sZCBhdm9pZA0KPj4gZGVhZGxvY2suDQo+Pg0KPj4gUmVwb3J0ZWQtYnk6 IFlhbiwgWmhlbmcgPHpoZW5nLnoueWFuQGludGVsLmNvbT4NCj4+IFNpZ25lZC1vZmYtYnk6IEpp YW5wZW5nIE1hIDxtYWppYW5wZW5nQGdtYWlsLmNvbT4NCj4+IC0tLQ0KPj4gIGZzL2NlcGgvZmls ZS5jIHwgMjUgKysrKysrKysrKysrKysrKysrKystLS0tLQ0KPj4gIDEgZmlsZSBjaGFuZ2VkLCAy MCBpbnNlcnRpb25zKCspLCA1IGRlbGV0aW9ucygtKQ0KPj4NCj4+IGRpZmYgLS1naXQgYS9mcy9j ZXBoL2ZpbGUuYyBiL2ZzL2NlcGgvZmlsZS5jDQo+PiBpbmRleCA3ZGEzNWM3Li5iYzAwYWNlIDEw MDY0NA0KPj4gLS0tIGEvZnMvY2VwaC9maWxlLmMNCj4+ICsrKyBiL2ZzL2NlcGgvZmlsZS5jDQo+ PiBAQCAtODM5LDcgKzgzOSwxNSBAQCBhZ2FpbjoNCj4+ICAgICAgICAgICAgICAgICByZXQgPSBj ZXBoX3N5bmNfcmVhZChpb2NiLCAmaSwgJmNoZWNrZW9mKTsNCj4+DQo+PiAgICAgICAgICAgICAg ICAgaWYgKGNoZWNrZW9mICYmIHJldCA+PSAwKSB7DQo+PiAtICAgICAgICAgICAgICAgICAgICAg ICBpbnQgc3RhdHJldCA9IGNlcGhfZG9fZ2V0YXR0cihpbm9kZSwNCj4+ICsgICAgICAgICAgICAg ICAgICAgICAgIGludCBzdGF0cmV0Ow0KPj4gKyAgICAgICAgICAgICAgICAgICAgICAgLyoNCj4+ ICsgICAgICAgICAgICAgICAgICAgICAgICAqQmVmb3JlIGdldGF0dHIsaXQgc2hvdWxkIHB1dCBj YXBzIGF2b2lkDQo+PiArICAgICAgICAgICAgICAgICAgICAgICAgKmRlYWRsb2NrLg0KPj4gKyAg ICAgICAgICAgICAgICAgICAgICAgICovDQo+PiArICAgICAgICAgICAgICAgICAgICAgICBjZXBo X3B1dF9jYXBfcmVmcyhjaSwgZ290KTsNCj4+ICsgICAgICAgICAgICAgICAgICAgICAgIGdvdCA9 IDA7DQo+PiArDQo+PiArICAgICAgICAgICAgICAgICAgICAgICBzdGF0cmV0ID0gY2VwaF9kb19n ZXRhdHRyKGlub2RlLA0KPj4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgQ0VQSF9TVEFUX0NBUF9TSVpFKTsNCj4+DQo+PiAgICAgICAgICAgICAg ICAgICAgICAgICAvKiBoaXQgRU9GIG9yIGhvbGU/ICovDQo+PiBAQCAtODUxLDE2ICs4NTksMjMg QEAgYWdhaW46DQo+Pg0KPj4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICByZWFkICs9 IHJldDsNCj4+ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgY2hlY2tlb2YgPSAwOw0K Pj4gLSAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBnb3RvIGFnYWluOw0KPj4gKyAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICByZXQgPSBjZXBoX2dldF9jYXBzKGNpLCBDRVBIX0NB UF9GSUxFX1JELA0KPj4gKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICB3YW50LCAmZ290LCAtMSk7DQo+PiArICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgIGlmIChyZXQgPCAwKQ0KPj4gKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgIHJldCA9IDA7DQo+PiArICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIGVs c2UNCj4+ICsgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBnb3RvIGFnYWlu Ow0KPg0KPkkgdGhpbmsgd2Ugc2hvdWxkIHRyeSBnZXR0aW5nIEZyYyBjYXBzIGhlcmUsIGJlY2F1 c2UgbWRzIG1heSBpc3N1ZSBGYw0KPmNhcCB0byB0aGUgY2xpZW50IHdoaWxlIHJlcXVlc3Rpbmcg dGhlIHNpemUuDQo+DQpZZXMsaXQgbWF5YmUgZ2V0IEZjLk15IHRob3VnaCB3aGV0aGVyIG9yIG5v dCwgdGhlIGxhdGVyIHJlYWQgb25seSB1c2luZyBzeW5jX21vZGUuDQpUaGUgcmVhc29uOg0KQTpm b3IgcmVxdWVzdGluZyBzaXplIG9ubHkgZm9yIHN5bmMtcmVhZCBub3QgY2FjaGUtcmVhZA0KQjpC eSB0aGUgb3JpZ2luYWwsIGl0IHdpbGwgbWFrZSB0aGUgY2FjaGUtcmVhZCB0byBkbyB0aGUgc2Ft ZSB0aGluZyB3aGljaCBzeW5jLXJlYWQgYWxyZWFkIGRvbmUuDQoNClRoYW5rcyENCkppYW5wZW5n IE1hDQo+WW91ciBwcmV2aW91cyBwYXRjaCBtb3ZlZCB0aGUgZ2V0YXR0ciBjb2RlIHRvIGhlcmUu IHBsZWFzZSByZXZlcnQgdGhlDQo+Y29kZSB0byBpdHMgb3JpZ2luYWwgc2hhcGUuIEkgdGhpbmsg aXQgZG9lcyB0aGUgcmlnaHQgdGhpbmcuDQo+DQo+DQo+UmVnYXJkcw0KPllhbiwgWmhlbmcNCj4N Cj4NCj4+ICAgICAgICAgICAgICAgICAgICAgICAgIH0NCj4+ICAgICAgICAgICAgICAgICB9DQo+ Pg0KPj4gICAgICAgICB9IGVsc2UNCj4+ICAgICAgICAgICAgICAgICByZXQgPSBnZW5lcmljX2Zp bGVfYWlvX3JlYWQoaW9jYiwgaW92LCBucl9zZWdzLCBwb3MpOw0KPj4NCj4+IC0gICAgICAgZG91 dCgiYWlvX3JlYWQgJXAgJWxseC4lbGx4IGRyb3BwaW5nIGNhcCByZWZzIG9uICVzID0gJWRcbiIs DQo+PiAtICAgICAgICAgICAgaW5vZGUsIGNlcGhfdmlub3AoaW5vZGUpLCBjZXBoX2NhcF9zdHJp bmcoZ290KSwgKGludClyZXQpOw0KPj4gLSAgICAgICBjZXBoX3B1dF9jYXBfcmVmcyhjaSwgZ290 KTsNCj4+ICsgICAgICAgaWYgKGdvdCkgew0KPj4gKyAgICAgICAgICAgICAgIGRvdXQoImFpb19y ZWFkICVwICVsbHguJWxseCBkcm9wcGluZyBjYXAgcmVmcyBvbiAlcyA9ICVkXG4iLA0KPj4gKyAg ICAgICAgICAgICAgICAgICAgaW5vZGUsIGNlcGhfdmlub3AoaW5vZGUpLCBjZXBoX2NhcF9zdHJp bmcoZ290KSwgKGludClyZXQpOw0KPj4gKyAgICAgICAgICAgICAgIGNlcGhfcHV0X2NhcF9yZWZz KGNpLCBnb3QpOw0KPj4gKyAgICAgICB9DQo+DQo+DQo+DQo+DQo+Pg0KPj4gICAgICAgICBpZiAo cmV0ID49IDApDQo+PiAgICAgICAgICAgICAgICAgcmV0ICs9IHJlYWQ7DQo+PiAtLQ0KPj4gMS44 LjQtcmMw