Dave Kleikamp
de956d4078
nfs: don't dirty kernel pages read by direct-io
[ Upstream commit ad3cba223a ]
When we use direct_IO with an NFS backing store, we can trigger a
WARNING in __set_page_dirty(), as below, since we're dirtying the page
unnecessarily in nfs_direct_read_completion().
To fix, replicate the logic in commit 53cbf3b157 ("fs: direct-io:
don't dirtying pages for ITER_BVEC/ITER_KVEC direct read").
Other filesystems that implement direct_IO handle this; most use
blockdev_direct_IO(). ceph and cifs have similar logic.
mount 127.0.0.1:/export /nfs
dd if=/dev/zero of=/nfs/image bs=1M count=200
losetup --direct-io=on -f /nfs/image
mkfs.btrfs /dev/loop0
mount -t btrfs /dev/loop0 /mnt/
kernel: WARNING: CPU: 0 PID: 8067 at fs/buffer.c:580 __set_page_dirty+0xaf/0xd0
kernel: Modules linked in: loop(E) nfsv3(E) rpcsec_gss_krb5(E) nfsv4(E) dns_resolver(E) nfs(E) fscache(E) nfsd(E) auth_rpcgss(E) nfs_acl(E) lockd(E) grace(E) fuse(E) tun(E) ip6t_rpfilter(E) ipt_REJECT(E) nf_
kernel: snd_seq(E) snd_seq_device(E) snd_pcm(E) video(E) snd_timer(E) snd(E) soundcore(E) ip_tables(E) xfs(E) libcrc32c(E) sd_mod(E) sr_mod(E) cdrom(E) ata_generic(E) pata_acpi(E) crc32c_intel(E) ahci(E) li
kernel: CPU: 0 PID: 8067 Comm: kworker/0:2 Tainted: G E 4.20.0-rc1.master.20181111.ol7.x86_64 #1
kernel: Hardware name: innotek GmbH VirtualBox/VirtualBox, BIOS VirtualBox 12/01/2006
kernel: Workqueue: nfsiod rpc_async_release [sunrpc]
kernel: RIP: 0010:__set_page_dirty+0xaf/0xd0
kernel: Code: c3 48 8b 02 f6 c4 04 74 d4 48 89 df e8 ba 05 f7 ff 48 89 c6 eb cb 48 8b 43 08 a8 01 75 1f 48 89 d8 48 8b 00 a8 04 74 02 eb 87 <0f> 0b eb 83 48 83 e8 01 eb 9f 48 83 ea 01 0f 1f 00 eb 8b 48 83 e8
kernel: RSP: 0000:ffffc1c8825b7d78 EFLAGS: 00013046
kernel: RAX: 000fffffc0020089 RBX: fffff2b603308b80 RCX: 0000000000000001
kernel: RDX: 0000000000000001 RSI: ffff9d11478115c8 RDI: ffff9d11478115d0
kernel: RBP: ffffc1c8825b7da0 R08: 0000646f6973666e R09: 8080808080808080
kernel: R10: 0000000000000001 R11: 0000000000000000 R12: ffff9d11478115d0
kernel: R13: ffff9d11478115c8 R14: 0000000000003246 R15: 0000000000000001
kernel: FS: 0000000000000000(0000) GS:ffff9d115ba00000(0000) knlGS:0000000000000000
kernel: CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
kernel: CR2: 00007f408686f640 CR3: 0000000104d8e004 CR4: 00000000000606f0
kernel: DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
kernel: DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
kernel: Call Trace:
kernel: __set_page_dirty_buffers+0xb6/0x110
kernel: set_page_dirty+0x52/0xb0
kernel: nfs_direct_read_completion+0xc4/0x120 [nfs]
kernel: nfs_pgio_release+0x10/0x20 [nfs]
kernel: rpc_free_task+0x30/0x70 [sunrpc]
kernel: rpc_async_release+0x12/0x20 [sunrpc]
kernel: process_one_work+0x174/0x390
kernel: worker_thread+0x4f/0x3e0
kernel: kthread+0x102/0x140
kernel: ? drain_workqueue+0x130/0x130
kernel: ? kthread_stop+0x110/0x110
kernel: ret_from_fork+0x35/0x40
kernel: ---[ end trace 01341980905412c9 ]---
Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>
Signed-off-by: Santosh Shilimkar <santosh.shilimkar@oracle.com>
[forward-ported to v4.20]
Signed-off-by: Calum Mackay <calum.mackay@oracle.com>
Reviewed-by: Dave Kleikamp <dave.kleikamp@oracle.com>
Reviewed-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
2018-12-21 14:15:17 +01:00
..
2018-12-01 09:37:27 +01:00
2018-08-22 10:52:51 -07:00
2018-12-17 09:24:40 +01:00
2018-08-22 12:34:08 -07:00
2018-12-01 09:37:27 +01:00
2018-12-17 09:24:34 +01:00
2018-12-17 09:24:40 +01:00
2018-11-27 16:13:05 +01:00
2018-12-13 09:16:20 +01:00
2018-07-17 06:14:07 -07:00
2018-11-13 11:08:55 -08:00
2018-11-13 11:08:46 -08:00
2018-08-21 23:54:17 -04:00
2018-07-22 14:13:44 +02:00
2018-11-27 16:13:00 +01:00
2018-12-17 09:24:35 +01:00
2018-12-05 19:32:11 +01:00
2018-11-21 09:19:22 +01:00
2018-11-13 11:08:39 -08:00
2018-10-13 09:31:03 +02:00
2018-12-17 09:24:40 +01:00
2018-12-19 19:19:51 +01:00
2018-12-01 09:37:34 +01:00
2018-12-17 09:24:41 +01:00
2018-12-17 09:24:41 +01:00
2018-08-17 16:20:28 -07:00
2018-08-25 12:42:33 -07:00
2018-08-22 10:52:44 -07:00
2018-08-21 11:37:41 +02:00
2018-11-13 11:08:43 -08:00
2018-11-13 11:08:16 -08:00
2018-08-15 22:47:23 -07:00
2018-08-18 11:44:53 -07:00
2018-11-13 11:08:49 -08:00
2018-12-21 14:15:17 +01:00
2018-11-21 09:19:23 +01:00
2018-09-04 16:45:02 -07:00
2018-12-01 09:37:31 +01:00
2018-08-17 16:20:27 -07:00
2018-12-17 09:24:41 +01:00
2018-08-14 12:07:14 -04:00
2018-12-19 19:19:51 +01:00
2018-11-13 11:08:46 -08:00
2018-12-17 09:24:39 +01:00
2018-08-22 18:17:48 +02:00
2018-11-27 16:12:59 +01:00
2018-08-02 09:34:02 -07:00
2018-08-18 11:44:53 -07:00
2018-12-17 09:24:30 +01:00
2018-07-31 11:32:44 -04:00
2018-10-13 11:05:02 +02:00
2018-12-05 19:32:06 +01:00
2018-08-17 16:20:27 -07:00
2018-10-06 11:44:39 +10:00
2018-12-19 19:19:50 +01:00
2018-07-12 10:04:27 -04:00
2018-07-03 16:44:45 -04:00
2018-07-12 10:04:20 -04:00
2018-08-13 19:24:32 -07:00
2018-07-10 23:29:03 -04:00
2018-08-14 10:23:25 -07:00
2018-08-30 12:56:40 +02:00
2018-07-30 16:21:49 -04:00
2018-12-17 09:24:41 +01:00
2018-08-17 16:20:28 -07:00
2018-12-05 19:32:08 +01:00
2018-06-28 10:40:47 -07:00
2018-08-22 10:52:49 -07:00
2018-08-21 13:47:29 -07:00
2018-08-15 23:03:20 -05:00
2018-08-21 18:19:09 -07:00
2018-11-21 09:19:25 +01:00
2018-08-21 18:19:09 -07:00
2018-11-21 09:19:14 +01:00
2018-12-19 19:19:50 +01:00
2018-08-02 08:06:55 +09:00
2018-08-21 18:19:09 -07:00
2018-08-17 16:20:29 -07:00
2018-08-23 18:48:43 -07:00
2018-11-21 09:19:22 +01:00
2018-08-21 18:19:09 -07:00
2018-08-13 19:58:36 -07:00
2018-09-24 10:54:01 +02:00
2018-06-28 10:40:47 -07:00
2018-08-17 16:20:28 -07:00
2018-07-12 14:49:48 +01:00
2018-08-26 11:48:42 -07:00
2018-08-13 20:56:23 -07:00
2018-12-19 19:19:50 +01:00
2018-09-18 07:30:48 -04:00