linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH V1] md:Add place to update ->recovery_cp.
@ 2012-10-29  1:51 kernelmail
  2012-10-31  0:20 ` NeilBrown
  0 siblings, 1 reply; 2+ messages in thread
From: kernelmail @ 2012-10-29  1:51 UTC (permalink / raw)
  To: NeilBrown; +Cc: linux-raid

In resyncing, recovery_cp only updated when resync aborted or completed.
But in md drives,many place used it to judge.So add a place to update.

Signed-off-by: Jianpeng Ma <majianpeng@gmail.com>
---
 drivers/md/md.c |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/md/md.c b/drivers/md/md.c
index 9ab768a..5bd7d93 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -7454,6 +7454,9 @@ void md_do_sync(struct md_thread *thread)
 			wait_event(mddev->recovery_wait,
 				   atomic_read(&mddev->recovery_active) == 0);
 			mddev->curr_resync_completed = j;
+			if (test_bit(MD_RECOVERY_SYNC, &mddev->recovery) &&
+				(j > mddev->recovery_cp))
+				mddev->recovery_cp = j;
 			set_bit(MD_CHANGE_CLEAN, &mddev->flags);
 			sysfs_notify(&mddev->kobj, NULL, "sync_completed");
 		}
-- 
1.7.9.5

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

* Re: [PATCH V1] md:Add place to update ->recovery_cp.
  2012-10-29  1:51 [PATCH V1] md:Add place to update ->recovery_cp kernelmail
@ 2012-10-31  0:20 ` NeilBrown
  0 siblings, 0 replies; 2+ messages in thread
From: NeilBrown @ 2012-10-31  0:20 UTC (permalink / raw)
  To: kedacomkernel; +Cc: linux-raid

[-- Attachment #1: Type: text/plain, Size: 976 bytes --]

On Mon, 29 Oct 2012 09:51:25 +0800 kernelmail <kedacomkernel@gmail.com> wrote:

> In resyncing, recovery_cp only updated when resync aborted or completed.
> But in md drives,many place used it to judge.So add a place to update.
> 
> Signed-off-by: Jianpeng Ma <majianpeng@gmail.com>
> ---
>  drivers/md/md.c |    3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/md/md.c b/drivers/md/md.c
> index 9ab768a..5bd7d93 100644
> --- a/drivers/md/md.c
> +++ b/drivers/md/md.c
> @@ -7454,6 +7454,9 @@ void md_do_sync(struct md_thread *thread)
>  			wait_event(mddev->recovery_wait,
>  				   atomic_read(&mddev->recovery_active) == 0);
>  			mddev->curr_resync_completed = j;
> +			if (test_bit(MD_RECOVERY_SYNC, &mddev->recovery) &&
> +				(j > mddev->recovery_cp))
> +				mddev->recovery_cp = j;
>  			set_bit(MD_CHANGE_CLEAN, &mddev->flags);
>  			sysfs_notify(&mddev->kobj, NULL, "sync_completed");
>  		}

Applied, thanks.

NeilBrown

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

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

end of thread, other threads:[~2012-10-31  0:20 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-10-29  1:51 [PATCH V1] md:Add place to update ->recovery_cp kernelmail
2012-10-31  0:20 ` NeilBrown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).