Re: [PATCH] coredump: Replace opencoded set_mask_bits()

From: Oleg Nesterov
Date: Fri Aug 07 2015 - 07:59:22 EST


On 08/07, Vineet Gupta wrote:
>
> --- a/fs/exec.c
> +++ b/fs/exec.c
> @@ -1690,15 +1690,10 @@ EXPORT_SYMBOL(set_binfmt);
> */
> void set_dumpable(struct mm_struct *mm, int value)
> {
> - unsigned long old, new;
> -
> if (WARN_ON((unsigned)value > SUID_DUMP_ROOT))
> return;
>
> - do {
> - old = ACCESS_ONCE(mm->flags);
> - new = (old & ~MMF_DUMPABLE_MASK) | value;
> - } while (cmpxchg(&mm->flags, old, new) != old);
> + set_mask_bits(&mm->flags, MMF_DUMPABLE_MASK, value);
> }

Acked-by: Oleg Nesterov <oleg@xxxxxxxxxx>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/