lkml.org 
[lkml]   [2020]   [Jun]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v3 05/20] seqlock: lockdep assert non-preemptibility on seqcount_t write
    Date
    Preemption must be disabled before entering a sequence count write side
    critical section. Failing to do so, the seqcount read side can preempt
    the write side section and spin for the entire scheduler tick. If that
    reader belongs to a real-time scheduling class, it can spin forever and
    the kernel will livelock.

    Assert through lockdep that preemption is disabled for seqcount writers.

    Signed-off-by: Ahmed S. Darwish <a.darwish@linutronix.de>
    ---
    include/linux/seqlock.h | 27 ++++++++++++++++++++++-----
    1 file changed, 22 insertions(+), 5 deletions(-)

    diff --git a/include/linux/seqlock.h b/include/linux/seqlock.h
    index 057f7326a877..679c440b17fe 100644
    --- a/include/linux/seqlock.h
    +++ b/include/linux/seqlock.h
    @@ -419,12 +419,29 @@ static inline void raw_write_seqcount_latch(seqcount_t *s)
    smp_wmb(); /* increment "sequence" before following stores */
    }

    -static inline void write_seqcount_begin_nested(seqcount_t *s, int subclass)
    +static inline void __write_seqcount_begin_nested(seqcount_t *s, int subclass)
    {
    raw_write_seqcount_begin(s);
    seqcount_acquire(&s->dep_map, subclass, 0, _RET_IP_);
    }

    +static inline void write_seqcount_begin_nested(seqcount_t *s, int subclass)
    +{
    + lockdep_assert_preemption_disabled();
    + __write_seqcount_begin_nested(s, subclass);
    +}
    +
    +/*
    + * write_seqcount_begin() without lockdep non-preemptibility checks.
    + *
    + * Use for internal seqlock.h code where it's known that preemption is
    + * already disabled. For example, seqlock_t write side functions.
    + */
    +static inline void __write_seqcount_begin(seqcount_t *s)
    +{
    + __write_seqcount_begin_nested(s, 0);
    +}
    +
    /**
    * write_seqcount_begin() - start a seqcount_t write-side critical section
    * @s: Pointer to &typedef seqcount_t
    @@ -563,7 +580,7 @@ static inline unsigned read_seqretry(const seqlock_t *sl, unsigned start)
    static inline void write_seqlock(seqlock_t *sl)
    {
    spin_lock(&sl->lock);
    - write_seqcount_begin(&sl->seqcount);
    + __write_seqcount_begin(&sl->seqcount);
    }

    /**
    @@ -591,7 +608,7 @@ static inline void write_sequnlock(seqlock_t *sl)
    static inline void write_seqlock_bh(seqlock_t *sl)
    {
    spin_lock_bh(&sl->lock);
    - write_seqcount_begin(&sl->seqcount);
    + __write_seqcount_begin(&sl->seqcount);
    }

    /**
    @@ -618,7 +635,7 @@ static inline void write_sequnlock_bh(seqlock_t *sl)
    static inline void write_seqlock_irq(seqlock_t *sl)
    {
    spin_lock_irq(&sl->lock);
    - write_seqcount_begin(&sl->seqcount);
    + __write_seqcount_begin(&sl->seqcount);
    }

    /**
    @@ -639,7 +656,7 @@ static inline unsigned long __write_seqlock_irqsave(seqlock_t *sl)
    unsigned long flags;

    spin_lock_irqsave(&sl->lock, flags);
    - write_seqcount_begin(&sl->seqcount);
    + __write_seqcount_begin(&sl->seqcount);
    return flags;
    }

    --
    2.20.1
    \
     
     \ /
      Last update: 2020-06-30 07:45    [W:4.199 / U:0.200 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site