All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ning Yao <zay11022@gmail.com>
To: 徐昕 <xuxinhfut@gmail.com>
Cc: Alexandre DERUMIER <aderumier@odiso.com>,
	ceph-devel <ceph-devel@vger.kernel.org>
Subject: Re: Can i improve the performance of rbd rollback in this way?
Date: Wed, 18 Mar 2015 14:59:45 +0800	[thread overview]
Message-ID: <CALZt5jwGBreRzLANtN2FSdtpqrR-YruqFqwKw4=RGwW4a60fqA@mail.gmail.com> (raw)
In-Reply-To: <CAO7bE7dpG970uZxPA3Bh8jbo=YGJtv6OZuUsrUrP91GPu9XNqQ@mail.gmail.com>

I'm not sure whether the user_version, user_at_version and mtime does
not change for the object. It seems to be ok now. you may wait others
to raise the problem.
Improve if you make snap, and all data in the rbd is not modified,
then you call rollback.

It seems not improve, if you make snap, and some data in the rbd is
modified but some not, then after calling rollback. It seems lots of
objects (which is not modified before) also run clone() and generate
the new object. I do not investigate the reasons, you may try it.


Regards
Ning Yao


2015-03-17 17:13 GMT+08:00 徐昕 <xuxinhfut@gmail.com>:
> Hi Ning,
>
> Do you mean that this change sacrifice the write performance to
> rollback performance. Without considering the write performance, does
> this method exist potential problem that crash ceph? Or it cannot get
> correct result in some corner case?
>
> Thanks
> Xin Xu
>
> 2015-03-17 16:40 GMT+08:00 Ning Yao <zay11022@gmail.com>:
>> 2015-03-17 15:25 GMT+08:00 徐昕 <xuxinhfut@gmail.com>:
>>> Hi Alexandre,
>>>
>>> I have tried this out. It can improve the performance of rbd rollback
>>> greatly when the difference between the image and the sanpshot is
>>> small.
>>>
>> If the clone does not happen in the rollback process, you may consider
>> it would properly happen when it is modified. I may agree, if one
>> object is always not modified since the snapshot N, it may be treated
>> as a cold object so that it is likely not to be modified in the
>> following period.
>>
>>> But i'm not sure if this change may cause potential problem.
>>>
>>> Thanks
>>> Xin Xu
>>>
>>> 2015-03-17 14:43 GMT+08:00 Alexandre DERUMIER <aderumier@odiso.com>:
>>>> Hi,
>>>> I'm not sure it's helping for snapshot rollback, but hammer have a new object_map feature:
>>>>
>>>> https://wiki.ceph.com/Planning/Blueprints/Hammer/librbd%3A_shared_flag,_object_map
>>>>
>>>>
>>>> which help for resize,flatten,...
>>>>
>>>>
>>>> ----- Mail original -----
>>>> De: "徐昕" <xuxinhfut@gmail.com>
>>>> À: "ceph-devel" <ceph-devel@vger.kernel.org>
>>>> Envoyé: Mardi 17 Mars 2015 04:24:28
>>>> Objet: Can i improve the performance of rbd rollback in this way?
>>>>
>>>> Hi,
>>>>
>>>> I am a newbie to ceph. Recently i need to improve the performance of
>>>> rbd rollback on xfs in my project. Through experiment i found that the
>>>> mechanism of rbd rollback is like this:
>>>>
>>>> When a image is rolled back to one of its snapshot (say N), for every
>>>> object that has been allocated,
>>>> case A) if the object has no snap version since snap N, then the head
>>>> version will be copied back to the latest snap (say N+M);
>>>> case B) if the object has snap version since snap N, and it has the
>>>> latest snap version (N+M), then the oldest snap version since snap N
>>>> (say N+x, 0<=x<=M) will be copied to cover the head version;
>>>> case C) if the object has snap version since snap N, and it has no
>>>> version in the latest snap (N+M), then the head version will be copied
>>>> back to the latest snap N+M, and the oldest snap version since snap N
>>>> (say N+x, 0<=x<M) will be copied to cover the head version.
>>>>
>>>> I think in case A, we need not to copied the head version back. So for
>>>> improving performance, i comment the following statement in function
>>>> ReplicatedPG::_rollback_to in osd/ReplicatedPG.cc,
>>>>
>>>> ......
>>>> else if (rollback_to->obs.oi.soid.snap == CEPH_NOSNAP) {
>>>> // rolling back to the head; we just need to clone it.
>>>> // ctx->modify = true;
>>>> }
>>>> ......
>>>>
>>>> Can i improve performance of rollback like this? If CAN NOT, what is the reason?
>>>>
>>>> Thank you.
>>>>
>>>> Xin Xu
>>>> --
>>>> 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
>>> --
>>> 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
--
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:[~2015-03-18  6:59 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-17  3:24 Can i improve the performance of rbd rollback in this way? 徐昕
     [not found] ` <576710578.4735463.1426574609418.JavaMail.zimbra@oxygem.tv>
2015-03-17  6:43   ` Alexandre DERUMIER
2015-03-17  7:25     ` 徐昕
2015-03-17  8:40       ` Ning Yao
2015-03-17  9:13         ` 徐昕
2015-03-18  6:59           ` Ning Yao [this message]

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='CALZt5jwGBreRzLANtN2FSdtpqrR-YruqFqwKw4=RGwW4a60fqA@mail.gmail.com' \
    --to=zay11022@gmail.com \
    --cc=aderumier@odiso.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=xuxinhfut@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.