All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] mdadm.8: Copy --no-degraded clarification from --help output
@ 2021-09-29  8:21 Vladimir Panteleev
  2021-10-13 10:44 ` Tkaczyk, Mariusz
  0 siblings, 1 reply; 2+ messages in thread
From: Vladimir Panteleev @ 2021-09-29  8:21 UTC (permalink / raw)
  To: linux-raid; +Cc: Vladimir Panteleev

Make it easier to discover this POLA violation.
---
 mdadm.8.in | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/mdadm.8.in b/mdadm.8.in
index 96846035..74cca534 100644
--- a/mdadm.8.in
+++ b/mdadm.8.in
@@ -1159,6 +1159,12 @@ are present.  This is only needed with
 and can be used if the physical connections to devices are
 not as reliable as you would like.
 
+Note that "all expected drives" means as many as were present the last
+time the array was running as recorded in the superblock.  If the
+array was already degraded, and the missing device is not a new
+problem, it will still be assembled.  It is only newly missing devices
+that cause the array not to be started.
+
 .TP
 .BR \-a ", " "\-\-auto{=no,yes,md,mdp,part}"
 See this option under Create and Build options.
-- 
2.33.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] mdadm.8: Copy --no-degraded clarification from --help output
  2021-09-29  8:21 [PATCH] mdadm.8: Copy --no-degraded clarification from --help output Vladimir Panteleev
@ 2021-10-13 10:44 ` Tkaczyk, Mariusz
  0 siblings, 0 replies; 2+ messages in thread
From: Tkaczyk, Mariusz @ 2021-10-13 10:44 UTC (permalink / raw)
  To: Vladimir Panteleev, linux-raid, NeilBrown, Jes Sorensen

On 29.09.2021 10:21, Vladimir Panteleev wrote:
> Make it easier to discover this POLA violation.
> ---
>   mdadm.8.in | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/mdadm.8.in b/mdadm.8.in
> index 96846035..74cca534 100644
> --- a/mdadm.8.in
> +++ b/mdadm.8.in
> @@ -1159,6 +1159,12 @@ are present.  This is only needed with
>   and can be used if the physical connections to devices are
>   not as reliable as you would like.
>   
> +Note that "all expected drives" means as many as were present the last
> +time the array was running as recorded in the superblock.  If the
> +array was already degraded, and the missing device is not a new
> +problem, it will still be assembled.  It is only newly missing devices
> +that cause the array not to be started.
> +
>   .TP
>   .BR \-a ", " "\-\-auto{=no,yes,md,mdp,part}"
>   See this option under Create and Build options.
> 

I think that it is not necessary anymore. This is default behavior,
at least for IMSM/external.
If we are using assemble with --scan then it is probably done
manually. Do we should care about unstable connections?

Normally, udev calls mdadm -I on each drive.
There is also a timer which calls #mdadm -R on array
after timeout.

I would consider to drop --no-degraded support and adapt
assemble behavior if is different.
Neil, Jes could you look?

Thanks,
Mariusz



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-10-13 10:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-29  8:21 [PATCH] mdadm.8: Copy --no-degraded clarification from --help output Vladimir Panteleev
2021-10-13 10:44 ` Tkaczyk, Mariusz

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.