Re: [PATCH v2] Monitor: print message before quit for no array to monitor

On 9/2/21 3:08 PM, Tkaczyk, Mariusz wrote:

Hi Coly,
On 02.09.2021 04:36, Coly Li wrote:

quoted

@@ -258,6 +258,7 @@ int Monitor(struct mddev_dev *devlist,

              if (oneshot)
                  break;
              else if (!anyredundant) {
+                pr_err("Stop for no array to monitor\n");
                  break;

IMO the message is not precise enough. See that if there is only raid0
then it also stops. Something like:
"No array with redundancy detected, stopping" is more detailed.

Copied. I will post v3 patch with your suggestion.

Thanks.

Coly Li