On Thu, Sep 03, 2015 at 07:50:07AM -0700, Josh Triplett wrote:
> On Thu, Sep 03, 2015 at 11:16:40AM +0200, Laszlo Ersek wrote:
> > I'm excited because for virtualization we might have the reverse use
> > case: use BITS to test / debug / develop the ACPI generator of QEMU. :)
On Thu, Sep 03, 2015 at 05:53:45PM +0200, Laszlo Ersek wrote:
> On 09/03/15 16:50, Josh Triplett wrote:
> > On Thu, Sep 03, 2015 at 11:16:40AM +0200, Laszlo Ersek wrote:
> >> Then this payload is passed to the guest firmware (SeaBIOS or OVMF) over
> >> "fw_
On Thu, Sep 03, 2015 at 07:19:40PM +0200, Laszlo Ersek wrote:
> On 09/03/15 18:41, j...@joshtriplett.org wrote:
> > On Thu, Sep 03, 2015 at 05:53:45PM +0200, Laszlo Ersek wrote:
> >> On 09/03/15 16:50, Josh Triplett wrote:
>
> >>> Do you virtualize those I/O ports
ould also
take advantage of the ability to directly mmap a file from outside the
VM into the VM's address space.
In any case, we're getting a bit far afield for the original thread. :)
- Josh Triplett
Change the cadence_uart such that tx/rx is enabled on reset. Assuming
both are enabled makes debugging early Linux kernel bootup a little bit
easier.
Signed-off-by: Josh Cartwright
---
I've used this patch mostly for my own testing, but it may be of general
use. On a real system, I
*** This bug is a duplicate of bug 1054558 ***
https://bugs.launchpad.net/bugs/1054558
** This bug has been marked a duplicate of bug 1054558
1366x768 resolution missing
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://
** Bug watch added: Debian Bug tracker #700055
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=700055
** Also affects: debian via
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=700055
Importance: Unknown
Status: Unknown
--
You received this bug notification because you are a m
check back about the status of that patch to make sure it
didn't get lost.
See also the Launchpad bug about this:
https://bugs.launchpad.net/qemu/+bug/1054558
- Josh Triplett
ad
librbd, and explicitly make it resident.
Signed-off-by: Josh Durgin
---
block/Makefile.objs |3 +-
block/rbd.c | 292 ---
block/rbd_types.h | 95 +
configure | 41 +---
4 files changed, 309 ins
On 04/02/2013 07:10 AM, Kevin Wolf wrote:
Am 29.03.2013 um 21:03 hat Josh Durgin geschrieben:
The existing bdrv_co_flush_to_disk implementation uses rbd_flush(),
which is sychronous and causes the main qemu thread to block until it
is complete. This results in unresponsiveness and extra latency
On 04/10/2013 01:09 AM, Stefan Hajnoczi wrote:
On Wed, Apr 10, 2013 at 2:05 AM, Josh Durgin wrote:
This allows the rbd block driver to detect symbols in the installed
version of librbd, and enable or disable features appropriately. This
obviates the #ifdefs regarding librbd versions.
Loading
On 04/11/2013 01:48 AM, Kevin Wolf wrote:
Am 11.04.2013 um 10:02 hat Stefan Hajnoczi geschrieben:
On Wed, Apr 10, 2013 at 07:03:39AM -0700, Josh Durgin wrote:
On 04/02/2013 07:10 AM, Kevin Wolf wrote:
Am 29.03.2013 um 21:03 hat Josh Durgin geschrieben:
The existing bdrv_co_flush_to_disk
On 08/08/2013 05:40 AM, Oliver Francke wrote:
Hi Josh,
I have a session logged with:
debug_ms=1:debug_rbd=20:debug_objectcacher=30
as you requested from Mike, even if I think, we do have another story
here, anyway.
Host-kernel is: 3.10.0-rc7, qemu-client 1.6.0-rc2, client-kernel is
g quickly, so it's a separate bug.
Josh,
In addition to the Ceph logs you can also use QEMU tracing with the
following events enabled:
virtio_blk_handle_write
virtio_blk_handle_read
virtio_blk_rw_complete
See docs/tracing.txt for details on usage.
Inspecting the trace output will let yo
imers, or BHs stay registered when
requests have been drained.
Cc: Josh Durgin
Signed-off-by: Stefan Hajnoczi
---
block/rbd.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index dbc79f4..41f7bdc 100644
--- a/block/rbd.c
+++ b/block/
)
+{
+BDRVRBDState *s = bs->opaque;
+int r = rbd_invalidate_cache(s->image);
+if (r < 0) {
+error_setg_errno(errp, -r, "Failed to invalidate the cache");
+}
+return r;
No return needed here. With that fixed:
Reviewed-by: Josh Durgin
+}
+#endif
+
pports strings as keys, so I had to format the MSR as a hex string
before looking it up.
The third patch in the series provides a bugfix for CPU definition parsing,
which will otherwise add a partial CPU definition (up to the parse failure) to
the list of CPU definitions.
Written on the plane to linux
availability of MSRs and their associated features by handling GPFs.
Signed-off-by: Josh Triplet
---
qemu-config.c |3 +++
target-i386/cpu.h |1 +
target-i386/cpuid.c |9 +
target-i386/op_helper.c | 16
4 files changed, 25 insertions(+), 4
CPU definitions can now define arbitrary additional MSRs, and rdmsr will
support those MSRs and return the corresponding values.
Signed-off-by: Josh Triplett
---
qemu-config.c |3 +++
target-i386/cpu.h |5 +
target-i386/cpuid.c | 45
Without this change, a parse failure would stop the processing of the
cpudef entry, but the partially-parsed CPU definition would still get
added to the list of CPUs.
Signed-off-by: Josh Triplett
---
target-i386/cpuid.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git
This patchset moves the complexity of the rbd format into librbd and
adds truncation support.
Changes since v3:
* trivially rebased
* updated copyright header
Changes since v2:
* return values are checked in rbd_aio_rw_vector
* bdrv_truncate added
Josh Durgin (4):
rbd: use the higher
luster to be used by
specifying different monitor addresses, as well as having different
logging levels or locations for different volumes.
Signed-off-by: Josh Durgin
---
block/rbd.c | 119 ++
1 files changed, 102 insertions(+), 17 deletions(-)
Signed-off-by: Josh Durgin
---
block/rbd.c | 15 +++
1 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index c9f32e4..015ae8e 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -687,6 +687,20 @@ static int64_t qemu_rbd_getlength(BlockDriverState
If scheduling fails, the number of outstanding I/Os must be correct,
or there will be a hang when waiting for everything to be flushed.
Reported-by: Stefan Hajnoczi
Signed-off-by: Josh Durgin
---
block/rbd.c | 24
1 files changed, 20 insertions(+), 4 deletions
librbd stacks on top of librados to provide access
to rbd images.
Using librbd simplifies the qemu code, and allows
qemu to use new versions of the rbd format
with few (if any) changes.
Signed-off-by: Josh Durgin
Signed-off-by: Yehuda Sadeh
---
block/rbd.c | 790
rbd_aio_rw_vector
* bdrv_truncate added
Josh Durgin (4):
rbd: use the higher level librbd instead of just librados
rbd: allow configuration of rados from the rbd filename
rbd: check return values when scheduling aio
rbd: Add bdrv_truncate implementation
block/rbd.c | 897
If scheduling fails, the number of outstanding I/Os must be correct,
or there will be a hang when waiting for everything to be flushed.
Reported-by: Stefan Hajnoczi
Signed-off-by: Josh Durgin
---
block/rbd.c | 24
1 files changed, 20 insertions(+), 4 deletions
luster to be used by
specifying different monitor addresses, as well as having different
logging levels or locations for different volumes.
Signed-off-by: Josh Durgin
---
block/rbd.c | 119 ++
1 files changed, 102 insertions(+), 17 deletions(-)
Signed-off-by: Josh Durgin
---
block/rbd.c | 15 +++
1 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index a44d160..b95b1eb 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -688,6 +688,20 @@ static int64_t qemu_rbd_getlength(BlockDriverState
librbd stacks on top of librados to provide access
to rbd images.
Using librbd simplifies the qemu code, and allows
qemu to use new versions of the rbd format
with few (if any) changes.
Signed-off-by: Josh Durgin
Signed-off-by: Yehuda Sadeh
---
block/rbd.c | 795
On 05/25/2011 12:18 PM, Christian Brunner wrote:
Apart from two cosmetic issues (see below), I think this patch is
ready to replace the old rbd driver. You can add:
Reviewed-by: Christian Brunner
Regards
Christian
Thanks, I fixed these in v5.
Josh
2011/5/24 Josh Durgin:
librbd stacks on
org/>; fix tested the same way, for both 32-bit and
64-bit x86.
Signed-off-by: Josh Triplett
---
op_helper.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff -Naur a/target-i386/op_helper.c b/target-i386/op_helper.c
--- a/target-i386/op_helper.c 2011-02-07 15:13:34.0
On 05/26/2011 01:13 AM, Kevin Wolf wrote:
Am 25.05.2011 22:34, schrieb Josh Durgin:
The new format is rbd:pool/image[@snapshot][:option1=value1[:option2=value2...]]
Each option is used to configure rados, and may be any Ceph option, or "conf".
The "conf" option specifies
On 05/26/2011 01:05 AM, Kevin Wolf wrote:
Am 25.05.2011 22:34, schrieb Josh Durgin:
Signed-off-by: Josh Durgin
---
block/rbd.c | 15 +++
1 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index a44d160..b95b1eb 100644
--- a/block/rbd.c
ristian Brunner
Changes since v3:
* trivially rebased
* updated copyright header
Changes since v2:
* return values are checked in rbd_aio_rw_vector
* bdrv_truncate added
Josh Durgin (4):
rbd: use the higher level librbd instead of just librados
rbd: allow configuration of rados from the rbd
Reviewed-by: Christian Brunner
Signed-off-by: Josh Durgin
---
block/rbd.c | 14 ++
1 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index f4da6ab..bdc448a 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -688,6 +688,19 @@ static int64_t
luster to be used by
specifying different monitor addresses, as well as having different
logging levels or locations for different volumes.
Reviewed-by: Christian Brunner
Signed-off-by: Josh Durgin
---
block/rbd.c | 119 ++
1 files changed, 1
librbd stacks on top of librados to provide access
to rbd images.
Using librbd simplifies the qemu code, and allows
qemu to use new versions of the rbd format
with few (if any) changes.
Reviewed-by: Christian Brunner
Signed-off-by: Josh Durgin
Signed-off-by: Yehuda Sadeh
---
block/rbd.c
If scheduling fails, the number of outstanding I/Os must be correct,
or there will be a hang when waiting for everything to be flushed.
Reviewed-by: Christian Brunner
Reported-by: Stefan Hajnoczi
Signed-off-by: Josh Durgin
---
block/rbd.c | 24
1 files changed, 20
On Thu, May 26, 2011 at 11:12:12AM +0200, Alexander Graf wrote:
> On 26.05.2011, at 11:08, Josh Triplett wrote:
> > qemu currently returns 0 for rdmsr on invalid MSRs, and ignores wrmsr on
> > invalid MSRs. Real x86 processors GPF on invalid MSRs, which allows
> > software
On Fri, May 27, 2011 at 05:16:56PM +0200, Alexander Graf wrote:
>
> On 27.05.2011, at 17:13, Josh Triplett wrote:
>
> > On Thu, May 26, 2011 at 11:12:12AM +0200, Alexander Graf wrote:
> >> On 26.05.2011, at 11:08, Josh Triplett wrote:
> >>> qemu currently retu
On 06/10/2011 01:05 PM, Stefan Weil wrote:
Variable 'snap' is assigned a value that is never used.
Remove snap and the related code.
v2:
The unused variable which was in function rbd_open is now in function
qemu_rbd_create, so the patch needed an update.
Cc: Christian Brunne
On 12/16/2011 12:33 PM, Bruce Rogers wrote:
When migrating a vm using vhost-net we hit the following assertion:
qemu-kvm: /usr/src/packages/BUILD/qemu-kvm-0.15.1/hw/vhost.c:30:
vhost_dev_sync_region: Assertion `start / (0x1000 * (8 *
sizeof(vhost_log_chunk_t)))< dev->log_size' failed.
I consi
We're working on libvirt support for block device authentication [1]. To
authenticate, rbd needs a username and a secret. Normally, to
avoid putting the secret on the command line, you can store the secret
in a file and pass the file to qemu, but when this is automated,
there's no good way to know
On 10/20/2011 12:24 PM, Daniel P. Berrange wrote:
On Thu, Oct 20, 2011 at 11:30:42AM -0700, Josh Durgin wrote:
We're working on libvirt support for block device authentication [1]. To
authenticate, rbd needs a username and a secret. Normally, to
avoid putting the secret on the command line
point,
the size can be refreshed as if the device were resized. This works
fine with the virtio driver, but other drivers, like ide, don't
implement the resize callback. What do you think is the best way to
solve this?
Josh Durgin (2):
block: let bdrv_set_key be used for authentication
We can't connect to the cluster before the key is set, so this step is
moved to qemu_rbd_open_finish. Each bdrv function will fail before the
key is set.
Signed-off-by: Josh Durgin
---
block/rbd.c | 160 +-
1 files changed, 113 inser
Add a BlockDriverState member 'needs_auth' to distinguish between
drives that require authentication and those that are encrypted.
Update the block_passwd monitor command and error strings to be about
generic passwords instead of encryption.
Signed-off-by: Josh Durgin
--
noczi
---
Josh: This patch has not been tested. I have just compiled it.
Sorry for the delay. This is a great simplification, and it passed my
tests. Thanks!
Reviewed- and Tested-by: Josh Durgin
able
releases are 0.67 and 0.72).
Reviewed-by: Josh Durgin
gt;> processor : 3
>> processor : 4
>> processor : 5
>> processor : 6
>> processor : 7
>>
>> so, both of two question's answer is 'yes'.
>> When I runing random write test, the host use 0 swap.
>
> Great. That means the host is not over
On 01/13/2014 10:39 PM, ?? wrote:
Hi Josh
Thanks for your reply.
librbd version : 0.67.5-1
/etc/ceph/ceph.conf config file:
the ip and hostname was hiddened.
[global]
;open auth.
auth cluster required = cephx
auth service required = cephx
auth client
On 01/15/2014 01:40 AM, ?? wrote:
Hi Josh
there is some issues
1. use 'none' cache mode in xml, and unset 'rbd cache=true' in
ceph.conf, the network latency issue not show.
2. use 'writethrough' cache mode in xml, and unset 'rbd cache=true' in
ceph.
On 01/15/2014 10:12 PM, ?? wrote:
Hi Josh
# strings /usr/bin/qemu-system-x86_64 | grep rbd_aio
rbd_aio_write
rbd_aio_flush
rbd_aio_read
rbd_aio_create_completion
rbd_aio_release
rbd_aio_discard
rbd_aio_get_return_value
So, librbd's asynchronous flush is being used.
I set log set
Thanks!
- Josh Triplett
On Fri, May 03, 2013 at 04:41:51PM -0400, John Baboval wrote:
> Josh,
>
> Somehow missed this message going by. Sorry about that.
>
> The 720p support patch (and variable VRAM size) slipped off my
> radar. I'm going to try and get another batch
Could you add the patch:
commit dc7588c1eb3008bda53dde1d6b890cd299758155
Author: Josh Durgin
Date: Fri Mar 29 13:03:23 2013 -0700
rbd: add an asynchronous flush
The existing bdrv_co_flush_to_disk implementation uses rbd_flush(),
which is sychronous and causes the main qemu thread to block u
g into its cache before the
destination starts
4) raw format images have no extra metadata that needs invalidation
If librbd populated its cache when the disk was opened, the rbd driver
would need to implement bdrv_invalidate(), but since it does not, it's
unnecessary.
Is this correct K
00644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -857,3 +857,41 @@ Stable 0.10
L: qemu-sta...@nongnu.org
T: git git://git.qemu-project.org/qemu-stable-0.10.git
S: Orphan
+
+Block drivers
+-
+VMDK
+M: Fam Zheng
+S: Supported
+F: block/vmdk.c
+
+RBD
+M: Josh Durgin
+S: Supported
+F:
rbd already supports snapshots, cloning, thin provisioning, and
differential backup natively, so putting qcow2 on top tends to just add
overhead.
Josh
So use "-o preallocation=falloc or -o preallocation=full", it not a good
solution.
Regards,
Jun Li
I can't say a lot about makin
On 12/09/2014 01:49 AM, Stefan Hajnoczi wrote:
On Tue, Dec 9, 2014 at 3:52 AM, Josh Durgin wrote:
On 12/08/2014 05:58 AM, Jun Li wrote:
On Fri, 12/05 18:01, Max Reitz wrote:
On 2014-12-05 at 16:32, Jun Li wrote:
Currently, qemu-img can not create qcow2 image format on rbd server
n EINVAL from rbd_aio_{read,write,discard} in rbd_start_aio()
to avoid duplicating the check. An out of bounds i/o is the only way
EINVAL can be returned from these functions in librbd.
Josh
o_release(acb);
return NULL;
}
Reviewed-by: Josh Durgin
FWIW this error path can only happen in certain combinations of odd
circumstances, including specific forms of disk corruption of certain
objects in concert with other factors, so it's quite unlikely to occur
in practice.
ey.b64
$QEMU -object secret,id=secret0,file=poolkey.b64,format=base64 \
-drive driver=rbd,filename=rbd:pool/image:id=myname:\
auth_supported=cephx,passwordid=secret0
Signed-off-by: Daniel P. Berrange
Looks good to me, thanks!
Reviewed-by: Josh Durgin
Ping
On 06/10/2015 08:28 PM, Josh Durgin wrote:
Patches 1 and 2 are simple cleanups. 3 and 4 fix the precedence of
cache options and ceph settings. The cache option precedence in
particular is important due to the potential for misconfigurations
(ceph.conf setting rbd_cache=true, qemu setting
On 11/19/2012 12:39 PM, Stefan Priebe wrote:
From: Stefan Priebe
This one fixes a race qemu also had in iscsi block driver between
cancellation and io completition.
qemu_rbd_aio_cancel was not synchronously waiting for the end of
the command.
It also removes the useless cancelled flag and int
On 11/22/2012 02:00 AM, Stefan Priebe wrote:
This one fixes a race which qemu had also in iscsi block driver
between cancellation and io completition.
qemu_rbd_aio_cancel was not synchronously waiting for the end of
the command.
To archieve this it introduces a new status flag which uses
-EINPR
asynchronous version is available.
Reported-by: Oliver Francke
Signed-off-by: Josh Durgin
---
block/rbd.c | 34 --
1 file changed, 32 insertions(+), 2 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index 1a8ea6d..568b279 100644
--- a/block/rbd.c
+++ b/block/rbd.c
asynchronous version is available.
Reported-by: Oliver Francke
Signed-off-by: Josh Durgin
---
v2:
* include hunk treating write, discard, and flush completions
the same, since they have no result data
block/rbd.c | 37 +
1 file changed, 33 insertions(+), 4
asynchronous version is available.
Reported-by: Oliver Francke
Signed-off-by: Josh Durgin
---
Since v2:
* remove #ifdefs around bdrv_aio_flush and qemu_rbd_aio_flush, since
they are dealt with at runtime in the next patch
block/rbd.c | 30 ++
1 file changed, 26
initial version of librbd, so only they need to be checked.
Signed-off-by: Josh Durgin
---
block/rbd.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/block/rbd.c b/block/rbd.c
index 037d82b..69a339a 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -44,6 +44,15
On 04/04/2013 03:10 AM, Kevin Wolf wrote:
Am 04.04.2013 um 10:35 hat Josh Durgin geschrieben:
QEMU may be compiled against a newer version of librbd, but run and
dynamically linked with an older version that does not support these
functions. Declare them as weak symbols so they can be checked
n my machine would die a few mins later.
>
> Of course now I read my fedora kernel emails and notice vhost_net does
> bad things,
>
> disabling vhost_net seems to make it work fine, hopefully the next
> Fedora kernel will bring the magic fixes.
We hope so too. 3.9.9-302.fc19 in koji should contain the fixes we
know about. There's still another bug open that's tracking issues
with Windows guest VMs and vhost-net.
josh
ff-by: Josh Durgin
---
block/rbd.c | 89 --
1 files changed, 73 insertions(+), 16 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index 6cd8448..1280d66 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -44,6 +44,13 @@
* le
Writeback caching was added in Ceph 0.46, and writethrough will be in
0.47. These are controlled by general config options, so there's no
need to check for librbd version.
Signed-off-by: Josh Durgin
---
block/rbd.c | 19 +++
1 files changed, 19 insertions(+), 0 dele
On 05/22/2012 02:18 AM, Paolo Bonzini wrote:
Il 17/05/2012 22:42, Josh Durgin ha scritto:
+ * Fallback to more conservative semantics if setting cache
+ * options fails. Ignore errors from setting rbd_cache because the
+ * only possible error is that the option does not exist, and
On 06/25/2013 04:39 AM, Kevin Wolf wrote:
Can you please review for the gluster, rbd, sheepdog and ssh driver
whether it's safe to assume that the image reads back as zeros after
bdrv_create?
This is always safe for rbd.
On 04/08/2011 01:43 AM, Stefan Hajnoczi wrote:
On Mon, Mar 28, 2011 at 04:15:57PM -0700, Josh Durgin wrote:
librbd stacks on top of librados to provide access
to rbd images.
Using librbd simplifies the qemu code, and allows
qemu to use new versions of the rbd format
with few (if any) changes
On 04/08/2011 01:43 AM, Stefan Hajnoczi wrote:
On Mon, Mar 28, 2011 at 04:15:57PM -0700, Josh Durgin wrote:
librbd stacks on top of librados to provide access
to rbd images.
Using librbd simplifies the qemu code, and allows
qemu to use new versions of the rbd format
with few (if any) changes
On 04/12/2011 01:54 AM, Stefan Hajnoczi wrote:
Is qemu-iotests passing for you now?
Yes, they all pass when 016 is skipped.
Josh
This patchset moves the complexity of the rbd format into librbd and
adds truncation support. This passes qemu-iotests when 016 is
skipped. A patch for this is in a separate email.
Changes since v2:
* return values are checked in rbd_aio_rw_vector
* bdrv_truncate added
Josh Durgin (4):
rbd
rbd is not growable, so test 016 is skipped.
rbd implements bdrv_truncate, so test 025 will work.
Signed-off-by: Josh Durgin
---
016 |4
025 |2 +-
2 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/016 b/016
index 16f151f..8fa921f 100755
--- a/016
+++ b/016
@@ -43,6
Signed-off-by: Josh Durgin
---
block/rbd.c | 15 +++
1 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index ac20282..4514dbd 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -686,6 +686,20 @@ static int64_t qemu_rbd_getlength(BlockDriverState
If scheduling fails, the number of outstanding I/Os must be correct,
or there will be a hang when waiting for everything to be flushed.
Reported-by: Stefan Hajnoczi
Signed-off-by: Josh Durgin
---
block/rbd.c | 24
1 files changed, 20 insertions(+), 4 deletions
luster to be used by
specifying different monitor addresses, as well as having different
logging levels or locations for different volumes.
Signed-off-by: Josh Durgin
---
block/rbd.c | 119 ++
1 files changed, 102 insertions(+), 17 deletions(-)
librbd stacks on top of librados to provide access
to rbd images.
Using librbd simplifies the qemu code, and allows
qemu to use new versions of the rbd format
with few (if any) changes.
Signed-off-by: Josh Durgin
Signed-off-by: Yehuda Sadeh
---
block/rbd.c | 787
On Wed, 13 Apr 2011 04:43:39 +0200, Christoph Hellwig
wrote:
>> @@ -43,6 +43,10 @@ _supported_fmt raw
>> _supported_proto generic
>> _supported_os Linux
>>
>> +# rbd images are not growable
>> +if [ "$IMGPROTO" = "rbd" ]; then
>> +_notrun "image protocol $IMGPROTO does not support growable i
(with Josh Durgin's
patches) against the latest git version of Ceph.
However, this error is still occurring:
terminate called after throwing an instance of 'ceph::buffer::end_of_buffer'
what(): buffer::end_of_buffer
Aborted (core dumped)
Here's another bac
The caller expects psn_tab to be NULL when there are no snapshots or
an error occurs. This results in calling g_free on an invalid address.
Reported-by: Oliver Francke
Signed-off-by: Josh Durgin
---
block/rbd.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/block
On 10/19/2015 08:09 AM, Daniel P. Berrange wrote:
Currently RBD passwords must be provided on the command line
via
$QEMU -drive file=rbd:pool/image:id=myname:\
key=QVFDVm41aE82SHpGQWhBQXEwTkN2OGp0SmNJY0UrSE9CbE1RMUE=:\
auth_supported=cephx
This is insecure because the key is visibl
Patches 1 and 2 are simple cleanups. 3 and 4 fix the precedence of
cache options and ceph settings. The cache option precedence in
particular is important due to the potential for misconfigurations
(ceph.conf setting rbd_cache=true, qemu setting cache=none) to
accidentally put data at risk.
Josh
, although this would just be a performance problem.
Fix this by changing rbd's cache setting to match qemu after all other
ceph settings have been applied.
Signed-off-by: Josh Durgin
---
block/rbd.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/
Apply the ceph settings from a config file before any ceph settings
from the command line. Since the ceph config file location may be
specified on the command line, parse it once to read the config file,
and do a second pass to apply the rest of the command line ceph
options.
Signed-off-by: Josh
RBDAIOCB.status was only used for cancel, which was removed in
7691e24dbebb46658e89b3f950fda6ec78bbb823.
RBDAIOCB.sector_num was never used.
RADOSCB.done and rcbid were never used.
RBD_FD* are obsolete since the pipe was removed in
e04fb07fd1676e9facd7f3f878c1bbe03bccd26b.
Signed-off-by: Josh
The old one still works for now, but will not work indefinitely.
Signed-off-by: Josh Durgin
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 4ed8215..34eeb88 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1135,7 +1135,7 @@ S
On Thu, Sep 03, 2015 at 11:16:40AM +0200, Laszlo Ersek wrote:
> Hi Josh,
>
> I just found the LWN.net article with the title in the subject:
>
> http://lwn.net/SubscriberLink/655992/04701db2bbb7e716/
>
> ("Thank you LWN", of course. :))
>
> This got me
EFAULT_OBJ_ORDER
<https://lxr.missinglinkelectronics.com/qemu+v2.1.0/+code=OBJ_DEFAULT_OBJ_ORDER>,
if someone knows , please tell me. Thanks a lot.
I have CCed the rbd.c maintainer for you:
$ scripts/get_maintainer.pl -f block/rbd.c
Josh Durgin (supporter:RBD)
Thanks for CCing me, I missed it before.
You can s
7;re seeing.
In-flight requests do have buffers and structures allocated for them in
librbd, but these should have lower overhead than cow. If these are the
problem, it seems to me a generic limit on in flight ops in qemu would
be a reasonable fix. Other backends have resources tied up by in-flight
ops as well.
Josh
[1] https://github.com/ceph/ceph/pull/3410
On 08/26/2015 04:47 PM, Andrey Korolyov wrote:
On Thu, Aug 27, 2015 at 2:31 AM, Josh Durgin wrote:
On 08/26/2015 10:10 AM, Andrey Korolyov wrote:
On Thu, May 14, 2015 at 4:42 PM, Andrey Korolyov wrote:
On Wed, Aug 27, 2014 at 9:43 AM, Chris Friesen
wrote:
On 08/25/2014 03:50 PM, Chris
he main culprit here is rbd cache letting itself burst too
large, rather than the bounce buffers.
Andrey, does this still occur with caching off?
Josh
On 07/14/2015 07:20 AM, Kevin Wolf wrote:
Am 09.07.2015 um 11:59 hat Stefan Hajnoczi geschrieben:
On Tue, Jun 30, 2015 at 11:28:18AM -0700, Josh Durgin wrote:
Ping
On 06/10/2015 08:28 PM, Josh Durgin wrote:
Patches 1 and 2 are simple cleanups. 3 and 4 fix the precedence of
cache options and
1 - 100 of 191 matches
Mail list logo
