On Mon, Sep 21, 2015 at 11:23 PM, Ingo Molnar <[email protected]> wrote:
> +
> + for_each_process(g) {
> + struct task_struct *p;
> + struct mm_struct *mm;
> pgd_t *pgd;
> spinlock_t *pgt_lock;
>
> + p = find_lock_task_mm(g);
> + if (!p)
> + continue;
> +
> + mm = p->mm;
So quite frankly, this is *much* better than the earlier version that
walked over all threads.
However, this now becomes a pattern for the series, and that just makes me think
"Why is this not a 'for_each_mm()' pattern helper?"
if it only showed up once, that would be one thing. But this
patch-series makes it a thing. Which is why I wonder..
Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
