All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ning Yao <zay11022@gmail.com>
To: "Xinze Chi (信泽)" <xmdxcxz@gmail.com>
Cc: "ceph-devel@vger.kernel.org" <ceph-devel@vger.kernel.org>
Subject: Re: some issue about peering progress
Date: Thu, 2 Nov 2017 00:42:02 +0800	[thread overview]
Message-ID: <CALZt5jzHVy3x28fV8ix8oEU1NRonUbs5G289z=FjUToT-vRUJQ@mail.gmail.com> (raw)
In-Reply-To: <CANE=7sWZ9sBTdqbNZ_UuUZ=K5UJDB=kZbrXh3u09dNaGeUzXRQ@mail.gmail.com>

Can anyone take a look?

If anything will go wrong when divergent occurs and peer_info is reset
in Primary::react(const
MNotifyRec& notevt)?

@Sage  do you think so?
Regards
Ning Yao


2017-10-27 15:45 GMT+08:00 Xinze Chi (信泽) <xmdxcxz@gmail.com>:
> hi, all:
>
>      I confuse about the notify message during peering. Such as:
>
>     epoch 1, primary osd do Pering , GetInfo and GetMissing, it
> calling the func  proc_replica_log. in this func the last_complete and
> last_update maybe reset.
>
>     Before go to Activate. the OSDMap change (the new osdmap do not
> lead to restart peering), the non-primary osd send the notify to
> primary.
>
>     When the primary receive the notify, Primary::react(const
> MNotifyRec& notevt), so it call the func proc_replica_info.
>
>     In the func, we update the pg info including last_complete and
> last_update which modified in proc_replica_log.
>
>     When the primary call the func activate, the primary osd  process
> recovering based on pg info got by notify instead of proc_replica_log.
>
>     so it is a bug?
>
> --
> Regards,
> Xinze Chi
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2017-11-01 16:42 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-27  7:45 some issue about peering progress Xinze Chi (信泽)
2017-11-01 16:42 ` Ning Yao [this message]
2017-11-01 20:26 ` Gregory Farnum
     [not found]   ` <CANE=7sXWMXTpTfgG6NmwxYYyjYA2_UZ3oNun4eAw+QNiht2nkg@mail.gmail.com>
     [not found]     ` <CAJ4mKGaFP+5vePBNzm0fk3pqONirfCyGhrGynZ4BTD3AwfNMuw@mail.gmail.com>
2017-11-02  3:35       ` Xinze Chi (信泽)
2017-11-28 23:35         ` Gregory Farnum

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CALZt5jzHVy3x28fV8ix8oEU1NRonUbs5G289z=FjUToT-vRUJQ@mail.gmail.com' \
    --to=zay11022@gmail.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=xmdxcxz@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.