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-4000 Lustre <p>lloop fails to build on ppc64 with 64k pages due to the block layer API that limits the logical block size to sizes representable with an unsigned short.</p> <p>The logical block size shouldn't be set to the page size since that will force any file systems on that loop device to also require a 64k page size. </p> LU-4000

Fix build failure on ppc64 w/ 64k pages

Bug Major Closed Won't Do James A Simmons Jeff Mahoney patch Tue, 24 Sep 2013 16:06:25 +0000 Tue, 10 Jan 2017 17:16:53 +0000 Tue, 10 Jan 2017 17:16:53 +0000 Lustre 2.10.0 0 5 <p>Fix here: <a href="http://review.whamcloud.com/7745" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/7745</a></p> <p>Thanks Jeff!</p> <p>Minh</p> <p>Could you please take care of this patch?</p> <p>Thanks</p> <p>Peter</p> <p>Patch landed to Master.</p> <p>This patch broke loop device as follows:</p> <blockquote> <p>-----------<del>[ cut here ]</del>-----------<br/> kernel BUG at /root/lustre/lustre/llite/lloop.c:226!<br/> invalid opcode: 0000 <a href="#1" target="_blank" rel="noopener">1</a> SMP <br/> last sysfs file: /sys/devices/virtual/block/lloop0/dev<br/> CPU 4 <br/> Modules linked in: llite_lloop(U) lustre(U) ofd(U) osp(U) lod(U) ost(U) mdt(U) osd_ldiskfs(U) fsfilt_ldiskfs(U) ldiskfs(U) mdd(U) mgs(U) lquota(U) lfsck(U) obdecho(U) mgc(U) lov(U) osc(U) mdc(U) lmv(U) fid(U) fld(U) ptlrpc(U) obdclass(U) lvfs(U) ksocklnd(U) lnet(U) libcfs(U) jbd2 sha512_generic sha256_generic crc32c_intel nfsd exportfs nfs lockd fscache auth_rpcgss nfs_acl sunrpc cpufreq_ondemand acpi_cpufreq freq_table mperf ib_ipoib rdma_ucm ib_ucm ib_uverbs ib_umad rdma_cm ib_cm iw_cm ib_addr ipv6 mlx4_ib ib_sa ib_mad ib_core mlx4_en mlx4_core e1000e microcode serio_raw i2c_i801 i2c_core sg iTCO_wdt iTCO_vendor_support ioatdma dca i7core_edac edac_core shpchp ext3 jbd mbcache sd_mod crc_t10dif pata_acpi ata_generic ata_piix dm_mirror dm_region_hash dm_log dm_mod <span class="error">&#91;last unloaded: libcfs&#93;</span></p> <p>Pid: 20050, comm: lloop0 Not tainted 2.6.32-358.18.1.el6_lustre.ga0a1066.x86_64 #1 Supermicro X8DTT-H/X8DTT-H<br/> RIP: 0010:<span class="error">&#91;&lt;ffffffffa02ad4ca&gt;&#93;</span> <span class="error">&#91;&lt;ffffffffa02ad4ca&gt;&#93;</span> loop_thread+0x71a/0x860 <span class="error">&#91;llite_lloop&#93;</span><br/> RSP: 0018:ffff8807d282fdf0 EFLAGS: 00010206<br/> RAX: 0000000000000000 RBX: ffff8807b8c20000 RCX: 0000000000000000<br/> RDX: ffff880638efd370 RSI: 0000000000000000 RDI: 0000000000000002<br/> RBP: ffff8807d282fee0 R08: 0000000000000400 R09: ffff8807c1a0a138<br/> R10: ffff880638efd300 R11: 0000000000000000 R12: ffff880638efd300<br/> R13: ffff8807b8c201f0 R14: ffff8807b8c209f0 R15: 0000000000000001<br/> FS: 0000000000000000(0000) GS:ffff880045680000(0000) knlGS:0000000000000000<br/> CS: 0010 DS: 0018 ES: 0018 CR0: 000000008005003b<br/> CR2: 00000034b161d9d0 CR3: 000000082f466000 CR4: 00000000000007e0<br/> DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000<br/> DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400<br/> Process lloop0 (pid: 20050, threadinfo ffff8807d282e000, task ffff88082ee60080)<br/> Stack:<br/> 0000000000000000 ffff8807d282fe80 ffff8805fc19be18 ffff8807c1a0a138<br/> &lt;d&gt; ffff88082ee60080 ffff8807d282fe98 ffff8807b8c20098 ffff8807b8c20048<br/> &lt;d&gt; ffff8807b8c201c8 00000000b7ec5c60 ffff8807b8c20060 ffff8807b8c200a0<br/> Call Trace:<br/> <span class="error">&#91;&lt;ffffffff81096da0&gt;&#93;</span> ? autoremove_wake_function+0x0/0x40<br/> <span class="error">&#91;&lt;ffffffffa02acdb0&gt;&#93;</span> ? loop_thread+0x0/0x860 <span class="error">&#91;llite_lloop&#93;</span><br/> <span class="error">&#91;&lt;ffffffff81096a36&gt;&#93;</span> kthread+0x96/0xa0<br/> <span class="error">&#91;&lt;ffffffff8100c0ca&gt;&#93;</span> child_rip+0xa/0x20<br/> <span class="error">&#91;&lt;ffffffff810969a0&gt;&#93;</span> ? kthread+0x0/0xa0<br/> <span class="error">&#91;&lt;ffffffff8100c0c0&gt;&#93;</span> ? child_rip+0x0/0x20<br/> Code: 60 e9 2a a0 48 c7 05 d2 14 00 00 90 e9 2a a0 c7 05 c0 14 00 00 00 04 00 00 31 c0 8b 13 e8 af 22 94 00 e9 02 fa ff ff 0f 0b eb fe &lt;0f&gt; 0b 0f 1f 40 00 eb fa 48 c7 c7 00 e3 2a a0 48 c7 c2 ba da 2a <br/> RIP <span class="error">&#91;&lt;ffffffffa02ad4ca&gt;&#93;</span> loop_thread+0x71a/0x860 <span class="error">&#91;llite_lloop&#93;</span><br/> RSP &lt;ffff8807d282fdf0&gt;</p></blockquote> <p>We have to revert this patch or make a new fix.</p> <p>Ah, ok. It looks like my fix was incomplete. Those assertions essentially back up the ones made by the build assertion my patch removed. My initial analysis of where the logical block size is used missed the directio case. I expect that's where these split bios are coming from.</p> <p>So, those two BUG_ONs need to be removed to avoid the Oops. The original fix should probably be changed to use min(PAGE_SIZE, 32768) to keep the original lloop performance, at least until the size of the logical_block_size queue limit variable is increased (if that happens).</p> <p>Updated but untested fix here: <a href="http://review.whamcloud.com/8096" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/8096</a></p> <p>The llite_lloop back device is no longer supported so this can be closed.</p> Related LU-6585 Development Rank 1|hzw3sv: Rank (Obsolete) 10710 Severity