Veerendra wrote:
> This patch has 4 files..
> 
> check_netns_enabled.c - checks for the network ns is enabled 
> initialize.sh - Initializes the common functions and variables
> Makefile  - Makefile
> runnetnstest.sh - Creates a batch file to run the tests.

Looks like you didn't fix these typos:

testcases/kernel/containers/netns/check_netns_enabled.c
     * There's  a typo in the comment:
       "trying to use CLONE_NEWNET and CLONE_NETNS."
           should be                       -----------
       "trying to use CLONE_NEWNET and CLONE_NEWNS."

     * You left some commented code in the test:
         //if (!kernel_version_newenough())

Otherwise, these changes looks good.

Acked-by: Benjamin Thery <[EMAIL PROTECTED]>

Benjamin
> 
> 
> 
> 
> 
> 
> 
> 
> ------------------------------------------------------------------------
> 
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Ltp-list mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ltp-list


-- 
B e n j a m i n   T h e r y  - BULL/DT/Open Software R&D

    http://www.bull.com

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to