Re: [PATCH v3 1/3] mm/free_pcppages_bulk: update pcp->count inside

2018-02-26 Thread Aaron Lu
On Tue, Feb 27, 2018 at 09:56:13AM +0800, Aaron Lu wrote: > On Mon, Feb 26, 2018 at 01:48:14PM -0800, David Rientjes wrote: > > On Mon, 26 Feb 2018, Aaron Lu wrote: > > > > > Matthew Wilcox found that all callers of free_pcppages_bulk() currently > > > update pcp->count immediately after so it's

Re: [PATCH v3 1/3] mm/free_pcppages_bulk: update pcp->count inside

2018-02-26 Thread Aaron Lu
On Tue, Feb 27, 2018 at 09:56:13AM +0800, Aaron Lu wrote: > On Mon, Feb 26, 2018 at 01:48:14PM -0800, David Rientjes wrote: > > On Mon, 26 Feb 2018, Aaron Lu wrote: > > > > > Matthew Wilcox found that all callers of free_pcppages_bulk() currently > > > update pcp->count immediately after so it's

Re: [PATCH v3 1/3] mm/free_pcppages_bulk: update pcp->count inside

2018-02-26 Thread Aaron Lu
On Mon, Feb 26, 2018 at 01:48:14PM -0800, David Rientjes wrote: > On Mon, 26 Feb 2018, Aaron Lu wrote: > > > Matthew Wilcox found that all callers of free_pcppages_bulk() currently > > update pcp->count immediately after so it's natural to do it inside > > free_pcppages_bulk(). > > > > No

Re: [PATCH v3 1/3] mm/free_pcppages_bulk: update pcp->count inside

2018-02-26 Thread Aaron Lu
On Mon, Feb 26, 2018 at 01:48:14PM -0800, David Rientjes wrote: > On Mon, 26 Feb 2018, Aaron Lu wrote: > > > Matthew Wilcox found that all callers of free_pcppages_bulk() currently > > update pcp->count immediately after so it's natural to do it inside > > free_pcppages_bulk(). > > > > No

Re: [PATCH v3 1/3] mm/free_pcppages_bulk: update pcp->count inside

2018-02-26 Thread David Rientjes
On Mon, 26 Feb 2018, Aaron Lu wrote: > Matthew Wilcox found that all callers of free_pcppages_bulk() currently > update pcp->count immediately after so it's natural to do it inside > free_pcppages_bulk(). > > No functionality or performance change is expected from this patch. > > Suggested-by:

Re: [PATCH v3 1/3] mm/free_pcppages_bulk: update pcp->count inside

2018-02-26 Thread David Rientjes
On Mon, 26 Feb 2018, Aaron Lu wrote: > Matthew Wilcox found that all callers of free_pcppages_bulk() currently > update pcp->count immediately after so it's natural to do it inside > free_pcppages_bulk(). > > No functionality or performance change is expected from this patch. > > Suggested-by:

[PATCH v3 1/3] mm/free_pcppages_bulk: update pcp->count inside

2018-02-26 Thread Aaron Lu
Matthew Wilcox found that all callers of free_pcppages_bulk() currently update pcp->count immediately after so it's natural to do it inside free_pcppages_bulk(). No functionality or performance change is expected from this patch. Suggested-by: Matthew Wilcox Signed-off-by:

[PATCH v3 1/3] mm/free_pcppages_bulk: update pcp->count inside

2018-02-26 Thread Aaron Lu
Matthew Wilcox found that all callers of free_pcppages_bulk() currently update pcp->count immediately after so it's natural to do it inside free_pcppages_bulk(). No functionality or performance change is expected from this patch. Suggested-by: Matthew Wilcox Signed-off-by: Aaron Lu ---