From: Jim Fehlig <jfehlig@suse.com>
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: andrew.cooper3@citrix.com, Olaf Hering <olaf@aepfle.de>,
Ian Jackson <Ian.Jackson@eu.citrix.com>,
Ian Campbell <Ian.Campbell@citrix.com>,
xen-devel@lists.xen.org
Subject: Re: new knob to tweak caching mode for backends
Date: Tue, 27 May 2014 14:18:26 -0600 [thread overview]
Message-ID: <5384F312.20109@suse.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1405271806050.4779@kaball.uk.xensource.com>
Stefano Stabellini wrote:
> On Tue, 27 May 2014, Jim Fehlig wrote:
>
>> Olaf Hering wrote:
>>
>>> Currently libxl (and xend) has no knob to control cache mode of backend
>>> driver for block devices. libvirt and qemu have:
>>> cache=off|none|directsync|writeback|unsafe|writethrough.
>>>
>>> The xen qdisk driver in qemu defaults to "writeback". If the diskspec in
>>> domU…cfg has 'direct-io-safe' then qdisk will default to directsync AIO.
>>>
>>> I think these defaults are fine as they provide some sort of data
>>> integrity.
>>>
>>>
>>> But there is one issue: all the flushing thats going on during guest
>>> triggered writes does slows down the guest. There should be a knob to
>>> skip the regular flushes on the host. If the given backing file will
>>> contain throw-away data its up the the admin to make this decision.
>>>
>>>
>> ACK to the idea of admin control of cache mode. As you say, admins
>> already enjoy this capability with libvirt and qemu.
>>
>
> There is nothing wrong with having advanced options for people that know
> what they are doing and/or they are happy with taking risks and loosing
> data. What's important is not to rely on these "advanced options" to get
> a decent default configuration.
>
Agreed. The current default, which is used by qemu too, is the best
choice IMO.
> So I think that as long as we keep the default behaviour sane, we can
> expose whatever advanced options we like.
Cool. So in libvirt terms, cache not specified or cache='default' would
yield the same behavior as today.
Regards,
Jim
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2014-05-27 20:18 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-26 17:06 new knob to tweak caching mode for backends Olaf Hering
2014-05-27 16:50 ` Jim Fehlig
2014-05-27 17:08 ` Stefano Stabellini
2014-05-27 20:18 ` Jim Fehlig [this message]
2014-05-28 8:53 ` Olaf Hering
2014-05-28 11:26 ` Stefano Stabellini
2014-06-10 13:34 ` Ian Jackson
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=5384F312.20109@suse.com \
--to=jfehlig@suse.com \
--cc=Ian.Campbell@citrix.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=olaf@aepfle.de \
--cc=stefano.stabellini@eu.citrix.com \
--cc=xen-devel@lists.xen.org \
/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.