All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Marczykowski <marmarek@invisiblethingslab.com>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: George Dunlap <George.Dunlap@eu.citrix.com>,
	Jim Fehlig <jfehlig@suse.com>,
	xen-devel <xen-devel@lists.xen.org>,
	Dave Scott <Dave.Scott@eu.citrix.com>,
	Stefano Stabellini <Stefano.Stabellini@eu.citrix.com>
Subject: Re: libvirt, libxl and QDISKs
Date: Fri, 26 Apr 2013 13:50:18 +0200	[thread overview]
Message-ID: <517A69FA.6080402@invisiblethingslab.com> (raw)
In-Reply-To: <1366976452.3142.73.camel@zakaz.uk.xensource.com>


[-- Attachment #1.1: Type: text/plain, Size: 1398 bytes --]

On 26.04.2013 13:40, Ian Campbell wrote:
> On Fri, 2013-04-26 at 12:31 +0100, Marek Marczykowski wrote:
>> What about old good loop+phy based backend for file disk images? I don't want
>> whole qemu in dom0 for PV domains, only for handling simple disk backend.
>> Additionally sparse images + loop + phy + mount -o discard in domU makes the
>> images "auto shrinking". Don't know if qemu is able to do this.
> 
> IIRC the problem with loop+phy is that loop doesn't do O_DIRECT and
> therefore your data isn't actually on the disk when you might think it
> is, which can lead to filesystem corruption even if the f/s is doing
> correct barriers.
> 
>> Attached patch, which I currently use for that. If it is close to something
>> that would be accepted, I will send it in new thread.
> 
> I think you can use a block script for this (i.e. it does the loop
> mount) and avoid patching libxl at all. That's what xend did at least...

This also was solution I've evaluated, but libvirt developers don't want to
hear about disk->script setting in official libvirt, so I still need to patch
some library. Choosing libxl seems to be more universal. Setting loop in libxl
is also much faster than calling script for that.
Perhaps I can rewrite this patch to allow the choice (LIBXL_DISK_BACKEND_LOOP)?

-- 
Best Regards / Pozdrawiam,
Marek Marczykowski
Invisible Things Lab


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 553 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  parent reply	other threads:[~2013-04-26 11:50 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-24 14:22 libvirt, libxl and QDISKs David Scott
2013-04-24 15:55 ` Stefano Stabellini
2013-04-25  8:55 ` Ian Campbell
2013-04-25 10:36   ` George Dunlap
2013-04-25 11:33     ` Ian Campbell
2013-04-25 11:55       ` Stefano Stabellini
2013-04-25 11:57         ` Ian Campbell
2013-04-25 12:12           ` David Scott
2013-04-25 12:43             ` Ian Campbell
2013-04-25 13:22               ` David Scott
2013-04-25 13:56                 ` Ian Campbell
2013-04-26  1:27                   ` Jim Fehlig
2013-04-26  4:50                     ` Jim Fehlig
2013-04-26 10:37                       ` David Scott
2013-04-26 23:44                         ` Jim Fehlig
     [not found]                         ` <517B1170.4020300@suse.com>
2013-04-29  9:41                           ` David Scott
2013-04-26  8:49                     ` Ian Campbell
2013-04-26 10:10                     ` Stefano Stabellini
2013-04-26 10:13                       ` Ian Campbell
2013-04-26 10:28                         ` Stefano Stabellini
2013-04-26 11:31                       ` Marek Marczykowski
2013-04-26 11:40                         ` Ian Campbell
2013-04-26 11:48                           ` Stefano Stabellini
2013-04-26 13:27                             ` Ian Campbell
2013-04-26 17:10                               ` Stefano Stabellini
2013-04-29  8:23                                 ` Ian Campbell
2013-04-29 10:07                                   ` Stefano Stabellini
2013-04-26 11:50                           ` Marek Marczykowski [this message]
2013-04-29 21:00                             ` Jim Fehlig
2013-04-26 11:45                         ` Stefano Stabellini
2013-04-26 14:36                       ` Roger Pau Monné
2013-04-25 18:26         ` Sylvain Munaut

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=517A69FA.6080402@invisiblethingslab.com \
    --to=marmarek@invisiblethingslab.com \
    --cc=Dave.Scott@eu.citrix.com \
    --cc=George.Dunlap@eu.citrix.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=Stefano.Stabellini@eu.citrix.com \
    --cc=jfehlig@suse.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.