Re: [RFC][PATCH] dm: Remove dm_bufio_cond_resched()

From: Mike Snitzer
Date: Tue Sep 13 2016 - 09:40:09 EST


On Tue, Sep 13 2016 at 4:45am -0400,
Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:

> Hi all,
>
> While grepping for PREEMPT_VOLUNTARY I ran into dm_bufio_cond_resched()
> and wondered WTH it was about.
>
> Is there anything wrong with the below patch?

No, I'll pick it up for 4.9 merge. Mikulas added it for sparc or
something. I cannot recall _the_ reason (I wasn't maintaining DM back
then) but at the time both Alasdair and Joe Thornber reasoned that it
needed to go -- and that if it was really needed that it should be done
in terms of a proper patch to sched.h, etc.

So I'm not sure how this dm-bufio local cond_resched() wrapper still got
in... happy to take your patch.

Please respond with whatever SOB you'd like applied to the patch header.

Thanks,
Mike