From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id 5A8C07F3F for ; Sun, 29 Dec 2013 07:19:26 -0600 (CST) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay1.corp.sgi.com (Postfix) with ESMTP id 46CED8F8035 for ; Sun, 29 Dec 2013 05:19:23 -0800 (PST) Message-ID: <52C02151.7020904@oracle.com> Date: Sun, 29 Dec 2013 21:19:13 +0800 From: Jeff Liu MIME-Version: 1.0 Subject: Re: [PATCH 2/4] xfs: always releasing EFD's reference to EFI in xfs_efd_item_committed References: <52B98295.8050704@oracle.com> <52BEFA2E.5070006@sgi.com> In-Reply-To: <52BEFA2E.5070006@sgi.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Mark Tinguely Cc: "xfs@oss.sgi.com" T24gMjAxM+W5tDEy5pyIMjnml6UgMDA6MTksIE1hcmsgVGluZ3VlbHkgd3JvdGU6Cj4gT24gMTIv MjQvMTMgMDY6NDgsIEplZmYgTGl1IHdyb3RlOgo+PiBGcm9tOiBKaWUgTGl1PGplZmYubGl1QG9y YWNsZS5jb20+Cj4+Cj4+IFdpdGggZnNzdHJlc3MrZ29kb3duIHRlc3QgSSBvYnNlcnZlZCBhbiBY RlMgaGFuZyB1cCBkdXJpbmcgdW1vdW50IHdoaWNoCj4+IHlpZWxkaW5nIGEgYmFja3RyYWNlIGxp a2UgYmVsb3c6Cj4+Cj4+IFsyMDg3Ni4xOTM2MzVdIElORk86IHRhc2sgdW1vdW50Ojk4NTMgYmxv Y2tlZCBmb3IgbW9yZSB0aGFuIDEyMCBzZWNvbmRzLgo+PiBbMjA4NzYuMTkzNjQxXSAgICAgICBU YWludGVkOiBQRiAgIE8gMy4xMy4wLXJjMisgIzgKPj4gWzIwODc2LjE5MzY0M10gImVjaG8gMD4g IC9wcm9jL3N5cy9rZXJuZWwvaHVuZ190YXNrX3RpbWVvdXRfc2VjcyIKPj4gZGlzYWJsZXMgdGhp cyBtZXNzYWdlLgo+PiBbMjA4NzYuMTkzNjQ1XSB1bW91bnQgICAgICAgICAgRCBmZmZmODgwMjZm Mjk0NDQwICAgICAwICA5ODUzICAgOTM3Mgo+PiA8c25pcD4KPj4gWzIwODc2LjE5MzY2M10gQ2Fs bCBUcmFjZToKPj4gWzIwODc2LjE5MzY3Ml0gIFs8ZmZmZmZmZmY4MTZmMzgyOT5dIHNjaGVkdWxl KzB4MjkvMHg3MAo+PiBbMjA4NzYuMTkzNzAxXSAgWzxmZmZmZmZmZmEwYzRhM2U5Pl0geGZzX2Fp bF9wdXNoX2FsbF9zeW5jKzB4YTkvMHhlMAo+PiBbeGZzXQo+PiBbMjA4NzYuMTkzNzA3XSAgWzxm ZmZmZmZmZjgxMGE4M2YwPl0gPyBwcmVwYXJlX3RvX3dhaXRfZXZlbnQrMHgxMDAvMHgxMDAKPj4g WzIwODc2LjE5MzcyNl0gIFs8ZmZmZmZmZmZhMGJlOWRmMT5dIHhmc191bm1vdW50ZnMrMHg2MS8w eDE1MCBbeGZzXQo+PiBbMjA4NzYuMTkzNzQ2XSAgWzxmZmZmZmZmZmEwYmVjZDQxPl0geGZzX2Zz X3B1dF9zdXBlcisweDIxLzB4NjAgW3hmc10KPj4gWzIwODc2LjE5Mzc1MV0gIFs8ZmZmZmZmZmY4 MTFiYmY2Mj5dIGdlbmVyaWNfc2h1dGRvd25fc3VwZXIrMHg3Mi8weGYwCj4+IFsyMDg3Ni4xOTM3 NTRdICBbPGZmZmZmZmZmODExYmMyMTc+XSBraWxsX2Jsb2NrX3N1cGVyKzB4MjcvMHg3MAo+PiBb MjA4NzYuMTkzNzU3XSAgWzxmZmZmZmZmZjgxMWJjNGZkPl0gZGVhY3RpdmF0ZV9sb2NrZWRfc3Vw ZXIrMHgzZC8weDYwCj4+IFsyMDg3Ni4xOTM3NjFdICBbPGZmZmZmZmZmODExYmNhYjY+XSBkZWFj dGl2YXRlX3N1cGVyKzB4NDYvMHg2MAo+PiBbMjA4NzYuMTkzNzY1XSAgWzxmZmZmZmZmZjgxMWQ5 MTQ2Pl0gbW50cHV0X25vX2V4cGlyZSsweGQ2LzB4MTcwCj4+IFsyMDg3Ni4xOTM3NjldICBbPGZm ZmZmZmZmODExZGE2NGU+XSBTeVNfdW1vdW50KzB4OGUvMHgxMDAKPj4gWzIwODc2LjE5Mzc3NF0g IFs8ZmZmZmZmZmY4MTZmZmQ2ZD5dIHN5c3RlbV9jYWxsX2Zhc3RwYXRoKzB4MWEvMHgxZgo+Pgo+ PiBBcyBwZXIgYWJvdmUgYmFja3RyYWNlcywgdGhlIHVtb3VudCBwcm9jZXNzIGlzIGFscmVhZHkg c2NoZWR1bGVkIG91dAo+PiBpbiB4ZnNfYWlsX3B1c2hfYWxsX3N5bmMoKSBiZWNhdXNlIGl0IHNo b3VsZCBwdXNoIG91dCBhbGwgb2YgcGVuZGluZwo+PiBjaGFuZ2VzIGluIEFJTCBhbmQgd2FpdCB1 bnRpbCB0aGUgQUlMIGlzIGVtcHR5LiAgVGhlbiBpdCB3aWxsIHdha2UgdXAKPj4geGZzYWlsZCB0 aHJlYWQgdG8gZG8gdGhlIGFjdHVhbCBmbHVzaGluZyBidXNpbmVzcy4gIEhvd2V2ZXIsIEkgZm91 bmQKPj4gdGhhdCB0aGUgQUlMIGRvZXMgbm90IGJlY2FtZSBlbXB0eSBpbiBzb21lIHNpdHVhdGlv bnMgYmVjYXVzZSBvZiBzb21lCj4+IEVGSSBhcmUgc3RpbGwgYmVpbmcgb24gaXQsIGJ1dCBpbiBF RkkncyBpb3BfcHVzaCBvcGVyYXRpb24sIHdlIGFsd2F5cwo+PiByZXR1cm5pbmcgWEZTX0lURU1f UElOTkVEIHdoaWNoIGxlYWRzIHRvIHRoZSB4ZnNhaWxkIHRocmVhZCBzdWZmZXJpbmcKPj4gaW50 byBhbiBpbmZpbml0ZSBsb29wLgo+Pgo+PiBTaW5jZSBFRkkgaXRlbXMgaGF2ZSBubyBsb2NraW5n IG9yIHB1c2hpbmcsIHRoZXkgYXJlIHB1bGxlZCBmcm9tIHRoZQo+PiBBSUwgd2hlbiB0aGVpciBj b3JyZXNwb25kaW5nIEVGRHMgYXJlIGNvbW1pdHRlZCB0byBkaXNrLCBhbmQgd2UgaGF2ZQo+PiBn dWFyYW50ZWVkIHRoYXQgdGhlIEVGSSBzaG91bGQgbm90IGJlIGZyZWVkIHVudGlsIGl0IGhhcyBi ZWVuIHVucHBpbmVkCj4+IGFuZCB0aGUgRUZEIGhhcyBiZWVuIGNvbW1pdHRlZCBpbiBjb21taXQg NjY2ZDY0NGNkNywgdGhpcyBpcyBkb25lIHZpYQo+PiBhbiBFRkkgcmVmZXJlbmNlIGNvdW50IGJ5 IGluaXRpYWxpemluZyBpdCB0byAyIGluIHhmc19lZmlfaW5pdCgpIC0tIG9uZQo+PiBpcyBpdCdz IG93biBjb3VudCB3aGljaCBpcyBub3QgcmVsZWFzZWQgdW50aWwgaXQgaXMgdW5waW5uZWQsIHRo ZSBvdGhlcgo+PiBvbmUgaXMgdGFrZW4gYnkgaXRzIGNvcnJlc3BvbmRpbmcgRUZEIHdoaWNoIHdp bGwgYmUgcmVsZWFzZWQgZHVyaW5nIEVGRAo+PiBjb21taXQgb3BlcmF0aW9uLgo+Pgo+PiBJTUhP IHdlIHNob3VsZCBhbHdheXMgcmVsZWFzaW5nIGl0J3MgcmVmZXJlbmNlIHRvIHRoZSBjb3JyZXNw b25kaW5nIEVGSQo+PiBpdGVtIG9uY2UgdGhlIEVGRCBpdGVtIGlzIGNvbW1pdHRlZCB0byBkaXNr IHJlZ2FyZGxlc3Mgb2YgdGhlIGxvZyBpdGVtCj4+IGlzIG1hcmtlZCB3aXRoIFhGU19MSV9BQk9S VEVEIGZsYWcgb3Igbm90Lgo+Pgo+PiBTaWduZWQtb2ZmLWJ5OiBKaWUgTGl1PGplZmYubGl1QG9y YWNsZS5jb20+Cj4+IC0tLQo+PiAgIGZzL3hmcy94ZnNfZXh0ZnJlZV9pdGVtLmMgfCA4ICstLS0t LS0tCj4+ICAgMSBmaWxlIGNoYW5nZWQsIDEgaW5zZXJ0aW9uKCspLCA3IGRlbGV0aW9ucygtKQo+ Pgo+PiBkaWZmIC0tZ2l0IGEvZnMveGZzL3hmc19leHRmcmVlX2l0ZW0uYyBiL2ZzL3hmcy94ZnNf ZXh0ZnJlZV9pdGVtLmMKPj4gaW5kZXggMzY4MGQwNC4uMTZjMDM5NiAxMDA2NDQKPj4gLS0tIGEv ZnMveGZzL3hmc19leHRmcmVlX2l0ZW0uYwo+PiArKysgYi9mcy94ZnMveGZzX2V4dGZyZWVfaXRl bS5jCj4+IEBAIC00MzcsMTMgKzQzNyw3IEBAIHhmc19lZmRfaXRlbV9jb21taXR0ZWQoCj4+ICAg ewo+PiAgICAgICBzdHJ1Y3QgeGZzX2VmZF9sb2dfaXRlbSAgICAqZWZkcCA9IEVGRF9JVEVNKGxp cCk7Cj4+Cj4+IC0gICAgLyoKPj4gLSAgICAgKiBJZiB3ZSBnb3QgYSBsb2cgSS9PIGVycm9yLCBp dCdzIGFsd2F5cyB0aGUgY2FzZSB0aGF0IHRoZSBMUgo+PiB3aXRoIHRoZQo+PiAtICAgICAqIEVG SSBnb3QgdW5waW5uZWQgYW5kIGZyZWVkIGJlZm9yZSB0aGUgRUZEIGdvdCBhYm9ydGVkLgo+PiAt ICAgICAqLwo+PiAtICAgIGlmICghKGxpcC0+bGlfZmxhZ3MmICBYRlNfTElfQUJPUlRFRCkpCj4+ IC0gICAgICAgIHhmc19lZmlfcmVsZWFzZShlZmRwLT5lZmRfZWZpcCwgZWZkcC0+ZWZkX2Zvcm1h dC5lZmRfbmV4dGVudHMpOwo+PiAtCj4+ICsgICAgeGZzX2VmaV9yZWxlYXNlKGVmZHAtPmVmZF9l ZmlwLCBlZmRwLT5lZmRfZm9ybWF0LmVmZF9uZXh0ZW50cyk7Cj4+ICAgICAgIHhmc19lZmRfaXRl bV9mcmVlKGVmZHApOwo+PiAgICAgICByZXR1cm4gKHhmc19sc25fdCktMTsKPj4gICB9Cj4gCkhp IE1hcmssCgo+IEhpIEplZmYuCj4gCj4gVGhpcyB3b3VsZCB3b3JrIGlmIHRoZSBmb3JjZWQgc2h1 dGRvd24gaGFwcGVuZWQgYWZ0ZXIgYm90aCB0aGUgRUZJIGFuZAo+IEVGRCB0cmFuc2FjdGlvbiB3 ZXJlIGNvbW1pdHRlZCBhbmQgc3VjY2Vzc2Z1bGx5IHBsYWNlZCBvbiB0aGUgQ0lMLgpZZXAuCj4g SWYgdGhlIHNlcXVlbmNlIHdlbnQgRUZJIGNvbW1pdCwgQ0lMIHB1c2ggKEVGSSBpcyBub3cgaW4g dGhlIEFJTCksCj4gZm9yY2VkIHNodXRkb3duLCBhbmQgdGhlbiBFRkQgY29tbWl0LiBJbiB0aGlz IHNlcXVlbmNlLCB0aGUgRUZEIGl0ZW0KPiB3b3VsZCBub3QgYmUgcGxhY2VkIG9uIHRoZSBDSUwg YW5kIHRoZXJlZm9yZSB0aGUgaW9wLmNvbW1pdHRlZCB3b3VsZCBub3QKPiBiZSBjYWxsZWQuIElu IHRoaXMgcGF0Y2ggb25seSBpb3BfY29tbWl0dGluZyBhbmQgaW9wX3VubG9jayB3b3VsZCBiZSBy dW4KPiBvbiB0aGUgRUZEIGl0ZW0uCkFncmVlLCBpdCBzZWVtcyB3ZSBoYXZlIHRvIHJlbGVhc2Ug RUZJIHJlZmVyZW5jZSBjb3VudCBpbiB4ZnNfZWZkX2l0ZW1fdW5sb2NrKCkgaWYKWEZTX0xJX0FC T1JURUQgaXMgZGV0ZWN0ZWQuCgpIb3dldmVyLCB0aGUgc3Rvcnkgd2FzIG5vdCB5ZXQgY29tcGxl dGVkLCBJIGp1c3QgdGhvdWdodCBhbm90aGVyIHBvc3NpYmxlIG9yZGVyIGJldHdlZW4KRUZJL0VG RC4gIElmIEVGRCBpcyBjb21taXR0ZWQgaW50byBDSUwgYW5kIGp1c3QgYmVmb3JlIGNhbGxpbmcg eGZzX2VmZF9pdGVtX2NvbW1pdHRlZCgpLAp0aGUgRUZJIHRyYW5zYWN0aW9uIGNvbW1pdCBpcyBh Ym9ydGVkIGFuZCB0aHVzIHRoZSBFRkkgaXRlbSBpcyByZWxlYXNlZCBpbiBpdCdzIGlvcF91bmxv Y2soKQpjYWxsYmFjaywgaGVuY2UgbXkgY3VycmVudCBmaXggYWxzbyBoYXMgcHJvYmxlbSwgaS5l LCBpbiBpb3BfY29tbWl0dGVkKCksIGl0IHdpbGwgdHJ5IHRvIGRyb3AKYW4gRUZJIHJlZmVyZW5j ZSBjb3VudCB3aGljaCBpcyBhbHJlYWR5IHJlbGVhc2VkLgoKVGhpcyBwcm9ibGVtIHNlZW1zIGEg Yml0IGNvbXBsaWNhdGVkIHRoYW4gSSB0aG91Z2h0IGJlZm9yZSwgbGV0IG1lIHRoaW5rIGl0IG92 ZXIgb25jZSBiYWNrCmZyb20gdmFjYXRpb25zIHVudGlsIDAyLCBKYW4gLS0gbm8gZGV2ZWxvcG1l bnQgZW52aXJvbm1lbnQgb24gaGFuZC4uLgoKVGhhbmtzLAotSmVmZgoKX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KeGZzIG1haWxpbmcgbGlzdAp4ZnNAb3Nz LnNnaS5jb20KaHR0cDovL29zcy5zZ2kuY29tL21haWxtYW4vbGlzdGluZm8veGZzCg==