[PATCH] Update documentation for sync_min and sync_max entries

[PATCH] Update documentation for sync_min and sync_max entries

am 15.03.2011 13:30:38 von CoolCold

This patch is against 2.6.37 (
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-2 .6.37.y.git
, commit id 97b197f07d15dbcd53328e75754e2f947814c733 ) but I didn't
notice any changes in 2.6.38 though.

linux/Documentation/md.txt is missing description for sync_min and
sync_max entries.
This patch adds description for sync_min and sync_max entries.

Signed-off-by: Roman Ovchinnikov
---
Documentation/md.txt | 10 ++++++++++
1 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/Documentation/md.txt b/Documentation/md.txt
index a81c7b4..8f8cb4f 100644
--- a/Documentation/md.txt
+++ b/Documentation/md.txt
@@ -552,6 +552,16 @@ also have
within the array where IO will be blocked. This is currently
only supported for raid4/5/6.

+ sync_min
+ sync_max
+ The two values, given as numbers of sectors, indicate a range
+ withing the array where 'check'/'repair' will operate. Must be
+ a multiple of chunk_size. When it reaches "sync_max" it will
+ pause, rather than complete.
+ You can use 'select' or 'poll' on "sync_completed" to wait for
+ that number to reach sync_max. Then you can either increase
+ "sync_max", or can write 'idle' to "sync_action".
+

Each active md device may also have attributes specific to the
personality module that manages it.
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Update documentation for sync_min and sync_max entries

am 18.03.2011 03:34:41 von NeilBrown

On Tue, 15 Mar 2011 15:30:38 +0300 CoolCold wrote:

> This patch is against 2.6.37 (
> git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-2 .6.37.y.git
> , commit id 97b197f07d15dbcd53328e75754e2f947814c733 ) but I didn't
> notice any changes in 2.6.38 though.
>
> linux/Documentation/md.txt is missing description for sync_min and
> sync_max entries.
> This patch adds description for sync_min and sync_max entries.

Applied, thanks.

It will go upsteam in due course.

NeilBrown


>
> Signed-off-by: Roman Ovchinnikov
> ---
> Documentation/md.txt | 10 ++++++++++
> 1 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/Documentation/md.txt b/Documentation/md.txt
> index a81c7b4..8f8cb4f 100644
> --- a/Documentation/md.txt
> +++ b/Documentation/md.txt
> @@ -552,6 +552,16 @@ also have
> within the array where IO will be blocked. This is currently
> only supported for raid4/5/6.
>
> + sync_min
> + sync_max
> + The two values, given as numbers of sectors, indicate a range
> + withing the array where 'check'/'repair' will operate. Must be
> + a multiple of chunk_size. When it reaches "sync_max" it will
> + pause, rather than complete.
> + You can use 'select' or 'poll' on "sync_completed" to wait for
> + that number to reach sync_max. Then you can either increase
> + "sync_max", or can write 'idle' to "sync_action".
> +
>
> Each active md device may also have attributes specific to the
> personality module that manages it.

--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Update documentation for sync_min and sync_max entries

am 11.05.2011 19:11:39 von CoolCold

There is orthographical mistake in patch, in word "withing", should be
"within". I guess this is too little for one more emailed patch, so
just notifying & marking line.

On Tue, Mar 15, 2011 at 3:30 PM, CoolCold wrote=
:
> This patch is against 2.6.37 (
> git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-2 .6.37.y.g=
it
> , commit id 97b197f07d15dbcd53328e75754e2f947814c733 ) but I didn't
> notice any changes in 2.6.38 though.
>
> linux/Documentation/md.txt is missing description for sync_min and
> sync_max entries.
> This patch adds description for sync_min and sync_max entries.
>
> Signed-off-by: Roman Ovchinnikov
> ---
> =A0Documentation/md.txt | =A0 10 ++++++++++
> =A01 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/Documentation/md.txt b/Documentation/md.txt
> index a81c7b4..8f8cb4f 100644
> --- a/Documentation/md.txt
> +++ b/Documentation/md.txt
> @@ -552,6 +552,16 @@ also have
> =A0 =A0 =A0within the array where IO will be blocked. =A0This is curr=
ently
> =A0 =A0 =A0only supported for raid4/5/6.
>
> + =A0 sync_min
> + =A0 sync_max
> + =A0 =A0 The two values, given as numbers of sectors, indicate a ran=
ge
-+ =A0 =A0 withing the array where 'check'/'repair' will operate. Must =
be
++ within the array where 'check'/'repair' will operate. Must be
> + =A0 =A0 a multiple of chunk_size. When it reaches "sync_max" it wil=
l
> + =A0 =A0 pause, rather than complete.
> + =A0 =A0 You can use 'select' or 'poll' on "sync_completed" to wait =
for
> + =A0 =A0 that number to reach sync_max. =A0Then you can either incre=
ase
> + =A0 =A0 "sync_max", or can write 'idle' to "sync_action".
> +
>
> =A0Each active md device may also have attributes specific to the
> =A0personality module that manages it.
>



--=20
Best regards,
[COOLCOLD-RIPN]
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" i=
n
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Update documentation for sync_min and sync_max entries

am 12.05.2011 01:11:33 von NeilBrown

On Wed, 11 May 2011 21:11:39 +0400 CoolCold wro=
te:

> There is orthographical mistake in patch, in word "withing", should b=
e
> "within". I guess this is too little for one more emailed patch, so
> just notifying & marking line.
>=20
> On Tue, Mar 15, 2011 at 3:30 PM, CoolCold wro=
te:
> > This patch is against 2.6.37 (
> > git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-2 .6.37.y=
git
> > , commit id 97b197f07d15dbcd53328e75754e2f947814c733 ) but I didn't
> > notice any changes in 2.6.38 though.
> >
> > linux/Documentation/md.txt is missing description for sync_min and
> > sync_max entries.
> > This patch adds description for sync_min and sync_max entries.
> >
> > Signed-off-by: Roman Ovchinnikov
> > ---
> > =A0Documentation/md.txt | =A0 10 ++++++++++
> > =A01 files changed, 10 insertions(+), 0 deletions(-)
> >
> > diff --git a/Documentation/md.txt b/Documentation/md.txt
> > index a81c7b4..8f8cb4f 100644
> > --- a/Documentation/md.txt
> > +++ b/Documentation/md.txt
> > @@ -552,6 +552,16 @@ also have
> > =A0 =A0 =A0within the array where IO will be blocked. =A0This is cu=
rrently
> > =A0 =A0 =A0only supported for raid4/5/6.
> >
> > + =A0 sync_min
> > + =A0 sync_max
> > + =A0 =A0 The two values, given as numbers of sectors, indicate a r=
ange
> -+ =A0 =A0 withing the array where 'check'/'repair' will operate. Mus=
t be
> ++ within the array where 'check'/'repair' will operate. Must be
> > + =A0 =A0 a multiple of chunk_size. When it reaches "sync_max" it w=
ill
> > + =A0 =A0 pause, rather than complete.
> > + =A0 =A0 You can use 'select' or 'poll' on "sync_completed" to wai=
t for
> > + =A0 =A0 that number to reach sync_max. =A0Then you can either inc=
rease
> > + =A0 =A0 "sync_max", or can write 'idle' to "sync_action".
> > +
> >
> > =A0Each active md device may also have attributes specific to the
> > =A0personality module that manages it.
> >
>=20
>=20
>=20

Thanks. I have added this to my queue. I might go through the whole f=
ile
and see what else needs fixing before submitting it though.

NeilBrown

--
To unsubscribe from this list: send the line "unsubscribe linux-raid" i=
n
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html