Re: [LTP] [PATCH 3/3] syscalls/madvise02: fix madvise case due to kernel change

Zhouping Liu
2013-03-07T08:26:50Z

[-- Attachment #1.1: Type: text/plain, Size: 1253 bytes --]

On 03/07/2013 03:38 PM, Caspar Zhang wrote:
> since kernel commit 1998cc0 (after kernel-3.8 released),
> madvise(MADV_WILLNEED) to anonymous memory doesn't return -EBADF now, as
> now we support swap prefetch. Thus the 5th testcase in madvise02 will
> not pass. This patch fixes this by adding a tst_kvercmp() condition, and
> will give a pass if kernel is >=3.9 (including the RCs).
>
> Signed-off-by: Caspar Zhang <caspar@casparzhang.com>

met the issue on 3.9.0-rc1+, and the patch looks good for me,
so Reviewed-by: Zhouping Liu <zliu@redhat.com>

> ---
> testcases/kernel/syscalls/madvise/madvise02.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
>
>
> ------------------------------------------------------------------------------
> Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester
> Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the
> endpoint security space. For insight on selecting the right partner to
> tackle endpoint security challenges, access the full report.
> http://p.sf.net/sfu/symantec-dev2dev
>
>
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list

[-- Attachment #1.2: Type: text/html, Size: 2440 bytes --]

[-- Attachment #2: Type: text/plain, Size: 397 bytes --]

------------------------------------------------------------------------------
Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester
Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the
endpoint security space. For insight on selecting the right partner to
tackle endpoint security challenges, access the full report.
http://p.sf.net/sfu/symantec-dev2dev

[-- Attachment #3: Type: text/plain, Size: 155 bytes --]