Re: [PATCH] z3fold: fix wrong handling of headless pages

2018-11-08 Thread Andrew Morton
On Thu, 8 Nov 2018 22:45:40 +0900 Jongseok Kim wrote: > Yes, you are right. > I think that's the best way to deal it. > Thank you. I did this: Link: http://lkml.kernel.org/r/20181105162225.74e8837d03583a9b707cf...@gmail.com Signed-off-by: Vitaly Wool Signed-off-by: Jongseok Kim Reported-by-

Re: [PATCH] z3fold: fix wrong handling of headless pages

2018-11-08 Thread Jongseok Kim
Yes, you are right. I think that's the best way to deal it. Thank you. Best regards, Jongseok > Den tors 8 nov. 2018 kl 13:34 skrev �� : > > > > Hi Vitaly, > > thank you for the reply. > > > > I agree your a new solution is more comprehensive and drop my patch is > > simple way. > > But, I t

Re: [PATCH] z3fold: fix wrong handling of headless pages

2018-11-08 Thread Vitaly Wool
Den tors 8 nov. 2018 kl 13:34 skrev 김종석 : > > Hi Vitaly, > thank you for the reply. > > I agree your a new solution is more comprehensive and drop my patch is simple > way. > But, I think it's not fair. > If my previous patch was not wrong, is (my patch -> your patch) the right way? I could apply

Re: [PATCH] z3fold: fix wrong handling of headless pages

2018-11-05 Thread Vitaly Wool
Hi Jongseok, Den fre 6 juli 2018 kl 07:11 skrev Jongseok Kim : > > During the processing of headless pages in z3fold_reclaim_page(), > there was a problem that the zhdr pointed to another page > or a page was already released in z3fold_free(). So, the wrong page > is encoded in headless, or test_b