Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1320042
| From | Ross Zwisler <ross.zwisler@linux.intel.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: mm: WARNING in __delete_from_page_cache |
| Date | 2016-01-27 23:10 +0100 |
| Message-ID | <qVGrN-1Vy-45@gated-at.bofh.it> (permalink) |
| References | (2 earlier) <qUOrn-3Og-1@gated-at.bofh.it> <qV2NH-5Hq-1@gated-at.bofh.it> <qVbo0-4eF-47@gated-at.bofh.it> <qVc0G-4LK-19@gated-at.bofh.it> <qVf8g-6St-41@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Tue, Jan 26, 2016 at 04:59:27PM +0000, Williams, Dan J wrote:
> On Tue, 2016-01-26 at 14:36 +0100, Jan Kara wrote:
> > On Tue 26-01-16 07:54:56, Matthew Wilcox wrote:
> > > On Tue, Jan 26, 2016 at 03:42:34AM +0000, Williams, Dan J wrote:
> > > > @@ -2907,7 +2912,12 @@ extern void replace_mount_options(struct
> > > > super_block *sb, char *options);
> > > >
> > > > static inline bool io_is_direct(struct file *filp)
> > > > {
> > > > - return (filp->f_flags & O_DIRECT) ||
> > > > IS_DAX(file_inode(filp));
> > >
> > > I think this should just be a one-liner:
> > >
> > > - return (filp->f_flags & O_DIRECT) ||
> > > IS_DAX(file_inode(filp));
> > > + return (filp->f_flags & O_DIRECT) || IS_DAX(filp-
> > > >f_mapping->host);
> > >
> > > This does the right thing for block device inodes and filesystem
> > > inodes.
> > > (see the opening stanzas of __dax_fault for an example).
> >
> > Ah, right. This looks indeed better.
> >
>
>
> Oh, yeah, looks good.
>
> 8<---- (git am --scissors)
> Subject: fs, block: force direct-I/O for dax-enabled block devices
>
> From: Dan Williams <dan.j.williams@intel.com>
>
> Similar to the file I/O path, re-direct all I/O to the DAX path for I/O
> to a block-device special file. Both regular files and device special
> files can use the common filp->f_mapping->host lookup to determing is
> DAX is enabled.
>
> Otherwise, we confuse the DAX code that does not expect to find live
> data in the page cache:
>
> ------------[ cut here ]------------
> WARNING: CPU: 0 PID: 7676 at mm/filemap.c:217
> __delete_from_page_cache+0x9f6/0xb60()
> Modules linked in:
> CPU: 0 PID: 7676 Comm: a.out Not tainted 4.4.0+ #276
> Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011
> 00000000ffffffff ffff88006d3f7738 ffffffff82999e2d 0000000000000000
> ffff8800620a0000 ffffffff86473d20 ffff88006d3f7778 ffffffff81352089
> ffffffff81658d36 ffffffff86473d20 00000000000000d9 ffffea0000009d60
> Call Trace:
> [< inline >] __dump_stack lib/dump_stack.c:15
> [<ffffffff82999e2d>] dump_stack+0x6f/0xa2 lib/dump_stack.c:50
> [<ffffffff81352089>] warn_slowpath_common+0xd9/0x140 kernel/panic.c:482
> [<ffffffff813522b9>] warn_slowpath_null+0x29/0x30 kernel/panic.c:515
> [<ffffffff81658d36>] __delete_from_page_cache+0x9f6/0xb60 mm/filemap.c:217
> [<ffffffff81658fb2>] delete_from_page_cache+0x112/0x200 mm/filemap.c:244
> [<ffffffff818af369>] __dax_fault+0x859/0x1800 fs/dax.c:487
> [<ffffffff8186f4f6>] blkdev_dax_fault+0x26/0x30 fs/block_dev.c:1730
> [< inline >] wp_pfn_shared mm/memory.c:2208
> [<ffffffff816e9145>] do_wp_page+0xc85/0x14f0 mm/memory.c:2307
> [< inline >] handle_pte_fault mm/memory.c:3323
> [< inline >] __handle_mm_fault mm/memory.c:3417
> [<ffffffff816ecec3>] handle_mm_fault+0x2483/0x4640 mm/memory.c:3446
> [<ffffffff8127eff6>] __do_page_fault+0x376/0x960 arch/x86/mm/fault.c:1238
> [<ffffffff8127f738>] trace_do_page_fault+0xe8/0x420 arch/x86/mm/fault.c:1331
> [<ffffffff812705c4>] do_async_page_fault+0x14/0xd0 arch/x86/kernel/kvm.c:264
> [<ffffffff86338f78>] async_page_fault+0x28/0x30 arch/x86/entry/entry_64.S:986
> [<ffffffff86336c36>] entry_SYSCALL_64_fastpath+0x16/0x7a
> arch/x86/entry/entry_64.S:185
> ---[ end trace dae21e0f85f1f98c ]---
>
> Cc: Ross Zwisler <ross.zwisler@linux.intel.com>
> Fixes: 5a023cdba50c ("block: enable dax for raw block devices")
> Reported-by: Dmitry Vyukov <dvyukov@google.com>
> Reported-by: Kirill A. Shutemov <kirill@shutemov.name>
> Suggested-by: Jan Kara <jack@suse.cz>
> Reviewed-by: Jan Kara <jack@suse.cz>
> Suggested-by: Matthew Wilcox <willy@linux.intel.com>
> Signed-off-by: Dan Williams <dan.j.williams@intel.com>
I had a test case where I was hitting a warning while inserting into the page
cache when the inode was supposed to be DAX, and this clears up my issue as
well.
Tested-by: Ross Zwisler <ross.zwisler@linux.intel.com>
> ---
> include/linux/fs.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/fs.h b/include/linux/fs.h
> index 1a2046275cdf..b10002d4a5f5 100644
> --- a/include/linux/fs.h
> +++ b/include/linux/fs.h
> @@ -2907,7 +2907,7 @@ extern void replace_mount_options(struct super_block *sb, char *options);
>
> static inline bool io_is_direct(struct file *filp)
> {
> - return (filp->f_flags & O_DIRECT) || IS_DAX(file_inode(filp));
> + return (filp->f_flags & O_DIRECT) || IS_DAX(filp->f_mapping->host);
> }
>
> static inline int iocb_flags(struct file *file)
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
mm: WARNING in __delete_from_page_cache Dmitry Vyukov <dvyukov@google.com> - 2016-01-24 11:50 +0100
Re: mm: WARNING in __delete_from_page_cache "Kirill A. Shutemov" <kirill@shutemov.name> - 2016-01-25 00:10 +0100
Re: mm: WARNING in __delete_from_page_cache Jan Kara <jack@suse.cz> - 2016-01-25 13:30 +0100
Re: mm: WARNING in __delete_from_page_cache "Williams, Dan J" <dan.j.williams@intel.com> - 2016-01-26 04:50 +0100
Re: mm: WARNING in __delete_from_page_cache Jan Kara <jack@suse.cz> - 2016-01-26 09:30 +0100
Re: mm: WARNING in __delete_from_page_cache Matthew Wilcox <willy@linux.intel.com> - 2016-01-26 14:00 +0100
Re: mm: WARNING in __delete_from_page_cache Jan Kara <jack@suse.cz> - 2016-01-26 14:40 +0100
Re: mm: WARNING in __delete_from_page_cache "Williams, Dan J" <dan.j.williams@intel.com> - 2016-01-26 18:00 +0100
Re: mm: WARNING in __delete_from_page_cache Ross Zwisler <ross.zwisler@linux.intel.com> - 2016-01-27 23:10 +0100
Re: mm: WARNING in __delete_from_page_cache Ross Zwisler <ross.zwisler@linux.intel.com> - 2016-01-28 21:00 +0100
Re: mm: WARNING in __delete_from_page_cache Ross Zwisler <zwisler@gmail.com> - 2016-01-27 19:10 +0100
Re: mm: WARNING in __delete_from_page_cache Dmitry Vyukov <dvyukov@google.com> - 2016-01-27 19:10 +0100
csiph-web