Re: [PATCH v2 0/4] test for XFS umount hang caused by the pending dquota log item in AIL

[Eryu Guan <eguan@redhat.com>]

On Wed, Nov 08, 2017 at 04:02:46PM +0800, Hou Tao wrote:
> Hi,
> 
> This is the v2 patchset for testing the XFS umount hang problem caused by
> the pending dquota log item in AIL. The problem has not been fixed in
> XFS yet, and Carlos Maiolino is working on it [1]. I had tested
> it on 4.14-rc7, and the problem can be reproduced reliably.

Thanks a lot for the updated version! But unfortunately, I still
couldn't reproduce the hang, I'll provide more information in comments
to each specific patch.

Thanks,
Eryu

> 
> Changes since v1:
> * dmflakey: support multiple dm targets for a dm-flakey device
> 	* no update
> 
> * xfs: test for umount hang caused by the pending dquota log item in AIL
> 	* a bunch of suggestions from Eryu Guan
> 	* update the dquota buffer after the error injection to better
> 	  reproduce the problem
> 
> * common/rc: support checking the version of dm-target in
> _require_dm_target()
> 	* new patch
> 	* introduce a way to check the availability of error_write feature
> 	in dm-flakey target, and accomplish it by checking the version of
> 	dm-target.
> 
> * common/rc: factor out _get|set_xfs_scratch_sb_field()
> 	* new patch
> 	* the modified test cases had been tested after applying the patch:
> 	(1) xfs/098 failed, but the failure has nothing to do with the patch
> 	(2) xfs/339 and xfs/340 were not runnable, because rtdev + rmap is not
> 	  supported now.
> 	(3) the remaing test cases passed
> 
> v1:
> 	* https://www.spinics.net/lists/fstests/msg07622.html
> 
> Comments and questions are welcome.
> 
> Regards,
> Tao
> 
> [1]: https://www.spinics.net/lists/linux-xfs/msg11614.html
> ---
> Hou Tao (4):
>   dmflakey: support multiple dm targets for a dm-flakey device
>   common/rc: support checking the version of dm-target in
>     _require_dm_target()
>   xfs: test for umount hang caused by the pending dquota log item in AIL
>   common/rc: factor out _get|set_xfs_scratch_sb_field()
> 
>  common/dmflakey   |   2 +-
>  common/rc         |  39 +++++++++++--
>  common/xfs        |  20 +++++++
>  tests/xfs/007     |   6 +-
>  tests/xfs/098     |   4 +-
>  tests/xfs/186     |   3 +-
>  tests/xfs/199     |  13 ++---
>  tests/xfs/307     |  11 +---
>  tests/xfs/308     |  11 +---
>  tests/xfs/339     |   6 +-
>  tests/xfs/340     |   2 +-
>  tests/xfs/999     | 163 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  tests/xfs/999.out |   2 +
>  tests/xfs/group   |   1 +
>  14 files changed, 238 insertions(+), 45 deletions(-)
>  create mode 100755 tests/xfs/999
>  create mode 100644 tests/xfs/999.out
> 
> -- 
> 2.9.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html