Messages in this thread | | | Date | Wed, 14 Aug 2013 16:07:48 -0400 | From | Tejun Heo <> | Subject | Re: [PATCH v7 2/2] mm: make lru_add_drain_all() selective |
| |
Hello, Chris.
On Wed, Aug 14, 2013 at 01:18:31PM -0400, Chris Metcalf wrote: > On 8/14/2013 12:57 PM, Tejun Heo wrote: > > Hello, Chris. > > > > On Wed, Aug 14, 2013 at 12:03:39PM -0400, Chris Metcalf wrote: > >> Tejun, I don't know if you have a better idea for how to mark a > >> work_struct as being "not used" so we can set and test it here. > >> Is setting entry.next to NULL good? Should we offer it as an API > >> in the workqueue header? > > Maybe simply defining a static cpumask would be cleaner? > > I think you're right, actually. Andrew, Tejun, how does this look?
Looks good to me. Please feel free to add
Reviewed-by: Tejun Heo <tj@kernel.org>
Thanks.
-- tejun
| |