From: "Marek Marczykowski-Górecki" <marmarek@invisiblethingslab.com> To: xen-devel@lists.xenproject.org Cc: "Marek Marczykowski-Górecki" <marmarek@invisiblethingslab.com>, "Ian Jackson" <iwj@xenproject.org>, "Wei Liu" <wl@xen.org>, "Anthony PERARD" <anthony.perard@citrix.com> Subject: [RFC PATCH 0/2] libxl: support common cases without block script Date: Tue, 27 Apr 2021 02:22:30 +0200 [thread overview] Message-ID: <cover.3a5d506462133586bd805b72a226916af6a33799.1619482896.git-series.marmarek@invisiblethingslab.com> (raw) This series in an attempt to speed up the domain start by removing slow block script from the picture. The current RFC covers the simplest possible case only - target being a block device directly. This case does not require locking at all. Further version will cover also setting up a loop device. This, compared to the default block script, saves about 0.5s of domain start time, per disk. Similar speedup can be achieved with a trivial lock-less script too. But for file-based disks, it won't be that simple with a script - setting up a loop device lock-less is tricky and ability to keep an FD open and call different ioctls on it greatly helps. Furthermore reusing the same loop device for the same file can be done significantly better with a cache (which can be stored in the libxl hosting process - like xl devd, or libvirt). This surely isn't the only option to improve disk setup time, but is a very atractive one. Few questions: 1. Is it acceptable approach at all? 2. Is empty 'script' parameter value going to fly? Unfortunately, NULL is already taken as "use default". Marek Marczykowski-Górecki (2): libxl: rename 'error' label to 'out' as it is used for success too libxl: allow to skip block script completely docs/man/xl-disk-configuration.5.pod.in | 4 ++- tools/libs/light/libxl_disk.c | 7 ++- tools/libs/light/libxl_linux.c | 68 ++++++++++++++++++++++++-- 3 files changed, 74 insertions(+), 5 deletions(-) base-commit: bea65a212c0581520203b6ad0d07615693f42f73 -- git-series 0.9.1
next reply other threads:[~2021-04-27 0:24 UTC|newest] Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-04-27 0:22 Marek Marczykowski-Górecki [this message] 2021-04-27 0:22 ` [RFC PATCH 1/2] libxl: rename 'error' label to 'out' as it is used for success too Marek Marczykowski-Górecki 2021-04-27 0:22 ` [RFC PATCH 2/2] libxl: allow to skip block script completely Marek Marczykowski-Górecki 2021-04-28 6:48 ` [RFC PATCH 0/2] libxl: support common cases without block script Demi Marie Obenour 2021-04-28 12:26 ` Jason Andryuk
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=cover.3a5d506462133586bd805b72a226916af6a33799.1619482896.git-series.marmarek@invisiblethingslab.com \ --to=marmarek@invisiblethingslab.com \ --cc=anthony.perard@citrix.com \ --cc=iwj@xenproject.org \ --cc=wl@xen.org \ --cc=xen-devel@lists.xenproject.org \ --subject='Re: [RFC PATCH 0/2] libxl: support common cases without block script' \ /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
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).