[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Use direct I/O for loop devices
- To: Demi Marie Obenour <demi@xxxxxxxxxxxxxxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Thu, 18 Aug 2022 09:02:11 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=FrWE1RnCb0Dx7PR3hCHuJ5jBVxaFzwCovy+rvhDN4KI=; b=mkpbKBkxfu2slBEuVM58nRY3+rfxOggMNbXZjmdQ35U3kCP7RWsB80Ca149lHg0nRO70X2zxIYqQltQWDNJiYWkrKJwuXAHAwYGjvl9FPDPR5NRa16a8lB4dyNqoDHjcnY0QXiryWczsfsrWGuqPjN7XA+o7TgVA1tBn7nHviDYVq7K/+Su+1wXSiw/AzrHmJZTo7MT4rX+oyp3UlCuHpGOSQDPCL19+QxPjmplKfbYDgl917W7e1RK11IymCgjqgSAukGVGKrq6O65pSaRhAkHQF59cr9v6iy5MHrNSc+AJd1Q9eKzUMCcj+0lnUgkH9A4pbm8RHvccLeIziYHQfw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=cE8MyhorDAUUPlPj21MKfWhjNKXBbKNIbfMf5DmSbjmWz4OkKT9uVwANThRJIU+xfOtJ/X0kbdTfFWvBXIibA5a6BMnqJoASPiwMSBei6uMIedfpsrA+dJbo+DdungXQVqVfq4JNhF9qoJbWZL/tf/9AZfS+0hSdHNYelqtsU46U3X345Ss2CN7JquOlzyR2Pokok4cNZOwCPAXaL7IQQpfDM9kGL6AYAn7Rwh7hxGrVyrgtOSbyg6I4sxIX6ZXuxg8bRRh46BysZW9fjKNHBV0jy2A2Qlysrpd9lCytnfpwKbqXTiQaEDUXjjiTRSXfoG7SH29V78cssqv5ied0Ww==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: Wei Liu <wl@xxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>, Xen developer discussion <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Thu, 18 Aug 2022 07:02:22 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 17.08.2022 22:46, Demi Marie Obenour wrote:
> This is a huge performance improvement for two reasons:
>
> 1. It uses the filesystem’s asynchronous I/O support, rather than using
> synchronous I/O.
> 2. It bypasses the page cache, removing a redundant layer of caching and
> associated overhead.
> ---
> tools/hotplug/Linux/block | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/hotplug/Linux/block b/tools/hotplug/Linux/block
> index
> 2691b56951c9b82094471a141b9e0bed04abb929..75785f3a5422c4a5f962a4cd4f6acae5080d036d
> 100644
> --- a/tools/hotplug/Linux/block
> +++ b/tools/hotplug/Linux/block
> @@ -330,7 +330,7 @@ mount it read-write in a guest domain."
> else
> roflag=''
> fi
> - do_or_die losetup $roflag "$loopdev" "$file"
> + do_or_die losetup --direct-io=on $roflag "$loopdev" "$file"
I guess you want to first check (maybe in tools/configure) that losetup
actually supports that option. The old-ish one I'm looking at doesn't,
according to its --help output at least.
Jan
|