Messages in this thread | | | Date | Fri, 14 Jul 2017 14:01:11 +0100 | From | Mel Gorman <> | Subject | Re: [PATCH 1/9] mm, page_alloc: rip out ZONELIST_ORDER_ZONE |
| |
On Fri, Jul 14, 2017 at 01:56:16PM +0100, Mel Gorman wrote: > > if (!write) { > > - int len = sizeof("Default"); > > - if (copy_to_user(buffer, "Default", len)) > > + int len = sizeof("Node"); > > + if (copy_to_user(buffer, "Node", len)) > > return -EFAULT; > > Ok for the name. But what's with using sizeof?
Bah, sizeof static compile-time string versus char *. Never mind.
-- Mel Gorman SUSE Labs
| |