<div dir="ltr"><div>Adding dm-devel since it involves LVM.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail-m_657265225312649470gmail_attr">On Thu, Jan 24, 2019 at 1:14 PM Zhang Xiaoxu <<a href="mailto:zhangxiaoxu5@huawei.com" target="_blank">zhangxiaoxu5@huawei.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If the lvm is stacked by different logical_block_size disks,<br>
when WRITE SAME on it, will bug_on:<br>
<br>
kernel BUG at drivers/scsi/sd.c:968!<br>
invalid opcode: 0000 [#1] SMP PTI<br>
CPU: 11 PID: 525 Comm: kworker/11:1H Tainted: G           O      5.0.0-rc3+ #2<br>
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.10.2-2.fc27 04/01/2014<br>
Workqueue: kblockd blk_mq_run_work_fn<br>
RIP: 0010:sd_init_command+0x7aa/0xdb0<br>
Code: 30 75 00 00 c7 85 44 01 00 00 18 00 00 00 0f 85 fa 04 00 00 48 83 c4 40 48<br>
      89 df 5b 5d 41 5c 41 5d 41 5e 41 5f e9 b6 ca fe ff <0f> 0b 41 bc 09<br>
RSP: 0018:ffffb55f80ddbca0 EFLAGS: 00010206<br>
RAX: 0000000000001000 RBX: ffff9ed23fb927a0 RCX: 000000000000f000<br>
RDX: ffff9ed23f0a8400 RSI: ffff9ed27bc79800 RDI: 0000000000000000<br>
RBP: ffff9ed23fb92680 R08: ffff9ed27c8c0000 R09: ffff9ed23fb927d8<br>
R10: 0000000000000000 R11: fefefefefefefeff R12: ffff9ed27bc79800<br>
R13: ffff9ed2787a0000 R14: ffff9ed27bdf3400 R15: ffff9ed23fb927a0<br>
FS:  0000000000000000(0000) GS:ffff9ed27c8c0000(0000) knlGS:0000000000000000<br>
CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033<br>
CR2: 00007f6b14cf9341 CR3: 0000000069058000 CR4: 00000000000006e0<br>
Call Trace:<br>
 ? vp_notify+0x12/0x20<br>
 scsi_queue_rq+0x525/0xa30<br>
 blk_mq_dispatch_rq_list+0x8d/0x580<br>
 ? syscall_return_via_sysret+0x10/0x7f<br>
 ? elv_rb_del+0x1f/0x30<br>
 ? deadline_remove_request+0x55/0xc0<br>
 blk_mq_do_dispatch_sched+0x76/0x110<br>
 blk_mq_sched_dispatch_requests+0xf9/0x170<br>
 __blk_mq_run_hw_queue+0x51/0xd0<br>
 process_one_work+0x195/0x380<br>
 worker_thread+0x30/0x390<br>
 ? process_one_work+0x380/0x380<br>
 kthread+0x113/0x130<br>
 ? kthread_park+0x90/0x90<br>
 ret_from_fork+0x35/0x40<br>
Modules linked in: alloc(O+)<br>
---[ end trace dc92ddeb2e6d1fe5 ]---<br>
<br>
The logical_block_size of the LVM is the max value of the sub disks,<br>
it maybe different with one of the sub disk. when WRITE SAME on the<br>
disk, it will BUG_ON when setup WRITE SAME cmd.<br>
<br>
Close WRITE_SAME feature on the LVM if it was stacked by different<br>
logical_block_size disk.<br>
<br>
Signed-off-by: Zhang Xiaoxu <<a href="mailto:zhangxiaoxu5@huawei.com" target="_blank">zhangxiaoxu5@huawei.com</a>><br>
---<br>
 block/blk-settings.c | 4 ++++<br>
 1 file changed, 4 insertions(+)<br>
<br>
diff --git a/block/blk-settings.c b/block/blk-settings.c<br>
index 3e7038e475ee..e1695155ad61 100644<br>
--- a/block/blk-settings.c<br>
+++ b/block/blk-settings.c<br>
@@ -537,6 +537,10 @@ int blk_stack_limits(struct queue_limits *t, struct queue_limits *b,<br>
                }<br>
        }<br>
<br>
+       /* If the logical block size is different, forbid write same */<br>
+       if (t->logical_block_size != b->logical_block_size)<br>
+               t->max_write_same_sectors = 0;<br>
+<br>
        t->logical_block_size = max(t->logical_block_size,<br>
                                    b->logical_block_size);<br>
<br>
-- <br>
2.14.4<br>
<br>
</blockquote></div>