# Re: \[PATCH\] IncRemove: Set "auto-read" only after successful excl open.

- From: NeilBrown \<hidden\>
- Date: 2015-03-04 05:01:11 
- List: linux-raid
- Page: https://ratatoskr.run/linux-raid/2015/03/6629145
- Thread: https://ratatoskr.run/linux-raid/2015/02/6629123/t.md

```text
On Fri, 27 Feb 2015 15:47:54 +0100 Pawel Baldysiak
[ref] wrote:

> "mdadm -If" - triggered from udev rules when disk is removed from OS -
> tries to set array in auto-read-only mode. This can interrupt rebuild
> process which is started automatically, e.g. if array is mounted and
> spare disk is available (I/O error is detected faster than removing
> failed disk by mdadm).
> This patch prevents "mdadm -If" from setting array into "auto-read-only",
> by requiring exclusive open to succeed.
> 
> Signed-off-by: Pawel Baldysiak <redacted>
> ---
>  Incremental.c |   16 ++++++++++------
>  1 file changed, 10 insertions(+), 6 deletions(-)
> 
> diff --git a/Incremental.c b/Incremental.c
> index 87d9114..b12f144 100644
> --- a/Incremental.c
> +++ b/Incremental.c
> @@ -1712,12 +1712,16 @@ int IncrementalRemove(char *devname, char *id_path, int verbose)
>  		return 1;
>  	}
>  	sysfs_init(&mdi, -1, ent->devnm);
> -	if (sysfs_get_str(&mdi, NULL, "array_state",
> -			  buf, sizeof(buf)) > 0) {
> -		if (strncmp(buf, "active", 6) == 0 ||
> -		    strncmp(buf, "clean", 5) == 0)
> -			sysfs_set_str(&mdi, NULL,
> -				      "array_state", "read-auto");
> +	mdfd = open_dev_excl(ent->devnm);
> +	if (mdfd > 0) {
> +		close(mdfd);
> +		if (sysfs_get_str(&mdi, NULL, "array_state",
> +				  buf, sizeof(buf)) > 0) {
> +			if (strncmp(buf, "active", 6) == 0 ||
> +			    strncmp(buf, "clean", 5) == 0)
> +				sysfs_set_str(&mdi, NULL,
> +					      "array_state", "read-auto");
> +		}
>  	}
>  	mdfd = open_dev(ent->devnm);
>  	if (mdfd < 0) {


Applied, thanks.
NeilBrown
```

## Attachments

- [(unnamed)](https://ratatoskr.run/linux-raid/2015/03/6629145/attachment/0) (application/pgp-signature, 811 bytes)