Note that I assumed qemu-img runs in a single context (like qemu), and
there are no concurrency issues. If that's not the case, the callback,
error handling need to be fixed.
Yehuda
On Mon, Sep 12, 2011 at 7:26 AM, Yehuda Sadeh wrote:
> In order to improve image conversion process, instead of sy
On Sun, Sep 11, 2011 at 8:17 PM, Yehuda Sadeh Weinraub
wrote:
> On Sun, Sep 11, 2011 at 8:14 PM, Sage Weil wrote:
>> On Fri, 9 Sep 2011, Kevin Wolf wrote:
>>> Am 08.09.2011 18:36, schrieb Sage Weil:
>>> > On Thu, 8 Sep 2011, Kevin Wolf wrote:
>>> >&
On Sun, Sep 11, 2011 at 8:14 PM, Sage Weil wrote:
> On Fri, 9 Sep 2011, Kevin Wolf wrote:
>> Am 08.09.2011 18:36, schrieb Sage Weil:
>> > On Thu, 8 Sep 2011, Kevin Wolf wrote:
>> >> Am 08.09.2011 01:06, schrieb Yehuda Sadeh:
>> >>> The following set of patches improve the qemu-img conversion proce
On Thu, Apr 7, 2011 at 2:54 AM, Yoshiaki Tamura
wrote:
> 2011/4/7 Stefan Hajnoczi :
>> On Thu, Apr 07, 2011 at 10:14:03AM +0900, Yoshiaki Tamura wrote:
>>> 2011/3/29 Josh Durgin :
>>> > The new format is
>>> > rbd:pool/image[@snapshot][:option1=value1[:option2=value2...]]
>>> > Each option is use
On Mon, Dec 6, 2010 at 4:48 AM, Kevin Wolf wrote:
>
>
> What's the easiest way to try it out? I tried to use vstart.sh and copy
> the generated ceph.conf to /etc/ceph/ceph.conf so that qemu-img etc.
> find the monitor address. However, that leads to a hang when I try "rbd
> list" or "./qemu-img cr
On Mon, 2010-11-29 at 11:02 +0100, Kevin Wolf wrote:
> Am 29.11.2010 09:59, schrieb Kevin Wolf:
> > Am 27.11.2010 08:12, schrieb Stefan Hajnoczi:
> >> On Fri, Nov 26, 2010 at 9:59 PM, Christian Brunner
> >> wrote:
> >>> Thanks for the review. What am I supposed to do now?
> >>
> >> Kevin is the bl
On Mon, Nov 15, 2010 at 9:04 AM, Christian Brunner
wrote:
> Hi Stefan,
>
> thanks for your feedback. Yehuda and Sage have already committed some
> pathes to our git repository.
>
> What I'm not sure about is the rados_(de)initialization for multiple
> rbd images. I suspect that _deinitialize shoul
See my comments:
On Wed, Oct 13, 2010 at 1:41 AM, Stefan Hajnoczi wrote:
>> +
>> + cpu_to_le32s(&snap_count);
>> + cpu_to_le64s(&snap_names_len);
Redone all endianity conversions, made it so that it keeps the header
as little endian, and whenever reading the header, do the endianity
conver
See my comments below, updated patch will follow later:
On Tue, Oct 12, 2010 at 3:57 PM, Anthony Liguori wrote:
...
>> +
>> +static int rbd_parsename(const char *filename, char *pool, char **snap,
>> + char *name)
>> +{
>> + const char *rbdname;
>> + char *p;
>> +
On Sat, Oct 9, 2010 at 2:21 AM, Stefan Hajnoczi wrote:
> On Fri, Oct 8, 2010 at 8:00 PM, Yehuda Sadeh wrote:
> No flush operation is supported. Can the guest be sure written data
> is on stable storage when it receives completion?
>
That's part of the consistency that rados provides.
>> +/*
>>
On Fri, Oct 8, 2010 at 7:06 AM, Anthony Liguori wrote:
> On 10/07/2010 05:45 PM, Sage Weil wrote:
>>
>> I'm sorry, I'm having a hard time understanding what it is you're
>> objecting to, or what you would prefer, as there are two different things
>> we're talking about here (callbacks and fd glue/
On Thu, Oct 7, 2010 at 2:04 PM, Anthony Liguori wrote:
> On 10/07/2010 03:47 PM, Yehuda Sadeh Weinraub wrote:
>>>
>>> How is that possible? Are the callbacks delivered in the context of a
>>> different thread? If so, don't you need locking?
>>>
>
On Thu, Oct 7, 2010 at 12:51 PM, Anthony Liguori wrote:
> On 10/07/2010 01:41 PM, Yehuda Sadeh Weinraub wrote:
>>
>> On Thu, Oct 7, 2010 at 11:38 AM, Anthony Liguori
>> wrote:
>>
>>>
>>> On 10/07/2010 01:08 PM, Yehuda Sadeh Weinraub wrote:
>>>
On Thu, Oct 7, 2010 at 11:38 AM, Anthony Liguori wrote:
> On 10/07/2010 01:08 PM, Yehuda Sadeh Weinraub wrote:
>>
>> On Thu, Oct 7, 2010 at 7:12 AM, Anthony Liguori
>> wrote:
>>
>>>
>>> On 08/03/2010 03:14 PM, Christian Brunner wrote:
>>>
&g
On Thu, Oct 7, 2010 at 7:12 AM, Anthony Liguori wrote:
> On 08/03/2010 03:14 PM, Christian Brunner wrote:
>>
>> +#include "qemu-common.h"
>> +#include "qemu-error.h"
>> +#include
>> +#include
>> +
>> +#include
>>
>
> This looks to be unnecessary. Generally, system includes shouldn't be
> required
Following up on this one, I'd like to know whether there is any
pending issue preventing rbd from being included upstream.
Thanks,
Yehuda
On Tue, Aug 3, 2010 at 1:14 PM, Christian Brunner wrote:
> On Tue, Aug 03, 2010 at 12:37:18AM +0400, malc wrote:
>>
>> Thare are whitespace issues in this pat
On Tue, Jul 13, 2010 at 12:23 PM, Christian Brunner wrote:
> On Tue, Jul 13, 2010 at 11:27:03AM -0700, Yehuda Sadeh Weinraub wrote:
>> >
>> > There is another problem with very large i/o requests. I suspect that
>> > this can be triggered only
>> > with qemu
On Sat, Jun 19, 2010 at 8:48 AM, Christian Brunner wrote:
>>
>> Are you going to send a final version which includes Simone's patch or
>> should I apply them as two patches and just accept that rbd is broken
>> after the first one? Or were there any other problems that need to be
>> solved first?
On Sun, May 23, 2010 at 12:59 AM, Blue Swirl wrote:
> On Thu, May 20, 2010 at 11:02 PM, Yehuda Sadeh Weinraub
> wrote:
>> On Thu, May 20, 2010 at 1:31 PM, Blue Swirl wrote:
>>> On Wed, May 19, 2010 at 7:22 PM, Christian Brunner wrote:
>>>> The attache
On Thu, May 20, 2010 at 1:31 PM, Blue Swirl wrote:
> On Wed, May 19, 2010 at 7:22 PM, Christian Brunner wrote:
>> The attached patch is a block driver for the distributed file system
>> Ceph (http://ceph.newdream.net/). This driver uses librados (which
>> is part of the Ceph server) for direct ac
20 matches
Mail list logo
