Re: [PATCH 2/3] cpumask: use new cpus_scnprintf function v3

From: Bert Wesarg
Date: Tue Apr 08 2008 - 15:40:22 EST


On Tue, Apr 8, 2008 at 8:43 PM, Mike Travis <travis@xxxxxxx> wrote:
> --- linux-2.6.x86.sched.orig/drivers/base/node.c
> +++ linux-2.6.x86.sched/drivers/base/node.c
> @@ -19,22 +19,34 @@ static struct sysdev_class node_class =
> };
>
>
> -static ssize_t node_read_cpumap(struct sys_device * dev, char * buf)
> +static ssize_t node_read_cpumap(struct sys_device *dev, int type, char *buf)
> {
> struct node *node_dev = to_node(dev);
> node_to_cpumask_ptr(mask, node_dev->sysdev.id);
> int len;
>
> - /* 2004/06/03: buf currently PAGE_SIZE, need > 1 char per 4 bits. */
> - BUILD_BUG_ON(MAX_NUMNODES/4 > PAGE_SIZE/2);
> + /* 2008/04/07: buf currently PAGE_SIZE, need 9 chars per 32 bits. */
> + BUILD_BUG_ON((NR_CPUS/32 * 9) > (PAGE_SIZE-1));
Is this right, that you switch from MAX_NUMNODES to NR_CPUS?

Regards
Bert
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/