In order to improve image conversion process, instead of synchronously
writing the destingation image, we keep a window of async writes.
Signed-off-by: Yehuda Sadeh
---
qemu-img.c | 47 +++
1 files changed, 43 insertions(+), 4 deletions(-)
diff
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, in
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 s
In order to improve image conversion process, instead of synchronously
writing the destingation image, we keep a window of async writes.
Signed-off-by: Yehuda Sadeh
---
qemu-img.c | 28 +++-
1 files changed, 23 insertions(+), 5 deletions(-)
diff --git a/qemu-img.c b
When doing convert, we check that the sectors that are written
are not empty. When holes are small, and interleaved with data
it can lead to a significant performance issue.
Signed-off-by: Yehuda Sadeh
---
qemu-img.c |6 --
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a
small enough.
Yehuda Sadeh (2):
qemu-img: async write to block device when converting image
qemu-img: don't skip writing small holes
qemu-img.c | 34 +++---
1 files changed, 27 insertions(+), 7 deletions(-)
--
1.7.5.1
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
844: error: nested extern declaration of
> 'rados_selfmanaged_snap_create'
> make: *** [block/rbd.o] Error 1
>
Right. The CEPH_OSD_TMAP_SET can be fixed and in theory we can get it
compiled without snapshots, but we're not sure that it is really a good
idea at this point
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 th
qemu restrictions. It also supports snapshots.
Signed-off-by: Christian Brunner
Signed-off-by: Yehuda Sadeh
---
Makefile.objs |1 +
block/rbd.c | 977 +
block/rbd_types.h | 71
configure | 31 ++
4 files changed,
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
25 matches
Mail list logo
