Hi,
On 05/04/19 14:36, Peter Zijlstra wrote:
> 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.
OK.
Thanks,
- Juri
