linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Sterba <dsterba@suse.cz>
To: Nikolay Borisov <nborisov@suse.com>
Cc: Dmitrii Tcvetkov <demfloro@demfloro.ru>,
	linux-btrfs@vger.kernel.org, jbacik@fb.com
Subject: Re: WARN_ON in __writeback_inodes_sb_nr when btrfs mounted with flushoncommit
Date: Thu, 14 Dec 2017 14:38:22 +0100	[thread overview]
Message-ID: <20171214133822.GG3553@twin.jikos.cz> (raw)
In-Reply-To: <ebc37379-1271-1c03-f740-2e88cfb315b2@suse.com>

On Thu, Dec 14, 2017 at 03:21:52PM +0200, Nikolay Borisov wrote:
> 
> 
> On 14.12.2017 13:02, Dmitrii Tcvetkov wrote:
> > Since 4.15-rc1 if btrfs filesystem is mounted with flushoncommit mount option
> > then during fsync this trace appears in dmesg:
> > 
> > [   17.323092] WARNING: CPU: 0 PID: 364 at fs/fs-writeback.c:2339 __writeback_inodes_sb_nr+0xbf/0xd0
> > [   17.323925] Modules linked in:
> > [   17.324697] CPU: 0 PID: 364 Comm: systemd-journal Not tainted 4.15.0-rc3 #2
> > [   17.325424] Hardware name: To be filled by O.E.M. To be filled by O.E.M./SABERTOOTH 990FX R2.0, BIOS 2901 05/04/2016
> > [   17.326177] RIP: 0010:__writeback_inodes_sb_nr+0xbf/0xd0
> > [   17.326875] RSP: 0018:ffff8bcd40a77d08 EFLAGS: 00010246
> > [   17.327598] RAX: 0000000000000000 RBX: ffff8a3fa9764488 RCX: 0000000000000000
> > [   17.328321] RDX: 0000000000000002 RSI: 00000000000018ae RDI: ffff8a3fa96c7070
> > [   17.329012] RBP: ffff8bcd40a77d0c R08: ffffffffffffff80 R09: 00000000000000ff
> > [   17.329740] R10: ffff8bcd40a77c10 R11: 0000000000001000 R12: 0000000000000000
> > [   17.330439] R13: ffff8a3fa915e698 R14: ffff8a3fb04ed780 R15: ffff8a3fa9a16610
> > [   17.331169] FS:  00007f72d53338c0(0000) GS:ffff8a3fbec00000(0000) knlGS:0000000000000000
> > [   17.331880] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> > [   17.332624] CR2: 00007f72d09a5000 CR3: 0000000329334000 CR4: 00000000000406f0
> > [   17.333383] Call Trace:
> > [   17.334113]  btrfs_commit_transaction+0x857/0x920
> > [   17.334874]  btrfs_sync_file+0x30c/0x3e0
> > [   17.335622]  do_fsync+0x33/0x60
> > [   17.336332]  SyS_fsync+0x7/0x10
> > [   17.337069]  do_syscall_64+0x63/0x360
> > [   17.337776]  entry_SYSCALL64_slow_path+0x25/0x25
> > [   17.338513] RIP: 0033:0x7f72d4f29094
> > [   17.339244] RSP: 002b:00007ffd71b078f8 EFLAGS: 00000246 ORIG_RAX: 000000000000004a
> > [   17.339962] RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00007f72d4f29094
> > [   17.340718] RDX: 0000000000090000 RSI: 00005630b6f8b090 RDI: 0000000000000010
> > [   17.341431] RBP: 00005630b6f8b090 R08: 00000000000fffff R09: 0000000000000000
> > [   17.342169] R10: 0000000000000000 R11: 0000000000000246 R12: 0000000000000010
> > [   17.342902] R13: 00005630b6f88f60 R14: 0000000000000001 R15: 0000000000000001
> > [   17.343604] Code: df 0f b6 d1 e8 a3 fc ff ff 48 89 ee 48 89 df e8 78 f5 ff
> > ff 48 8b 44 24 48 65 48 33 04 25 28 00 00 00 75 0b 48 83 c4 50 5b 5d c3 <0f> ff
> > eb ca e8 38 1e ec ff 0f 1f 84 00 00 00 00 00 41 54 55 48
> > [   17.344408] ---[ end trace ff4cf41ec70ec0a7 ]---
> 
> So this is due to writeback_inodes_sb being called without holding
> s_umount. So 4.15-rc1 the first kernel that started exhibiting this or
> did you also see it with earlier kernel

This is new in 4.15, there was an earlier report
https://marc.info/?l=linux-btrfs&m=151315564008773

  reply	other threads:[~2017-12-14 13:40 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-14 11:02 WARN_ON in __writeback_inodes_sb_nr when btrfs mounted with flushoncommit Dmitrii Tcvetkov
2017-12-14 13:21 ` Nikolay Borisov
2017-12-14 13:38   ` David Sterba [this message]
2017-12-14 15:05   ` Dmitrii Tcvetkov
  -- strict thread matches above, loose matches on Subject: below --
2022-10-23 23:17 Roman Mamedov
2022-10-24  7:37 ` Roman Mamedov
2022-10-24 14:16   ` David Sterba
2022-10-24 21:23     ` Roman Mamedov
2022-11-04 13:02       ` David Sterba

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171214133822.GG3553@twin.jikos.cz \
    --to=dsterba@suse.cz \
    --cc=demfloro@demfloro.ru \
    --cc=jbacik@fb.com \
    --cc=linux-btrfs@vger.kernel.org \
    --cc=nborisov@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).