On Wed, Apr 03, 2019 at 10:46:47AM +0200, Juri Lelli wrote:
> +static inline void cpuset_read_only_lock(unsigned long *flags)
> +{
> +     local_irq_save(*flags);
> +     preempt_disable();
> +}
> +
> +static inline void cpuset_read_only_unlock(unsigned long *flags)
> +{
> +     local_irq_restore(*flags);
> +     preempt_enable();
> +}

You can ditch the preempt stuff. IRQs disabled already very much implies
!preemptible.

Reply via email to