From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:39050) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UVQ2Z-0005uj-OJ for qemu-devel@nongnu.org; Thu, 25 Apr 2013 13:33:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UVQ2V-0002l7-1L for qemu-devel@nongnu.org; Thu, 25 Apr 2013 13:33:03 -0400 Date: Thu, 25 Apr 2013 19:32:52 +0200 From: Kevin Wolf Message-ID: <20130425173252.GD2486@dhcp-200-207.str.redhat.com> References: <5179293F.5010708@filoo.de> <20130425135142.GE3156@dhcp-200-207.str.redhat.com> <51793A17.5020303@filoo.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [Qemu-stable] Something broken with ceph-rbd images since couple of days ( after block-changes) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Priebe - Profihost AG Cc: Oliver Francke , "qemu@nongnu.org" , "qemu-stable@nongnu.org" Am 25.04.2013 um 19:25 hat Stefan Priebe - Profihost AG geschrieben: > Ist this qemu 1.4.1? Did you received the patch? >=20 > Stefan No, this is actually about git master. Not sure why this was directed to qemu-stable. Kevin > Am 25.04.2013 um 16:13 schrieb Oliver Francke = : >=20 > > Cool, > >=20 > > thnx for the quick response... Not a C-coder any more, only wild-not-= too-bad guessing ;) > >=20 > > Oliver. > >=20 > > On 04/25/2013 03:51 PM, Kevin Wolf wrote: > >> Am 25.04.2013 um 15:01 hat Oliver Francke geschrieben: > >>> Hi, > >>>=20 > >>> just encountered a bug while playing with latest aio_flush option i= n > >>> ceph + qemu handling. > >>>=20 > >>> I got a "invalid argument" even with "qemu-img info rbd:pool/image.= rbd". > >>>=20 > >>> Perhaps qemu_rbd_open is called with wrong number of params > >>> somewhere else after expanding params with "QDict *options" in > >>> block/rbd.c line around line 444 with current master. > >>> ( talking of qemu-54f106d) > >>>=20 > >>> Just as a quick note ;) > >> Thanks for the report, Oliver. The problem is the qemu_opts_del() in > >> line 481, which is too early. This frees the filename already when i= t's > >> still needed for the parsing. > >>=20 > >> I'm copying qemu-devel and Josh, just so everyone is informed. I'll = try > >> to send a patch myself in a few minutes. > >>=20 > >> Kevin > >=20 > >=20 > > --=20 > >=20 > > Oliver Francke > >=20 > > filoo GmbH > > Moltkestra=DFe 25a > > 33330 G=FCtersloh > > HRB4355 AG G=FCtersloh > >=20 > > Gesch=E4ftsf=FChrer: S.Grewing | J.Rehp=F6hler | C.Kunz > >=20 > > Folgen Sie uns auf Twitter: http://twitter.com/filoogmbh > >=20 > >=20