From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Wolf Subject: Re: [Qemu-devel] [PATCH 0/4] More RBD updates Date: Mon, 19 Sep 2011 16:09:17 +0200 Message-ID: <4E774D0D.2010406@redhat.com> References: <1316121071-7690-1-git-send-email-sage@newdream.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Return-path: Received: from mx1.redhat.com ([209.132.183.28]:17273 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751328Ab1ISOGV (ORCPT ); Mon, 19 Sep 2011 10:06:21 -0400 In-Reply-To: <1316121071-7690-1-git-send-email-sage@newdream.net> Sender: ceph-devel-owner@vger.kernel.org List-ID: To: Sage Weil Cc: qemu-devel@nongnu.org, ceph-devel@vger.kernel.org Am 15.09.2011 23:11, schrieb Sage Weil: > Hi, > > Here are a few more improvements to the qemu rbd support. The first > patch makes the configuration file handling cleaner (do not error out if > /etc/ceph/ceph.conf doesn't exist). One allows characters in the conf > string to be escaped, so you can (for example) specify an ip\:port (':' > is used as a delimiter). > > The last patch implements flush when rbd_flush() is available. This lets > us take advantage of write buffering in newer versions of librbd, which > improves performance significantly for many workloads (including the > trivial qemu-img convert). > > Thanks! > sage > > > Sage Weil (4): > rbd: ignore failures when reading from default conf location > rbd: allow escaping in config string > rbd: update comment heading > rbd: call flush, if available > > block/rbd.c | 83 +++++++++++++++++++++++++++++++++++++++------------------- > 1 files changed, 56 insertions(+), 27 deletions(-) > Thanks, applied patches 1/3/4, commented on patch 2. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:44623) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1R5eUM-0006Np-3P for qemu-devel@nongnu.org; Mon, 19 Sep 2011 10:06:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1R5eUH-0001ey-Qe for qemu-devel@nongnu.org; Mon, 19 Sep 2011 10:06:26 -0400 Received: from mx1.redhat.com ([209.132.183.28]:26868) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1R5eUH-0001eq-C5 for qemu-devel@nongnu.org; Mon, 19 Sep 2011 10:06:21 -0400 Message-ID: <4E774D0D.2010406@redhat.com> Date: Mon, 19 Sep 2011 16:09:17 +0200 From: Kevin Wolf MIME-Version: 1.0 References: <1316121071-7690-1-git-send-email-sage@newdream.net> In-Reply-To: <1316121071-7690-1-git-send-email-sage@newdream.net> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/4] More RBD updates List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Sage Weil Cc: ceph-devel@vger.kernel.org, qemu-devel@nongnu.org Am 15.09.2011 23:11, schrieb Sage Weil: > Hi, > > Here are a few more improvements to the qemu rbd support. The first > patch makes the configuration file handling cleaner (do not error out if > /etc/ceph/ceph.conf doesn't exist). One allows characters in the conf > string to be escaped, so you can (for example) specify an ip\:port (':' > is used as a delimiter). > > The last patch implements flush when rbd_flush() is available. This lets > us take advantage of write buffering in newer versions of librbd, which > improves performance significantly for many workloads (including the > trivial qemu-img convert). > > Thanks! > sage > > > Sage Weil (4): > rbd: ignore failures when reading from default conf location > rbd: allow escaping in config string > rbd: update comment heading > rbd: call flush, if available > > block/rbd.c | 83 +++++++++++++++++++++++++++++++++++++++------------------- > 1 files changed, 56 insertions(+), 27 deletions(-) > Thanks, applied patches 1/3/4, commented on patch 2. Kevin