On Tue, Dec 06, 2016 at 02:13:17PM +0800, JackieLiu wrote:
> When create the super-block information, We do not need to do this
> recovery stage, only need to initialize some variables.
This makes sense.
please do look at Documentation/SubmittingPatches. I can't apply patch without
'signed-off-by'.
> ---
> drivers/md/raid5-cache.c | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/md/raid5-cache.c b/drivers/md/raid5-cache.c
> index c3b3124..96f25df 100644
> --- a/drivers/md/raid5-cache.c
> +++ b/drivers/md/raid5-cache.c
> @@ -2545,7 +2545,13 @@ static int r5l_load_log(struct r5l_log *log)
>
> __free_page(page);
>
> - ret = r5l_recovery_log(log);
> + if (create_super) {
> + log->log_start = r5l_ring_add(log, cp, BLOCK_SECTORS);
> + log->seq = log->last_cp_seq + 1;
> + log->next_checkpoint = cp;
must set ret = 0 here. Next time please make sure there isn't compiling
warnning.
Thanks,
Shaohua
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
