[0/3] z3fold: stability / rt fixes

Message

Vitaly Wool Dec. 9, 2020, 2:51 p.m. UTC

This patchset addresses z3fold stability issues under stress load,
primarily in the reclaim and free aspects. Besides, it fixes the
locking problems that were only seen in real-time kernel
configuration.

Vitaly Wool (2):
  z3fold: simplify freeing slots
  z3fold: stricter locking and more careful reclaim

Mike Galbraith (1):
  z3fold: Remove preempt disabled sections for RT

 mm/z3fold.c | 191 ++++++++++++++++++++++++++--------------------------
 1 file changed, 96 insertions(+), 95 deletions(-)

Comments

Oleksandr Natalenko Dec. 18, 2020, 11:19 p.m. UTC | #1

On Wed, Dec 09, 2020 at 04:51:48PM +0200, Vitaly Wool wrote:
> This patchset addresses z3fold stability issues under stress load,
> primarily in the reclaim and free aspects. Besides, it fixes the
> locking problems that were only seen in real-time kernel
> configuration.
> 
> Vitaly Wool (2):
>   z3fold: simplify freeing slots
>   z3fold: stricter locking and more careful reclaim
> 
> Mike Galbraith (1):
>   z3fold: Remove preempt disabled sections for RT
> 
>  mm/z3fold.c | 191 ++++++++++++++++++++++++++--------------------------
>  1 file changed, 96 insertions(+), 95 deletions(-)
> 
> -- 
> 2.20.1
> 

Thanks for addressing this!

(I'd appreciate Cc'ing me on this though since I was the one who
reported it initially...)

Vitaly Wool Dec. 18, 2020, 11:26 p.m. UTC | #2

Hi Oleksandr,

On Sat, 19 Dec 2020, 00:19 Oleksandr Natalenko, <[e-mail]>
wrote:

> On Wed, Dec 09, 2020 at 04:51:48PM +0200, Vitaly Wool wrote:
> > This patchset addresses z3fold stability issues under stress load,
> > primarily in the reclaim and free aspects. Besides, it fixes the
> > locking problems that were only seen in real-time kernel
> > configuration.
> >
> > Vitaly Wool (2):
> >   z3fold: simplify freeing slots
> >   z3fold: stricter locking and more careful reclaim
> >
> > Mike Galbraith (1):
> >   z3fold: Remove preempt disabled sections for RT
> >
> >  mm/z3fold.c | 191 ++++++++++++++++++++++++++--------------------------
> >  1 file changed, 96 insertions(+), 95 deletions(-)
> >
> > --
> > 2.20.1
> >
>
> Thanks for addressing this!
>
> (I'd appreciate Cc'ing me on this though since I was the one who
> reported it initially...)
>

I'm glad it helped. Leaving you off the explicit CC-list is surely
unintentional. In fact, I didn't know about the original issue, I was just
added by Sebastian to the ongoing conversation and so I believe I CC'd
everyone who took part in it.

Best regards,
   Vitaly


> --
>   Oleksandr Natalenko (post-factum)
>