Re: [LTP] [PATCH] ltp: handle missing online file

2013-08-21 Thread Wanlong Gao
On 08/21/2013 04:34 AM, Dave Kleikamp wrote: gather_node_cpus() fails if the /sys/devices/system/node/nodeX/cpuX/online file is missing for any cpu other than cpu0. The absence of the online file should not be treated as a failure, but as an indication that the cpu is not hot-pluggable and

Re: [LTP] [PATCH] aio_fsync: wait until aio_fsync completes

2013-08-21 Thread Wanlong Gao
On 08/12/2013 07:03 PM, Jan Stancek wrote: This patch introduces wait so that testcases exit doesn't race with completion of aio_fsync. This is to avoid SIGSEGV: #0 0x03fffd1f4a24 in _IO_flush_all_lockp () at genops.c:850 #1 0x03fffd1f4b8e in _IO_cleanup () at genops.c:1010 #2

Re: [LTP] [PATCH] ltp: handle missing online file

2013-08-21 Thread Jan Stancek
- Original Message - From: Dave Kleikamp dave.kleik...@oracle.com To: ltp-list@lists.sourceforge.net Sent: Tuesday, 20 August, 2013 10:34:39 PM Subject: [LTP] [PATCH] ltp: handle missing online file gather_node_cpus() fails if the /sys/devices/system/node/nodeX/cpuX/online file

Re: [LTP] [PATCH] syscalls/migrate_pages: fix nodemask memory allocation

2013-08-21 Thread chrubis
Hi! --- .../syscalls/migrate_pages/migrate_pages01.c |5 - .../syscalls/migrate_pages/migrate_pages02.c |5 - 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/testcases/kernel/syscalls/migrate_pages/migrate_pages01.c

Re: [LTP] [PATCH] syscalls/migrate_pages: fix nodemask memory allocation

2013-08-21 Thread Stanislav Kholmanskikh
On 08/21/2013 01:17 PM, chru...@suse.cz wrote: Hi! Hi. --- .../syscalls/migrate_pages/migrate_pages01.c |5 - .../syscalls/migrate_pages/migrate_pages02.c |5 - 2 files changed, 8 insertions(+), 2 deletions(-) diff --git

Re: [LTP] [PATCH] syscalls/migrate_pages: fix nodemask memory allocation

2013-08-21 Thread chrubis
Hi! Also I would create a macro to do that NODEMASK_SIZE() and put it into migrate_pages_common.h header. I like the variant of a ALIGN general macro addition (for example, in test.h) because it may be useful in further testcases. Ah, I've missed that Jan replied to the patch before.

Re: [LTP] [PATCH/RFC] aiodio: make read_sparse use same mode as writers

2013-08-21 Thread chrubis
Hi! Starting with 3.10 dio_sparse sporadically fails, because read() in read_sparse returns garbage and dio_sparse testcase fails with: dio_sparse0 TINFO : Dirtying free blocks dio_sparse0 TINFO : Starting I/O tests non zero buffer at buf[0] =

[LTP] [PATCH V2 1/3] Added ALIGN, __ALIGN_MASK macroses

2013-08-21 Thread Stanislav Kholmanskikh
Signed-off-by: Stanislav Kholmanskikh stanislav.kholmansk...@oracle.com --- include/test.h |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/include/test.h b/include/test.h index d9eba31..a517ff9 100644 --- a/include/test.h +++ b/include/test.h @@ -105,6 +105,12 @@

[LTP] [PATCH V2 3/3] lib/numa_helper.c: fix nodemask_size

2013-08-21 Thread Stanislav Kholmanskikh
Now nodemask_size is rounded up to the next multiple of sizeof(nodemask_t). Signed-off-by: Stanislav Kholmanskikh stanislav.kholmansk...@oracle.com --- testcases/kernel/lib/numa_helper.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [LTP] [PATCH] ltp: handle missing online file

2013-08-21 Thread Dave Kleikamp
On 08/21/2013 01:50 AM, Jan Stancek wrote: - Original Message - From: Dave Kleikamp dave.kleik...@oracle.com To: ltp-list@lists.sourceforge.net Sent: Tuesday, 20 August, 2013 10:34:39 PM Subject: [LTP] [PATCH] ltp: handle missing online file gather_node_cpus() fails if the

Re: [LTP] [PATCH V2 3/3] lib/numa_helper.c: fix nodemask_size

2013-08-21 Thread Jan Stancek
- Original Message - From: Stanislav Kholmanskikh stanislav.kholmansk...@oracle.com To: ltp-list@lists.sourceforge.net Cc: vasily isaenko vasily.isae...@oracle.com, jstan...@redhat.com Sent: Wednesday, 21 August, 2013 1:54:58 PM Subject: [PATCH V2 3/3] lib/numa_helper.c: fix

Re: [LTP] [PATCH V2 3/3] lib/numa_helper.c: fix nodemask_size

2013-08-21 Thread Stanislav Kholmanskikh
On 08/21/2013 04:29 PM, Jan Stancek wrote: - Original Message - From: Stanislav Kholmanskikh stanislav.kholmansk...@oracle.com To: ltp-list@lists.sourceforge.net Cc: vasily isaenko vasily.isae...@oracle.com, jstan...@redhat.com Sent: Wednesday, 21 August, 2013 1:54:58 PM Subject:

Re: [LTP] [PATCH/RFC] aiodio: make read_sparse use same mode as writers

2013-08-21 Thread chrubis
Hi! The README in the testcases directory says that the tests explicitly tests for consistency between buffered I/O and O_DIRECT. I will consult some kernel guys if this was ever supported and we will either remove the tests or try to find a bug in the kernel. I've looked at the test and

Re: [LTP] [PATCH V2 3/3] lib/numa_helper.c: fix nodemask_size

2013-08-21 Thread Jan Stancek
- Original Message - From: Stanislav Kholmanskikh stanislav.kholmansk...@oracle.com To: Jan Stancek jstan...@redhat.com Cc: ltp-list@lists.sourceforge.net, vasily isaenko vasily.isae...@oracle.com Sent: Wednesday, 21 August, 2013 3:22:04 PM Subject: Re: [PATCH V2 3/3]

Re: [LTP] [PATCH V2 2/3] syscalls/migrate_pages: fix nodemask memory allocation

2013-08-21 Thread Jan Stancek
- Original Message - From: Stanislav Kholmanskikh stanislav.kholmansk...@oracle.com To: ltp-list@lists.sourceforge.net Cc: vasily isaenko vasily.isae...@oracle.com, jstan...@redhat.com Sent: Wednesday, 21 August, 2013 1:54:57 PM Subject: [PATCH V2 2/3] syscalls/migrate_pages: fix

Re: [LTP] [PATCH/RFC] aiodio: make read_sparse use same mode as writers

2013-08-21 Thread chrubis
Hi! I've looked at the test and what it does is: 1. create file and make sure it's empty 2. truncate it to defined size 3. parent starts writing zeroes to it with O_DIRECT 4. childs reads it and expect to get zeroes at any time Hi, That looks correct, here's an strace excerpt:

Re: [LTP] open posix testsuite compilation

2013-08-21 Thread chrubis
Hi! PFB the steps I followed to compile open_posix_testsuite under LTP.I had placed ltp-full-20130109 directory under the following path in my work space in development host./nb/devraj/realtime/nv/ct/ltp/ltp-full-20130109I have a main Makefile in /nb/devraj/realtime/nv/ct/ltp path which

Re: [LTP] [PATCH 1/2] Add support for xfs quota tests

2013-08-21 Thread DAN LI
Hi! On 08/12/2013 07:53 PM, chru...@suse.cz wrote: Hi! Now this is getting a bit out of hand. What about we change the tests (we have) to take only device to work with and let each test prepare the filesystem it needs? This will simplify the runltp script to only prepare loop device. If