Messages in this thread | | | Date | Wed, 21 Nov 2012 11:31:49 -0800 | From | Andrew Morton <> | Subject | Re: [RFT PATCH v1 1/5] mm: introduce new field "managed_pages" to struct zone |
| |
On Wed, 21 Nov 2012 22:36:56 +0800 Jiang Liu <liuj97@gmail.com> wrote:
> > void mod_zone_managed_pages(struct zone *zone, signed long delta) > > { > > WARN_ON(system_state != SYSTEM_BOOTING && > > !is_locked_memory_hotplug()); > > zone->managed_pages += delta; > > } > This seems a little overhead because __free_pages_bootmem() is on the hot path > and will be called many times at boot time.
Maybe, maybe not. These things are measurable so let's not just guess.
But I'm not really recommending that we do this - there are all sorts of things we *could* check and warn about, but we don't. Potential errors in this area don't seem terribly important.
| |