https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-6159 Lustre <p>There's no point in ignoring CL_CLOSE by default in changelogs. Robinhood needs these events else the database quickly becomes out of sync. It's absence has created problem with our users who forgot to add them. So let's have it by default, which is IMO a sensitive choice.</p> <p>Note that CL_CLOSE is only issued when the file was opened in writing mode.</p> LU-6159

Set CL_CLOSE in default changelog mask

Improvement Minor Resolved Fixed WC Triage Frank Zago patch Mon, 26 Jan 2015 17:47:30 +0000 Fri, 13 May 2016 05:36:04 +0000 Mon, 8 Jun 2015 23:01:37 +0000 Lustre 2.7.0 Lustre 2.5.3 Lustre 2.8.0 0 5 <p>frank zago (fzago@cray.com) uploaded a new patch: <a href="http://review.whamcloud.com/13526" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13526</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6159" title="Set CL_CLOSE in default changelog mask" class="issue-link" data-issue-key="LU-6159"><del>LU-6159</del></a> hsm: add CL_CLOSE to default changelog mask<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 56c9f07aae9229c83d6655bc5e96362a6958be85</p> <p>v4 of this patch is causing the MDS to crash in test_232. <br/> In mdd_changelog_data_store(), uc is NULL, so its dereference causes the oops.<br/> I will pamper over it it the next version, but I don't know whether that's the correct fix.</p> <p>Note that just running test_232 doesn't cause the oops. It's a combination of some<br/> of the previous tests, plus the umount_client in test_232 that creates the issue.</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>&lt;1&gt;BUG: unable to handle kernel NULL pointer dereference at 0000000000000048 &lt;1&gt;IP: [&lt;ffffffffa0fe1f0f&gt;] mdd_changelog_data_store+0x1cf/0x390 [mdd] &lt;4&gt;PGD 7c4d8067 PUD 7c4d9067 PMD 0 &lt;4&gt;Oops: 0000 [#1] SMP &lt;4&gt;last sysfs file: /sys/devices/system/cpu/online &lt;4&gt;CPU 2 &lt;4&gt;Modules linked in: lustre(U) ofd(U) osp(U) lod(U) ost(U) mdt(U) mdd(U) mgs(U) osd_ldiskfs(U) ldiskfs(U) lquota(U) lfsck(U) mgc(U) lov(U) osc(U) mdc(U) lmv(U) fid(U) fld(U) ptlrpc_gss(U) ptlrpc(U) obdclass(U) ksocklnd(U) lnet(U) libcfs(U) ext2 exportfs jbd sunrpc sha512_generic sha256_generic ebtable_nat ebtables ipt_MASQUERADE iptable_nat nf_nat xt_CHECKSUM iptable_mangle bridge stp llc autofs4 ipt_REJECT nf_conntrack_ipv4 nf_defrag_ipv4 iptable_filter ip_tables ip6t_REJECT nf_conntrack_ipv6 nf_defrag_ipv6 xt_state nf_conntrack ip6table_filter ip6_tables ipv6 zfs(P)(U) zcommon(P)(U) znvpair(P)(U) zavl(P)(U) zunicode(P)(U) spl(U) zlib_deflate vhost_net macvtap macvlan tun microcode sg virtio_balloon virtio_net snd_hda_intel snd_hda_codec snd_hwdep snd_seq snd_seq_device snd_pcm snd_timer snd soundcore snd_page_alloc i2c_piix4 i2c_core ext4 jbd2 mbcache virtio_blk sr_mod cdrom virtio_pci virtio_ring virtio pata_acpi ata_generic ata_piix dm_mirror dm_region_hash dm_log dm_mod [last unloaded: obdecho] &lt;4&gt; &lt;4&gt;Pid: 10317, comm: mdt00_000 Tainted: P --------------- 2.6.32-431.20.3.el6_lustre.x86_64 #1 Red Hat KVM &lt;4&gt;RIP: 0010:[&lt;ffffffffa0fe1f0f&gt;] [&lt;ffffffffa0fe1f0f&gt;] mdd_changelog_data_store+0x1cf/0x390 [mdd] &lt;4&gt;RSP: 0018:ffff88007b3a5b00 EFLAGS: 00010202 &lt;4&gt;RAX: 0000000000005042 RBX: ffff88007bd30000 RCX: 0000000000000000 &lt;4&gt;RDX: 0000000000001042 RSI: 0000000000000046 RDI: ffffffffa0ff457c &lt;4&gt;RBP: ffff88007b3a5b60 R08: 0000000000000000 R09: 0720072007200720 &lt;4&gt;R10: 0720072007200720 R11: 0720072007200720 R12: 000000000000000b &lt;4&gt;R13: ffff88007b3a5c70 R14: ffff88004db29960 R15: 0000000000000000 &lt;4&gt;FS: 0000000000000000(0000) GS:ffff880002280000(0000) knlGS:0000000000000000 &lt;4&gt;CS: 0010 DS: 0018 ES: 0018 CR0: 000000008005003b &lt;4&gt;CR2: 0000000000000048 CR3: 000000007ab7b000 CR4: 00000000000006e0 &lt;4&gt;DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000 &lt;4&gt;DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400 &lt;4&gt;Process mdt00_000 (pid: 10317, threadinfo ffff88007b3a4000, task ffff88001d4a0ae0) &lt;4&gt;Stack: &lt;4&gt; 0000000000000000 ffff880036326990 ffff88007b3a5b50 ffff88004db0d738 &lt;4&gt;&lt;d&gt; ffff88004ab2a5c0 000000424db29960 ffff88007b3a5c70 ffff88004db29960 &lt;4&gt;&lt;d&gt; ffff88007b3a5c70 0000000000000000 ffff88004ab2a5c0 ffff880036326990 &lt;4&gt;Call Trace: &lt;4&gt; [&lt;ffffffffa0fe6afe&gt;] mdd_close+0x34e/0xc50 [mdd] &lt;4&gt; [&lt;ffffffffa10686f1&gt;] mdt_mfd_close+0x3f1/0xac0 [mdt] &lt;4&gt; [&lt;ffffffffa103500e&gt;] ? mdt_ctxt_add_dirty_flag+0x13e/0x190 [mdt] &lt;4&gt; [&lt;ffffffffa10353f2&gt;] mdt_obd_disconnect+0x392/0x510 [mdt] &lt;4&gt; [&lt;ffffffffa08a54f1&gt;] target_handle_disconnect+0x1b1/0x480 [ptlrpc] &lt;4&gt; [&lt;ffffffffa0947ec9&gt;] tgt_disconnect+0x39/0x160 [ptlrpc] &lt;4&gt; [&lt;ffffffffa0948d9e&gt;] tgt_request_handle+0x8be/0x1000 [ptlrpc] &lt;4&gt; [&lt;ffffffffa08f8891&gt;] ptlrpc_main+0xe41/0x1960 [ptlrpc] &lt;4&gt; [&lt;ffffffffa08f7a50&gt;] ? ptlrpc_main+0x0/0x1960 [ptlrpc] &lt;4&gt; [&lt;ffffffff8109abf6&gt;] kthread+0x96/0xa0 &lt;4&gt; [&lt;ffffffff8100c20a&gt;] child_rip+0xa/0x20 &lt;4&gt; [&lt;ffffffff8109ab60&gt;] ? kthread+0x0/0xa0 &lt;4&gt; [&lt;ffffffff8100c200&gt;] ? child_rip+0x0/0x20 &lt;4&gt;Code: bf 31 00 4c 89 fe 48 c7 c7 6f 45 ff a0 31 c0 e8 6c 67 54 e0 8b 45 cc 48 c7 c7 7c 45 ff a0 25 ff 0f 00 00 89 c2 80 cc 50 80 ce 10 &lt;41&gt; 80 7f 48 00 0f 44 c2 89 45 cc 31 c0 e8 43 67 54 e0 8b 45 cc &lt;1&gt;RIP [&lt;ffffffffa0fe1f0f&gt;] mdd_changelog_data_store+0x1cf/0x390 [mdd] &lt;4&gt; RSP &lt;ffff88007b3a5b00&gt; &lt;4&gt;CR2: 0000000000000048 </pre> </div></div> <p>frank zago (fzago@cray.com) uploaded a new patch: <a href="http://review.whamcloud.com/13619" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13619</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6159" title="Set CL_CLOSE in default changelog mask" class="issue-link" data-issue-key="LU-6159"><del>LU-6159</del></a> mdd: fixed oops when dereferencing structure<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 06928d07ddc3281f61cbd7340178417267c0099f</p> <p>Same crash was already reported as <a href="https://jira.whamcloud.com/browse/LU-5938" title="sanity-hsm test_52 MDS OOPS: mdd_changelog_data_store" class="issue-link" data-issue-key="LU-5938"><del>LU-5938</del></a>. So setting CL_CLOSE just made it reproducible, and didn't introduce it. Yeah!</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13526/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13526/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6159" title="Set CL_CLOSE in default changelog mask" class="issue-link" data-issue-key="LU-6159"><del>LU-6159</del></a> hsm: add CL_CLOSE to default changelog mask<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: f27a9e92b2507a18872008f49f9bf0e09009f1cc</p> <p>Landed for 2.8</p> Duplicate LU-5938 Development Rank 1|hzx4vz: Rank (Obsolete) 17225