Messages in this thread | | | Date | Mon, 13 May 2013 22:20:53 +0300 | Subject | Re: [PATCH RESEND v3 00/11] mm: fixup changers of per cpu pageset's ->high and ->batch | From | Pekka Enberg <> |
| |
Hi Cody,
On Mon, May 13, 2013 at 10:08 PM, Cody P Schafer <cody@linux.vnet.ibm.com> wrote: > "Problems" with the current code: > 1. there is a lack of synchronization in setting ->high and ->batch in > percpu_pagelist_fraction_sysctl_handler() > 2. stop_machine() in zone_pcp_update() is unnecissary. > 3. zone_pcp_update() does not consider the case where percpu_pagelist_fraction is non-zero
Maybe it's just me but I find the above problem description confusing. How does the problem manifest itself? How did you find about it? Why do we need to fix all three problems in the same patch set?
Pekka
| |