1*4882a593SmuzhiyunFrom 39b381252c32275079344d30de18b76fda4bba26 Mon Sep 17 00:00:00 2001 2*4882a593SmuzhiyunFrom: Logan Gunthorpe <logang@deltatee.com> 3*4882a593SmuzhiyunDate: Wed, 27 Jul 2022 15:52:45 -0600 4*4882a593SmuzhiyunSubject: [PATCH] tests/00readonly: Run udevadm settle before setting ro 5*4882a593Smuzhiyun 6*4882a593SmuzhiyunIn some recent kernel versions, 00readonly fails with: 7*4882a593Smuzhiyun 8*4882a593Smuzhiyun mdadm: failed to set readonly for /dev/md0: Device or resource busy 9*4882a593Smuzhiyun ERROR: array is not read-only! 10*4882a593Smuzhiyun 11*4882a593SmuzhiyunThis was traced down to a race condition with udev holding a reference 12*4882a593Smuzhiyunto the block device at the same time as trying to set it read only. 13*4882a593Smuzhiyun 14*4882a593SmuzhiyunTo fix this, call udevadm settle before setting the array read only. 15*4882a593Smuzhiyun 16*4882a593SmuzhiyunUpstream-Status: Backport [https://git.kernel.org/pub/scm/utils/mdadm/mdadm.git/commit/?id=39b381252c32275079344d30de18b76fda4bba26] 17*4882a593Smuzhiyun 18*4882a593SmuzhiyunSigned-off-by: Logan Gunthorpe <logang@deltatee.com> 19*4882a593SmuzhiyunSigned-off-by: Jes Sorensen <jsorensen@fb.com> 20*4882a593SmuzhiyunSigned-off-by: Mingli Yu <mingli.yu@windriver.com> 21*4882a593Smuzhiyun--- 22*4882a593Smuzhiyun tests/00readonly | 1 + 23*4882a593Smuzhiyun 1 file changed, 1 insertion(+) 24*4882a593Smuzhiyun 25*4882a593Smuzhiyundiff --git a/tests/00readonly b/tests/00readonly 26*4882a593Smuzhiyunindex 39202487..afe243b3 100644 27*4882a593Smuzhiyun--- a/tests/00readonly 28*4882a593Smuzhiyun+++ b/tests/00readonly 29*4882a593Smuzhiyun@@ -12,6 +12,7 @@ do 30*4882a593Smuzhiyun $dev1 $dev2 $dev3 $dev4 --assume-clean 31*4882a593Smuzhiyun check nosync 32*4882a593Smuzhiyun check $level 33*4882a593Smuzhiyun+ udevadm settle 34*4882a593Smuzhiyun mdadm -ro $md0 35*4882a593Smuzhiyun check readonly 36*4882a593Smuzhiyun state=$(cat /sys/block/md0/md/array_state) 37*4882a593Smuzhiyun-- 38*4882a593Smuzhiyun2.25.1 39*4882a593Smuzhiyun 40