----- Original Message -----
> From: "Lans Zhang" <[email protected]>
> To: "Zhouping Liu" <[email protected]>, "Jan Stancek" <[email protected]>
> Cc: "Wanlong Gao" <[email protected]>, [email protected]
> Sent: Wednesday, 17 April, 2013 7:09:00 AM
> Subject: [LTP][PATCH V2] Fix short of nodemask array.
>
> Changes since V1:
> 1. Enhance patch header.
> 2. Use BITS_PER_LONG and set_node() to simplify the logic.
>
> In kernel, if user specified more nodes, e.g, 512 nodes, than the
> supported, e.g, 4 nodes, kernel will check if the non supported
> part, e.g, 508 nodes, is all zeroed in node bitmap. Currently, we
> are overrunning "nmask", whose length is shorter than MAXNODES,
> and where the unsupported bits should be cleared to pass the check.
>
> Signed-off-by: Lans Zhang <[email protected]>
Hi,
'v' should be probably dropped here, since it's not used and rest
of patch uses 2 parameters anyway:
> +static inline void set_node(unsigned long *array, unsigned int node,
> + unsigned int v)
> +{
> + array[node / BITS_PER_LONG] |= 1UL << (node % BITS_PER_LONG);
> +}
>
Other than that, it looks good to me.
Regards,
Jan
------------------------------------------------------------------------------
Precog is a next-generation analytics platform capable of advanced
analytics on semi-structured data. The platform includes APIs for building
apps and a phenomenal toolset for data science. Developers can use
our toolset for easy data analysis & visualization. Get a free account!
http://www2.precog.com/precogplatform/slashdotnewsletter
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
