From: Paolo Bonzini <pbonzini@redhat.com> To: Kevin Wolf <kwolf@redhat.com> Cc: anthony@codemonkey.ws, rusty@rustcorp.com.au, qemu-devel@nongnu.org, kvm@vger.kernel.org, mst@redhat.com Subject: Re: [Qemu-devel] [PATCH 2/2] virtio-blk: disable write cache if not negotiated Date: Wed, 04 Jul 2012 14:21:19 +0200 [thread overview] Message-ID: <4FF4353F.8010809@redhat.com> (raw) In-Reply-To: <4FF417F3.2090400@redhat.com> Il 04/07/2012 12:16, Kevin Wolf ha scritto: >> > Yes. It doesn't override cache=unsafe though. > When the guest doesn't support flushes, cache=writeback is equivalent to > cache=unsafe, so if you want the old behaviour back you can switch to > cache=unsafe without additional risks. > > We don't have a cache=directunsafe, though, so if you want to get the > old behaviour of cache=none back, you're out of luck. Not sure how > acceptable this is. If we want to fix this, let's take the occasion to split the parameters into cache=on/off (well, we have that already), flush=on/off, and a device-side wce=on/off. > Irrespective of this concern I've come to the conclusion that I agree > and we actually must enforce this for non-unsafe mode, and not doing it > is a bug. Thanks! Is that an Acked-by/Reviewed-by? :) Paolo
WARNING: multiple messages have this Message-ID (diff)
From: Paolo Bonzini <pbonzini@redhat.com> To: Kevin Wolf <kwolf@redhat.com> Cc: rusty@rustcorp.com.au, mst@redhat.com, qemu-devel@nongnu.org, anthony@codemonkey.ws, kvm@vger.kernel.org Subject: Re: [Qemu-devel] [PATCH 2/2] virtio-blk: disable write cache if not negotiated Date: Wed, 04 Jul 2012 14:21:19 +0200 [thread overview] Message-ID: <4FF4353F.8010809@redhat.com> (raw) In-Reply-To: <4FF417F3.2090400@redhat.com> Il 04/07/2012 12:16, Kevin Wolf ha scritto: >> > Yes. It doesn't override cache=unsafe though. > When the guest doesn't support flushes, cache=writeback is equivalent to > cache=unsafe, so if you want the old behaviour back you can switch to > cache=unsafe without additional risks. > > We don't have a cache=directunsafe, though, so if you want to get the > old behaviour of cache=none back, you're out of luck. Not sure how > acceptable this is. If we want to fix this, let's take the occasion to split the parameters into cache=on/off (well, we have that already), flush=on/off, and a device-side wce=on/off. > Irrespective of this concern I've come to the conclusion that I agree > and we actually must enforce this for non-unsafe mode, and not doing it > is a bug. Thanks! Is that an Acked-by/Reviewed-by? :) Paolo
next prev parent reply other threads:[~2012-07-04 12:21 UTC|newest] Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-07-03 13:20 [QEMU PATCH 0/2] virtio-blk: writeback cache enable improvements Paolo Bonzini 2012-07-03 13:20 ` [Qemu-devel] " Paolo Bonzini 2012-07-03 13:20 ` [PATCH 1/2] virtio-blk: support VIRTIO_BLK_F_CONFIG_WCE Paolo Bonzini 2012-07-03 13:20 ` [Qemu-devel] " Paolo Bonzini 2012-07-03 13:46 ` Kevin Wolf 2012-07-03 13:46 ` [Qemu-devel] " Kevin Wolf 2012-07-03 13:54 ` Paolo Bonzini 2012-07-03 13:54 ` [Qemu-devel] " Paolo Bonzini 2012-07-03 13:20 ` [PATCH 2/2] virtio-blk: disable write cache if not negotiated Paolo Bonzini 2012-07-03 13:20 ` [Qemu-devel] " Paolo Bonzini 2012-07-03 13:49 ` Kevin Wolf 2012-07-03 13:49 ` [Qemu-devel] " Kevin Wolf 2012-07-03 13:51 ` Paolo Bonzini 2012-07-03 13:51 ` [Qemu-devel] " Paolo Bonzini 2012-07-04 10:16 ` Kevin Wolf 2012-07-04 10:16 ` Kevin Wolf 2012-07-04 12:21 ` Paolo Bonzini [this message] 2012-07-04 12:21 ` Paolo Bonzini 2012-07-04 12:50 ` Kevin Wolf 2012-07-04 12:50 ` [Qemu-devel] " Kevin Wolf 2012-07-04 13:20 ` Paolo Bonzini 2012-07-04 13:20 ` Paolo Bonzini 2012-07-23 16:32 ` [QEMU PATCH 0/2] virtio-blk: writeback cache enable improvements Paolo Bonzini 2012-07-23 16:32 ` [Qemu-devel] " Paolo Bonzini 2012-08-01 15:52 ` Paolo Bonzini 2012-08-01 15:52 ` [Qemu-devel] " Paolo Bonzini 2012-08-01 16:25 ` Kevin Wolf 2012-08-01 16:25 ` [Qemu-devel] " Kevin Wolf
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=4FF4353F.8010809@redhat.com \ --to=pbonzini@redhat.com \ --cc=anthony@codemonkey.ws \ --cc=kvm@vger.kernel.org \ --cc=kwolf@redhat.com \ --cc=mst@redhat.com \ --cc=qemu-devel@nongnu.org \ --cc=rusty@rustcorp.com.au \ /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: linkBe 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.