Hi Alan,
On Tue, Nov 8, 2011 at 11:07 PM, Alan Cook <acook@xxxxxxxxxxxxxxxxxxxxxx> wrote:
> I am having issues using the O_DIRECT flags for writing to files that reside on
> the realtime subvolume of an XFS file system. I have found a failure case for
> xfstests test 090 as well as another case that I will describe below.
>
> For test 090, I have the following setup:
>
> mkfs.xfs -f -llogdev=/dev/ram1 -rrtdev=/dev/ram2 -bsize=4096 /dev/ram0
> mount -t xfs -o attr2,rtdev=/dev/ram2,logdev=/dev/ram1 /dev/ram0 /mnt/test
>
> mkfs.xfs -f -llogdev=/dev/ram4 -rrtdev=/dev/ram5 -bsize=4096 /dev/ram3
> mount -t xfs -o attr2,rtdev=/dev/ram5,logdev=/dev/ram4 /dev/ram3 \
> /mnt/test-scratch
>
> I have the following local.config in xfstests:
> #!/bin/bash
> export TEST_DEV="/dev/ram0"
> export TEST_MNT="/mnt/test"
> export TEST_DIR="/mnt/test"
> export TEST_LOGDEV="/dev/ram1"
> export TEST_RTDEV="/dev/ram2"
> export SCRATCH_DEV="/dev/ram3"
> export SCRATCH_MNT="/mnt/test-scratch"
> export SCRATCH_LOGDEV="/dev/ram4"
> export SCRATCH_RTDEV="/dev/ram5"
> export USE_EXTERNAL="yes"
>
> All devices are ramdisks.
>
> I find with the above setup, when I run 'xfstests/check 090', the test will
> block indefinitely on I/O for the first iteration of the test. This test was
> run using the latest code from the xfstests git repo and xfs git repo (kernel
> 3.1.0-rc9).
>
> For the other failure scenario, I have the following setup:
>
> mkfs.xfs -f -llogdev=/dev/ram1 -rrtdev=/dev/ram2 -bsize=4096 /dev/ram0
> mount -t xfs -o attr2,rtdev=/dev/ram2,logdev=/dev/ram1 /dev/ram0 /mnt/test
> xfs_io -c 'chattr +t' /mnt/test
>
> I have setup all files under /mnt/test to be placed in the realtime portion
> using the realtime inherit flag set through xfs_io. If I do not use the flag
> and instead create the file using xfs_io and 'pwrite', I still encounter the
> same issue O_DIRECT hang issue.
>
> Using a fairly simple C application (below), the call to write() will hang the
> system indefinitely on kernel 2.6.32, or cause the kernel to kill the process on
> kernel 3.1.0-rc9 reporting a NULL pointer dereference (message below). I have
> traced the issue back to the use of the O_DIRECT and O_TRUNC flags. If the file
> to write to has been preallocated using xfs_io's 'pwrite' and 'truncate',
> omitting the O_TRUNC flag will allow the test application to complete without
> issue. As I understand it, the realtime subvolume has no requirements that all
> files be preallocated. Omitting the O_DIRECT flag will also allow the test to
> complete without issue, but with absolutely terrible I/O performance.
>
> Have I uncovered a legitimate or known bug or is there something wrong with my
> XFS setup?
>
> I can supply more information if needed. Thanks for any help.
>
> ----
>
> Kernel 3.1.0-rc9 reports a NULL pointer dereference:
>
> [ 657.406892] BUG: unable to handle kernel NULL pointer dereference at
> 0000000000000090
> [ 657.406905] IP: [<ffffffff813bae29>] _raw_spin_lock+0x9/0x20
> [ 657.406918] PGD 113f2e067 PUD 1144b3067 PMD 0
> [ 657.406926] Oops: 0002 [#1] SMP
> [ 657.406932] CPU 0
> [ 657.406935] Modules linked in: xfs exportfs brd binfmt_misc snd_pcm_oss ...
> [ 657.407008]
> [ 657.407012] Pid: 4573, comm: write-bench Not tainted 3.1.0-rc9-0.5-acook-xfs
> #1 Gigabyte Technology Co., Ltd. X58A-UD3R/X58A-UD3R
> [ 657.407020] RIP: 0010:[<ffffffff813bae29>] [<ffffffff813bae29>]
> _raw_spin_lock+0x9/0x20
> [ 657.407027] RSP: 0018:ffff880115d83478 EFLAGS: 00010246
> [ 657.407031] RAX: 0000000000010000 RBX: 0000000000000000 RCX: 0000000000000003
> [ 657.407036] RDX: 0000000000000001 RSI: 000000004008aec1 RDI: 0000000000000090
> [ 657.407040] RBP: ffff880115d83478 R08: ffff880037407280 R09: 0000000000000000
> [ 657.407044] R10: 0000000000000001 R11: 0000000000000000 R12: 0080115d82000000
> [ 657.407048] R13: 0000000000001000 R14: 0000000000000000 R15: 0000000000000000
> [ 657.407053] FS: 00007f0a00ec3700(0000) GS:ffff88011fc00000(0000)
> knlGS:0000000000000000
> [ 657.407058] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [ 657.407062] CR2: 0000000000000090 CR3: 000000010df5d000 CR4: 00000000000006f0
> [ 657.407066] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [ 657.407071] DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
> [ 657.407076] Process write-bench (pid: 4573, threadinfo ffff880115d82000, task
> ffff880119720240)
> [ 657.407080] Stack:
> [ 657.407082] ffff880115d834d8 ffffffffa03b9954 0000000000000001 0000000000000
> 090
> [ 657.407088] 0002800900000000 ffff88003741a400 000000004008aec1 0000000000000
> 000
> [ 657.407094] 0000000000000008 00004008aec10000 0000000000028009 ffff8801161d4
> c80
> [ 657.407101] Call Trace:
> [ 657.407117] [<ffffffffa03b9954>] _xfs_buf_find+0x64/0x1e0 [xfs]
> [ 657.407129] [<ffffffffa03b9dd0>] xfs_buf_get+0x30/0x160 [xfs]
> [ 657.407140] [<ffffffffa03b9f16>] xfs_buf_read+0x16/0xa0 [xfs]
> [ 657.407158] [<ffffffffa040cf20>] xfs_trans_read_buf+0x1a0/0x2a0 [xfs]
> [ 657.407174] [<ffffffffa0414e1f>] xfs_rtbuf_get+0xcf/0xf0 [xfs]
> [ 657.407180] [<ffffffffa00c3664>] ? brd_make_request+0x54/0x4ac [brd]
> [ 657.407196] [<ffffffffa0415b6a>] xfs_rtget_summary+0x7a/0x110 [xfs]
> [ 657.407203] [<ffffffff811c82f0>] ? generic_make_request+0x2f0/0x3b0
> [ 657.407220] [<ffffffffa04162f2>] xfs_rtallocate_extent_size+0x82/0x2b0 [xfs]
> [ 657.407235] [<ffffffffa03cda17>] ? kmem_zone_alloc+0x77/0xe0 [xfs]
> [ 657.407252] [<ffffffffa0416aa0>] xfs_rtallocate_extent+0x140/0x1a0 [xfs]
> [ 657.407269] [<ffffffffa0403fe8>] ? xfs_trans_add_item+0x28/0x70 [xfs]
> [ 657.407286] [<ffffffffa03e106d>] xfs_bmap_rtalloc+0x18d/0x300 [xfs]
> [ 657.407303] [<ffffffffa03d923d>] ? xfs_bmap_search_multi_extents+0x6d/0x100
> [xfs]
> [ 657.407320] [<ffffffffa03e1215>] xfs_bmap_alloc+0x35/0x40 [xfs]
> [ 657.407337] [<ffffffffa03e12e9>] xfs_bmapi_allocate+0xc9/0x2c0 [xfs]
> [ 657.407354] [<ffffffffa03e18ec>] xfs_bmapi_write+0x40c/0x6f0 [xfs]
> [ 657.407368] [<ffffffffa03c276b>] xfs_iomap_write_direct+0x20b/0x3a0 [xfs]
> [ 657.407379] [<ffffffffa03b69e5>] __xfs_get_blocks+0x2b5/0x370 [xfs]
> [ 657.407389] [<ffffffffa03b6aaf>] xfs_get_blocks_direct+0xf/0x20 [xfs]
> [ 657.407396] [<ffffffff81142e4a>] __blockdev_direct_IO+0x5ba/0xb90
> [ 657.407407] [<ffffffffa03b651f>] xfs_vm_direct_IO+0x9f/0x120 [xfs]
> [ 657.407418] [<ffffffffa03b6aa0>] ? __xfs_get_blocks+0x370/0x370 [xfs]
> [ 657.407428] [<ffffffffa03b66d0>] ? xfs_finish_ioend_sync+0x30/0x30 [xfs]
> [ 657.407439] [<ffffffff810c17b8>] generic_file_direct_write+0xb8/0x190
> [ 657.407453] [<ffffffffa03bc8d0>] xfs_file_dio_aio_write+0x190/0x270 [xfs]
> [ 657.407469] [<ffffffffa03bcec2>] xfs_file_aio_write+0x252/0x260 [xfs]
> [ 657.407478] [<ffffffff812644ba>] ? tty_wakeup+0x3a/0x80
> [ 657.407486] [<ffffffff8110f6c1>] do_sync_write+0xd1/0x120
> [ 657.407495] [<ffffffff8119736d>] ? security_file_permission+0x1d/0xa0
> [ 657.407502] [<ffffffff8110fcfb>] vfs_write+0xcb/0x180
> [ 657.407509] [<ffffffff8110fea0>] sys_write+0x50/0x90
> [ 657.407517] [<ffffffff813c1492>] system_call_fastpath+0x16/0x1b
> [ 657.407522] Code: 90 00 00 01 00 75 04 f0 0f b1 17 0f 94 c2 0f b6 c2 85 c0 c9
> 0f 95 c0 0f b6 c0 c3 0f 1f 80 00 00 00 00 55 b8 00 00 01 00 48 89 e5 <f0> 0f
> [ 657.407566] RIP [<ffffffff813bae29>] _raw_spin_lock+0x9/0x20
> [ 657.407574] RSP <ffff880115d83478>
> [ 657.407578] CR2: 0000000000000090
> [ 657.407590] ---[ end trace e406d45b83e0d669 ]---
>
> Simple C application for testing:
>
> #include <stdio.h>
> #include <stdlib.h>
> #define __USE_GNU
> #include <fcntl.h>
>
> int
> main(int argc, char **argv)
> {
> int fd = -1;
> unsigned char *buffer, *buffer_orig;
> unsigned long size = 32 * 1024 * 1024; /* 32 MiB */
> unsigned long i = 0;
>
> if(argc < 2) {
> printf("%s <destfile>\n", argv[0]);
> return 1;
> }
>
> buffer_orig = buffer = malloc(size + 4096);
In my opinion, this is not the way to allocate memory and use for
O_DIRECT approach. Instead you can make use of valloc() or
posix_memalign()/memalign
Or if you are still using malloc() - then you probably did the right
thing by allocating for alignment like malloc(size + 4096);
but, store this is permanent pointer and make use of aligned pointer :
perm_pointer = malloc(size +4096);
For aligning you need an expression like this:
align_pointer = (perm_pointer + 4096) & ~(4095);// If you wish to
align for 512, do it like (p+512) & ~(511);
> if(!buffer) {
> perror("malloc");
> return 2;
> }
>
> /* open file for direct write on realtime partition */
> fd = open(argv[1], O_TRUNC | O_CREAT | O_DIRECT | O_WRONLY, 0666);
> if(fd >= 0) {
>
> /* write hangs machine, or kill process depending on kernel version */
> if(0 > write(fd, buffer, size)) {
> perror("write");
> }
>
> close(fd);
> }
>
> free(buffer_orig);
free(perm_pointer); // freeing the original pointer which has the size
history for malloc.
>
> return 0;
> }
>
>
>
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs
>
Thanks & Regards,
Amit Sahrawat
_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs
