Re: [PATCH 2/7] fsfreeze: emergency thaw will deadlock on s_umount

[Fernando Luis Vazquez Cao <fernando_b1@lab.ntt.co.jp>]

Hi Jan,

On 2012/07/11 18:06, Jan Kara wrote:

>> By the way, is there anything specific in patch 2
>> (apart from the conditional locking ugliness) that you want
>> to me to change? Maybe resolve the conflicts with your patch
>> set?
>    I would be happier if you based your patches on my patch set (less work
> for me ;). OTOH your patches are simpler so Al might decide to merge them
> first. So all in all my opinion doesn't matter much here. It is Al who
> decides...

I will rebase against Al's vfs tree. If your patches get merged first
I do not mind resolving the merge conflicts you mentioned.

>> Ok. Could you get your Acked-by or Reviewed-by for the changes
>> you agree with?
>    Please fix the patch 1 and the conditional locking in patch 2 (that will
> also require changes to other patches in the series - they might be smaller
> if you reorder patches as I suggested). Then I'll check the whole series
> and can give you my Reviewed-by.

Ok, I will fix patch 1 and 2 (probably without reordering) and repost.

Thank you again for your comments!

Regards,
Fernando