From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:56811) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QCro5-00065t-Pl for qemu-devel@nongnu.org; Thu, 21 Apr 2011 07:12:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QCro4-0006xR-HJ for qemu-devel@nongnu.org; Thu, 21 Apr 2011 07:12:21 -0400 Received: from mail-vw0-f45.google.com ([209.85.212.45]:63169) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QCro4-0006xH-8L for qemu-devel@nongnu.org; Thu, 21 Apr 2011 07:12:20 -0400 Received: by vws17 with SMTP id 17so1400613vws.4 for ; Thu, 21 Apr 2011 04:12:19 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1303375391-26679-1-git-send-email-ronniesahlberg@gmail.com> <20110421085026.GA4817@lst.de> <20110421090935.GA5381@lst.de> Date: Thu, 21 Apr 2011 21:12:19 +1000 Message-ID: From: ronnie sahlberg Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] iSCSI support for QEMU List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Christoph Hellwig , stefanha@linux.vnet.ibm.com, qemu-devel@nongnu.org On Thu, Apr 21, 2011 at 8:58 PM, Stefan Hajnoczi wrote= : > On Thu, Apr 21, 2011 at 10:28 AM, ronnie sahlberg > wrote: >> On Thu, Apr 21, 2011 at 7:09 PM, Christoph Hellwig wrote: >>> We only claim WCE=3D1 to the guest if cache=3Dwriteback or cache=3Dnone= are >>> set. =A0So ignoring the issue of having a cache on the initiator side >>> you must implement stable writes for the default cache=3Dwritethrough >>> behaviour by either seeting the FUA bit on your writes, or doing >>> a cache flush after every write in case the target does not support FUA= . >> >> My target right now does such flushes for writes. >> >> >> I fail to see why FUA, FUA_NV or flushes have any relevance to a test >> that just involves reading data off the lun. > > I'll try to rephrase what Christoph has pointed out. > > When QEMU is run with cache=3Dwritethrough (default), QEMU does not > report a write cache on the emulated disk. =A0The guest believes that > all writes are stable because there is no disk write cache. =A0Therefore > the guest does not need to issue synchronize cache commands ever. > > In order to meet these semantics with libiscsi, we would need to set > FUA or send a synchronize cache command for every write. =A0(QEMU's > raw-posix.c file I/O meets these semantics by opening the image file > with O_DSYNC when cache=3Dwritethrough.) > >> I do not understand why my target would have data integrity problem >> when used with libiscsi >> but not with open-iscsi mounted lun? > > In the open-iscsi cache=3Dwritethrough case, QEMU's raw-posix.c opens > the file with O_DSYNC. =A0Open-iscsi must set the FUA bit or synchronize > cache for each write request. > > How does libiscsi behave in this case? libiscsi ignores the O_DSYNC flag. It does not matter for two reasons: * my target always destage to disk before replying. I.e. my target ALWAYS write data synchronously to stable storage * this test we are talking about is for READ10, reads, not writes. Serioulsly, please explain, in what exact way are write semantics and FUA bits and write destage policy relevant here : sudo time dd if=3D/dev/sda of=3D/dev/null bs=3D1M I seriously do not understand. Please educate me. regards ronnie sahlberg