linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* parity scrub on 32-bit
@ 2017-04-10  8:53 Adam Borowski
  2017-04-10 12:44 ` Austin S. Hemmelgarn
  0 siblings, 1 reply; 2+ messages in thread
From: Adam Borowski @ 2017-04-10  8:53 UTC (permalink / raw)
  To: linux-btrfs

[-- Attachment #1: Type: text/plain, Size: 3412 bytes --]

Hi!
While messing with the division failure on current -next, I've noticed that
parity scrub splats immediately on all 32-bit archs I tried.  But, it's not
a regression: it bisects to 5a6ac9eacb49143cbad3bbfda72263101cb1f3df (merged
in 3.19) which happens to be when parity scrub was added.  Ie, it never
worked in the first place.

But this doesn't sound right to me -- while no one gives a damn about i386
(good riddance!), ARM32 NASes are quite popular.  Surely someone would have
noticed -- it fails not only when there's damage to repair but even when
everything is clean.

Am I missing something?

Test script attached (overkill, it dies on first scrub before I get to
damage it).

Trace from the earliest commit, i386_defconfig+btrfs:
[   83.254499] ------------[ cut here ]------------
[   83.255009] kernel BUG at mm/highmem.c:353!
[   83.255009] invalid opcode: 0000 [#1] SMP 
[   83.255009] Modules linked in:
[   83.255009] CPU: 0 PID: 3017 Comm: kworker/u4:7 Not tainted 3.18.0-rc6-defconfig+ #1
[   83.255009] Hardware name: Hewlett-Packard HP Compaq dc7100 SFF(DX878AV)/097Ch, BIOS 786C1 v01.05 06/16/2004
[   83.255009] Workqueue: btrfs-endio-raid56 btrfs_endio_raid56_helper
[   83.255009] task: f646f580 ti: f5ee0000 task.ti: f5ee0000
[   83.255009] EIP: 0060:[<c110dab0>] EFLAGS: 00010246 CPU: 0
[   83.255009] EIP is at kunmap_high+0x90/0xa0
[   83.255009] EAX: 000000ca EBX: 00000001 ECX: fffff000 EDX: 00000000
[   83.255009] ESI: 00000004 EDI: f65e6680 EBP: f5ee1e34 ESP: f5ee1e30
[   83.255009]  DS: 007b ES: 007b FS: 00d8 GS: 0000 SS: 0068
[   83.255009] CR0: 8005003b CR2: b555cfdc CR3: 20476000 CR4: 000007d0
[   83.255009] Stack:
[   83.255009]  f5820800 f5ee1e3c c103de19 f5ee1ea0 c12d1546 00000000 e2729000 de4f1000
[   83.255009]  e2941000 ff8c9000 e0fd3cac f5ee1e4c 00000002 f5ee1e5c f5820800 00000000
[   83.255009]  f7400300 ffffffff f7409240 f65e6680 f5ee1e48 00000003 0000000d 00000000
[   83.255009] Call Trace:
[   83.255009]  [<c103de19>] kunmap+0x49/0x50
[   83.255009]  [<c12d1546>] finish_parity_scrub+0x216/0x440
[   83.255009]  [<c12d280a>] validate_rbio_for_parity_scrub+0xda/0xe0
[   83.255009]  [<c12d2867>] raid56_parity_scrub_end_io+0x57/0x70
[   83.255009]  [<c1313ab1>] bio_endio+0x41/0x90
[   83.255009]  [<c1258064>] ? end_workqueue_fn+0x24/0x40
[   83.255009]  [<c1313b0c>] bio_endio_nodec+0xc/0x10
[   83.255009]  [<c125806d>] end_workqueue_fn+0x2d/0x40
[   83.255009]  [<c1290aba>] btrfs_scrubnc_helper+0xca/0x250
[   83.255009]  [<c1290ce8>] btrfs_endio_raid56_helper+0x8/0x10
[   83.255009]  [<c105574d>] process_one_work+0x1ad/0x3f0
[   83.255009]  [<c1055b8a>] worker_thread+0x1fa/0x490
[   83.255009]  [<c1055990>] ? process_one_work+0x3f0/0x3f0
[   83.255009]  [<c1059b16>] kthread+0x96/0xb0
[   83.255009]  [<c17f55c1>] ret_from_kernel_thread+0x21/0x30
[   83.255009]  [<c1059a80>] ? kthread_worker_fn+0x120/0x120
[   83.255009] Code: ba 03 00 00 00 e8 a1 53 f6 ff 58 8b 5d fc c9 c3 8d 76 00 31 c0 81 3d 24 7b aa c1 24 7b aa c1 0f 95 c0 eb c5 8d b4 26 00 00 00 00 <0f> 0b 8d b6 00 00 00 00 0f 0b 8d b6 00 00 00 00 55 89 e5 56 53
[   83.255009] EIP: [<c110dab0>] kunmap_high+0x90/0xa0 SS:ESP 0068:f5ee1e30
[   83.497432] ---[ end trace bf2c0560a0dc9e51 ]---

-- 
⢀⣴⠾⠻⢶⣦⠀ Meow!
⣾⠁⢠⠒⠀⣿⡁
⢿⡄⠘⠷⠚⠋⠀ Collisions shmolisions, let's see them find a collision or second
⠈⠳⣄⠀⠀⠀⠀ preimage for double rot13!

[-- Attachment #2: scrubtest --]
[-- Type: text/plain, Size: 1299 bytes --]

#!/bin/sh
set -x
DATA=/usr/bin  # use whole /usr on non-bloated VMs


mkdir -p /mnt/vol1
umount /mnt/vol1; losetup -D   # clean up after repeats

dd if=/dev/zero bs=1048576 count=1 seek=4095 of=ra
dd if=/dev/zero bs=1048576 count=1 seek=4095 of=rb
dd if=/dev/zero bs=1048576 count=1 seek=4095 of=rc
dd if=/dev/zero bs=1048576 count=1 seek=4095 of=rd

mkfs.btrfs -draid10 -mraid1 ra rb rc rd

losetup -D
losetup -f ra
losetup -f rb
losetup -f rc
losetup -f rd
sleep 2  # race with fsid detection
mount -onoatime /dev/loop0 /mnt/vol1 || exit $?
cp -pr "$DATA" /mnt/vol1
btrfs fi sync /mnt/vol1
btrfs fi us /mnt/vol1

btrfs balance start -dconvert=raid5 -mconvert=raid6 /mnt/vol1

btrfs scrub start -B /mnt/vol1
btrfs scrub start -B /mnt/vol1

umount /mnt/vol1
dd if=/dev/urandom of=rd bs=1048576 seek=96 count=4000
mount -onoatime /dev/loop0 /mnt/vol1 || exit $?
btrfs scrub start -B /mnt/vol1
btrfs scrub start -B /mnt/vol1

btrfs balance start -dconvert=raid10 -mconvert=raid1 /mnt/vol1

btrfs scrub start -B /mnt/vol1
btrfs scrub start -B /mnt/vol1

umount /mnt/vol1
dd if=/dev/urandom of=rd bs=1048576 seek=96 count=4000
mount -onoatime /dev/loop0 /mnt/vol1 || exit $?
btrfs scrub start -B /mnt/vol1
btrfs scrub start -B /mnt/vol1

diff -urd --no-dereference "$DATA" /mnt/vol1/*

umount /mnt/vol1

^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-04-10 12:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-04-10  8:53 parity scrub on 32-bit Adam Borowski
2017-04-10 12:44 ` Austin S. Hemmelgarn

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).