On Mon, Jul 30, 2012 at 7:57 PM, Caspar Zhang <[email protected]> wrote:
>
> This patch makes the tests in mem/ dir use numa_helper in libkerntest.
>
> Signed-off-by: Caspar Zhang <[email protected]>

Inline patches are easier to review...

>-              tst_brkm(TCONF, NULL, "required a NUMA system.");
>+              tst_brkm(TCONF, NULL, "require a NUMA system.");

`requires` is probably what was meant here, but nonetheless the
comment is incomplete.

> /* shared */
>-long count_numa(long nodes[]);
>+unsigned get_a_numa_node(void (*cleanup_fn)(void));

Please use `unsigned int`.

Thanks,
-Garrett

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to