On 2015-04-09 17:05, Stefan Hajnoczi wrote:
On Thu, Apr 9, 2015 at 1:48 PM, Chrysostomos Nanakos
wrote:
On 2015-04-09 06:48, Andreas Färber wrote:
Am 08.08.2014 um 19:39 schrieb Kevin Wolf:
From: Chrysostomos Nanakos
VM Image on Archipelago volume is specified like this:
file.driver
On 2015-04-09 06:48, Andreas Färber wrote:
Am 08.08.2014 um 19:39 schrieb Kevin Wolf:
From: Chrysostomos Nanakos
VM Image on Archipelago volume is specified like this:
file.driver=archipelago,file.volume=[,file.mport=[,
file.vport=][,file.segment=]]
'archipelago' is the protoco
ver / assist with the following areas:
>
> * blockjobs
> * archipelago
> * curl
> * gluster
> * nfs
> * rbd
> * sheepdog
> * ssh
> * vhdx
This patch should also CC Josh Durgin, Hitoshi Mitake, Liu Yuan,
Peter
Lieven, Richard W.M
Remove first email address and let the one from which I am contributing.
Signed-off-by: Chrysostomos Nanakos
---
MAINTAINERS |1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index bcb69e8..a33d8f7 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1089,7 +1089,6 @@ S
Remove first email address and let the one from which I am contributing.
Chrysostomos Nanakos (1):
MAINTAINERS: Update email addresses for Chrysostomos Nanakos
MAINTAINERS |1 -
1 file changed, 1 deletion(-)
--
1.7.10.4
initialize event notifier: Too many open files in system
Signed-off-by: Chrysostomos Nanakos
---
async.c | 16 +++-
include/block/aio.h |2 +-
include/qemu/main-loop.h |2 +-
iothread.c | 11 ++-
main-loop.c |9
ix code style
issues.
v1->v2
--
* aio_context_new() returns NULL if the initialization of event notifier fails.
* Add descriptive error messages if aio_context_new() and event_notifier_init()
fail.
* Fix gpollfds leak.
Chrysostomos Nanakos (1):
async: aio_context_new(): Handle ev
alization of event notifier fails.
* Add descriptive error messages if aio_context_new() and event_notifier_init()
fail.
* Fix gpollfds leak.
Chrysostomos Nanakos (1):
async: aio_context_new(): Handle event_notifier_init failure
async.c | 16 +++-
include
initialize event notifier: Too many open files in system
Signed-off-by: Chrysostomos Nanakos
---
async.c | 16 +++-
include/block/aio.h |2 +-
include/qemu/main-loop.h |2 +-
iothread.c | 11 ++-
main-loop.c |9
On 09/17/2014 04:01 PM, Eric Blake wrote:
On 09/17/2014 03:16 AM, Chrysostomos Nanakos wrote:
On 09/16/2014 10:40 PM, Eric Blake wrote:
On 09/16/2014 12:04 PM, Chrysostomos Nanakos wrote:
If event_notifier_init fails QEMU exits without printing
any error information to the user. This commit
On 09/16/2014 10:40 PM, Eric Blake wrote:
On 09/16/2014 12:04 PM, Chrysostomos Nanakos wrote:
If event_notifier_init fails QEMU exits without printing
any error information to the user. This commit adds an error
message on failure:
# qemu [...]
Showing the actual command line you used would
If event_notifier_init fails QEMU exits without printing
any error information to the user. This commit adds an error
message on failure:
# qemu [...]
qemu: Failed to initialize event notifier: Too many open files in system
Signed-off-by: Chrysostomos Nanakos
---
async.c
new() and event_notifier_init()
fail.
* Fix gpollfds leak.
Chrysostomos Nanakos (1):
async: aio_context_new(): Handle event_notifier_init failure
async.c | 16 +++-
include/block/aio.h |2 +-
include/qemu/main-loop.h |2 +-
iothread.c
On Tue, Sep 16, 2014 at 06:50:38PM +0300, Chrysostomos Nanakos wrote:
> On Tue, Sep 16, 2014 at 05:45:16PM +0200, Paolo Bonzini wrote:
> > Il 16/09/2014 17:43, Chrysostomos Nanakos ha scritto:
> > >> > error_propagate(errp, local_error);
> > >> >
On Tue, Sep 16, 2014 at 05:45:16PM +0200, Paolo Bonzini wrote:
> Il 16/09/2014 17:43, Chrysostomos Nanakos ha scritto:
> >> > error_propagate(errp, local_error);
> >> > return;
> > Just to note that after propagating the error and returning, QEMU fails
On Tue, Sep 16, 2014 at 03:57:25PM +0200, Paolo Bonzini wrote:
> Il 16/09/2014 15:53, Benoît Canet ha scritto:
> >> static void iothread_complete(UserCreatable *obj, Error **errp)
> >> > {
> >> > +Error *local_error = NULL;
> >> > IOThread *iothread = IOTHREAD(obj);
> >> >
> >> >
Chrysostomos Nanakos (1):
block/archipelago: Fix typo in qemu_archipelago_truncate()
block/archipelago.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--
1.7.10.4
Fix a typo introduced by 94c80a438c85f2c19698547fbb115ea46d80c5f1
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block/archipelago.c b/block/archipelago.c
index 93fb7c0..f4982fa 100644
--- a/block/archipelago.c
If event_notifier_init fails QEMU exits without printing
any error information to the user. This commit adds an error
message on failure:
# qemu [...]
qemu: Failed to initialize event notifier: Too many open files in system
qemu: qemu_init_main_loop failed
Signed-off-by: Chrysostomos Nanakos
v1->v2
--
* aio_context_new() returns NULL if the initialization of event notifier fails.
* Add descriptive error messages if aio_context_new() and event_notifier_init()
fail.
* Fix gpollfds leak.
Chrysostomos Nanakos (1):
async: aio_context_new(): Handle event_notifier_init fail
On Mon, Sep 15, 2014 at 09:59:00PM +0200, Benoît Canet wrote:
> The Monday 15 Sep 2014 à 21:03:18 (+0200), Benoît Canet wrote :
> > The Sunday 14 Sep 2014 à 13:23:13 (+0300), Chrysostomos Nanakos wrote :
> > > If event_notifier_init fails QEMU exits without printing
> > &g
Hi Benoit,
thanks for reviewing and replying.
On Mon, Sep 15, 2014 at 09:03:18PM +0200, Benoît Canet wrote:
> The Sunday 14 Sep 2014 à 13:23:13 (+0300), Chrysostomos Nanakos wrote :
> > If event_notifier_init fails QEMU exits without printing
> > any error information to the us
If event_notifier_init fails QEMU exits without printing
any error information to the user. This commit adds an error
message on failure:
# qemu [...]
qemu: event_notifier_init failed: Too many open files in system
qemu: qemu_init_main_loop failed
Signed-off-by: Chrysostomos Nanakos
Chrysostomos Nanakos (1):
async: aio_context_new(): Handle event_notifier_init failure
async.c | 19 +--
include/block/aio.h |2 +-
include/qemu/main-loop.h |2 +-
iothread.c | 12 +++-
main-loop.c | 11
On 09/10/2014 11:20 AM, Kevin Wolf wrote:
Am 09.09.2014 um 19:38 hat Chrysostomos Nanakos geschrieben:
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 63 +--
1 file changed, 61 insertions(+), 2 deletions(-)
Thanks, applied
Run resize grow test to ensure that existing data
is not lost during grow and new space is zeroed.
Signed-off-by: Chrysostomos Nanakos
---
tests/qemu-iotests/025 |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/qemu-iotests/025 b/tests/qemu-iotests/025
index a5f45b4
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 63 +--
1 file changed, 61 insertions(+), 2 deletions(-)
diff --git a/block/archipelago.c b/block/archipelago.c
index 22a7daa..40e5f76 100644
--- a/block/archipelago.c
+++ b/block
Replace __sync builtins with ones provided by QEMU
for atomic operations.
Special thanks goes to Paolo Bonzini for his refactoring
suggestion in order to use the already existing atomic builtins
interface.
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 76
On 09/01/2014 01:33 PM, Paolo Bonzini wrote:
Il 01/09/2014 12:13, Chrysostomos Nanakos ha scritto:
-if ((__sync_add_and_fetch(&segreq->ref, -segments_nr + i))
== 0) {
+if ((atomic_add_fetch(&segreq->ref, -segments_nr + i)) == 0) {
What about this one? It seems e
On 09/01/2014 12:43 PM, Paolo Bonzini wrote:
Il 01/09/2014 10:58, Chrysostomos Nanakos ha scritto:
Replace __sync builtins with the ones provided by QEMU
for atomic operations.
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 11 ++-
1 file changed, 6 insertions
Add __sync_*_and_fetch builtins used in several places.
Signed-off-by: Chrysostomos Nanakos
---
include/qemu/atomic.h |4
1 file changed, 4 insertions(+)
diff --git a/include/qemu/atomic.h b/include/qemu/atomic.h
index 492bce1..48fc283 100644
--- a/include/qemu/atomic.h
+++ b/include
Replace __sync builtins with the ones provided by QEMU
for atomic operations.
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/block/archipelago.c b/block/archipelago.c
index 34f72dc..fa8cd29 100644
Hello,
On Fri, Aug 29, 2014 at 04:58:20PM -0600, Cam Macdonell wrote:
> Hello,
>
> A bug was reported to me regarding mmaping of BAR0 in ivshmem. Indeed the
> mmap fails. This bug will affect those using the ivshmem server as BAR0
> contains the registers for sending and receiving interrupts.
ile.vport=1234
file.driver=archipelago,file.volume=my_vm_volume,file.mport=123,
file.vport=1234,file.segment=my_segment
Signed-off-by: Chrysostomos Nanakos
---
MAINTAINERS |6 +
block/Makefile.objs |2 +
block/archipelago.c | 787 ++
e name of the shared memory segment
Archipelago stack is using. This is optional
and if not specified, QEMU will make Archipelago
use the default value, 'archipelago'.
Signed-off-by: Chrysostomos Nanakos
--
:segment=my_segment
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 140 ++-
1 file changed, 138 insertions(+), 2 deletions(-)
diff --git a/block/archipelago.c b/block/archipelago.c
index 1c21d36..5a9fc68 100644
--- a/block
Signed-off-by: Chrysostomos Nanakos
---
tests/qemu-iotests/common|6 ++
tests/qemu-iotests/common.rc |9 -
2 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/tests/qemu-iotests/common b/tests/qemu-iotests/common
index e4083f4..70df659 100644
--- a/tests/qemu
ile.vport=1234
file.driver=archipelago,file.volume=my_vm_volume,file.mport=123,
file.vport=1234,file.segment=my_segment
Signed-off-by: Chrysostomos Nanakos
---
MAINTAINERS |6 +
block/Makefile.objs |2 +
block/archipelago.c | 785 ++
e name of the shared memory segment
Archipelago stack is using. This is optional
and if not specified, QEMU will make Archipelago
use the default value, 'archipelago'.
Signed-off-by: Chrysostomos Nanakos
--
qemu-img archipelago:[/mport=[:vport=]
[:segment=]] [size]
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 146 +++
1 file changed, 146 insertions(+)
diff --git a/block/archipelago.c b/block/archipelago.c
index 5a9fc68..b5c66fd
mpleted in
qemu_archipelago_complete_aio() instead.
- Resolve proposed changes from Kevin Wolf and miscellaneous style issues.
Chrysostomos Nanakos (5):
block: Support Archipelago as a QEMU block backend
block/archipelago: Implement bdrv_parse_filename()
block/archipelago: Add support fo
On 07/10/2014 03:23 AM, Jeff Cody wrote:
On Fri, Jun 27, 2014 at 11:24:08AM +0300, Chrysostomos Nanakos wrote:
VM Image on Archipelago volume is specified like this:
file.driver=archipelago,file.volume=[,file.mport=[,
file.vport=][,file.segment=]]
'archipelago' is the protocol.
On 07/10/2014 01:04 PM, Chrysostomos Nanakos wrote:
On 07/10/2014 03:23 AM, Jeff Cody wrote:
On Fri, Jun 27, 2014 at 11:24:08AM +0300, Chrysostomos Nanakos wrote:
VM Image on Archipelago volume is specified like this:
file.driver=archipelago,file.volume=[,file.mport=[,
file.vport
On 07/02/2014 04:59 PM, Eric Blake wrote:
On 06/27/2014 02:24 AM, Chrysostomos Nanakos wrote:
VM Image on Archipelago volume is specified like this:
file.driver=archipelago,file.volume=[,file.mport=[,
file.vport=][,file.segment=]]
'archipelago' is the protocol.
'mport' i
On 07/02/2014 04:58 PM, Eric Blake wrote:
On 06/27/2014 02:24 AM, Chrysostomos Nanakos wrote:
Introduce new enum BlockdevOptionsArchipelago.
@volume: #Name of the Archipelago volume image
@mport: #'mport' is the port number on which
On 07/02/2014 05:01 PM, Eric Blake wrote:
On 06/27/2014 02:24 AM, Chrysostomos Nanakos wrote:
qemu-img archipelago:[/mport=[:vport=]
[:segment=]] [size]
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 149 +++
1 file changed
Signed-off-by: Chrysostomos Nanakos
---
tests/qemu-iotests/common|6 ++
tests/qemu-iotests/common.rc |9 -
2 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/tests/qemu-iotests/common b/tests/qemu-iotests/common
index 0aaf84d..a0e35c4 100644
--- a/tests/qemu
qemu-img archipelago:[/mport=[:vport=]
[:segment=]] [size]
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 149 +++
1 file changed, 149 insertions(+)
diff --git a/block/archipelago.c b/block/archipelago.c
index 3549454..3d5aff1
e name of the shared memory segment
Archipelago stack is using. This is optional
and if not specified, QEMU will make Archipelago
use the default value, 'archipelago'.
Signed-off-by: Chrysostomos Nanakos
--
:segment=my_segment
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 139 ++-
1 file changed, 137 insertions(+), 2 deletions(-)
diff --git a/block/archipelago.c b/block/archipelago.c
index c56826a..3549454 100644
--- a/block
ile.vport=1234
file.driver=archipelago,file.volume=my_vm_volume,file.mport=123,
file.vport=1234,file.segment=my_segment
Signed-off-by: Chrysostomos Nanakos
---
MAINTAINERS |6 +
block/Makefile.objs |2 +
block/archipelago.c | 819 ++
a directly
to the destination buffer.
- Remove archipelago_aio_bh_cb() function, a full request is completed in
qemu_archipelago_complete_aio() instead.
- Resolve proposed changes from Kevin Wolf and miscellaneous style issues.
Chrysostomos Nanakos (5):
block: Support Archipelago as a
On Thu, Jun 26, 2014 at 09:06:03AM -0600, Eric Blake wrote:
> On 06/23/2014 06:32 AM, Chrysostomos Nanakos wrote:
> > VM Image on Archipelago volume is specified like this:
> >
> > file=archipelago:[/mport=[:vport=][:
> > segment=]]
> >
> > or
>
Signed-off-by: Chrysostomos Nanakos
---
tests/qemu-iotests/common|6 ++
tests/qemu-iotests/common.rc |9 -
2 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/tests/qemu-iotests/common b/tests/qemu-iotests/common
index 0aaf84d..a0e35c4 100644
--- a/tests/qemu
e name of the shared memory segment
Archipelago stack is using. This is optional
and if not specified, QEMU will make Archipelago
use the default value, 'archipelago'.
Signed-off-by: Chrysostomos Nanakos
--
Signed-off-by: Chrysostomos Nanakos
---
tests/qemu-iotests/common|6 ++
tests/qemu-iotests/common.rc |8 +++-
2 files changed, 13 insertions(+), 1 deletion(-)
diff --git a/tests/qemu-iotests/common b/tests/qemu-iotests/common
index 0aaf84d..a0e35c4 100644
--- a/tests/qemu
() instead.
- Resolve proposed changes from Kevin Wolf and miscellaneous style issues.
Chrysostomos Nanakos (3):
block: Support Archipelago as a QEMU block backend
block/archipelago: Add support for creating images
QMP: Add support for Archipelago
MAINTAINERS |6 +
block/Makefile.ob
olume=my_vm_volume,file.mport=123,
file.vport=1234,file.segment=my_segment
Signed-off-by: Chrysostomos Nanakos
---
MAINTAINERS |6 +
block/Makefile.objs |2 +
block/archipelago.c | 912 +++
configure | 40 +++
4 files ch
qemu-img archipelago:[/mport=[:vport=]
[:segment=]] [size]
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 146 +++
1 file changed, 146 insertions(+)
diff --git a/block/archipelago.c b/block/archipelago.c
index c0810ef..d382667
On 06/20/2014 05:33 PM, Stefan Hajnoczi wrote:
On Thu, Jun 19, 2014 at 05:48:46PM +0300, Chrysostomos Nanakos wrote:
+typedef struct BDRVArchipelagoState {
+int fds[2];
+int qemu_aio_count;
This field is never used. It's increment and decremented but nothing
ever checks the value
qemu-img archipelago:[/mport=[:vport=]]
[size]
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 137 +++
1 file changed, 137 insertions(+)
diff --git a/block/archipelago.c b/block/archipelago.c
index 3da7a1c..4553b2d 100644
--- a
go:my_vm_volume/mport=123
file=archipelago:my_vm_volume/mport=123:vport=1234
or
file.driver=archipelago,file.volume=my_vm_volume
file.driver=archipelago,file.volume=my_vm_volume,file.mport=123
file.driver=archipelago,file.volume=my_vm_volume,file.mport=123,
file.vport=1234
Signed-off-by: Chrysostomos Nanakos
-
() instead.
- Resolve proposed changes from Kevin Wolf and miscellaneous style issues.
Chrysostomos Nanakos (3):
block: Support Archipelago as a QEMU block backend
block/archipelago: Add support for creating images
QMP: Add support for Archipelago
MAINTAINERS |6 +
block
will make Archipelago to use the default port.
@vport: #'vport' is the port number on which vlmcd is
listening. This is optional and if not specified,
QEMU will make Archipelago to use the default port.
Signed-off-by: Chrysostomos Nana
On 06/04/2014 03:56 PM, Eric Blake wrote:
On 06/04/2014 05:30 AM, Chrysostomos Nanakos wrote:
Introduce new enum BlockdevOptionsArchipelago.
@volume: #Name of the Archipelago volume image
@mport: #'mport' is the port number on which
On 06/04/2014 02:38 PM, Kevin Wolf wrote:
Am 04.06.2014 um 13:30 hat Chrysostomos Nanakos geschrieben:
Introduce new enum BlockdevOptionsArchipelago.
@volume: #Name of the Archipelago volume image
@mport: #'mport' is the port number on which
Hi,
On 06/02/2014 06:31 PM, Kevin Wolf wrote:
Am 02.06.2014 um 17:04 hat Chrysostomos Nanakos geschrieben:
Hi,
On 06/02/2014 05:30 PM, Eric Blake wrote:
On 06/02/2014 08:15 AM, Chrysostomos Nanakos wrote:
VM Image on Archipelago volume is specified like this:
file=archipelago:[/mport
qemu_archipelago_complete_aio() instead.
- Resolve proposed changes from Kevin Wolf and miscellaneous style issues.
Chrysostomos Nanakos (3):
block: Support Archipelago as a QEMU block backend
block/archipelago: Add support for creating images
QMP: Add support for Archipelago
MAINTAINERS |5
will make Archipelago to use the default port.
@vport: #'vport' is the port number on which vlmcd is
listening. This is optional and if not specified,
QEMU will make Archipelago to use the default port.
Signed-off-by: Chrysostomo
qemu-img archipelago:[/mport=[:vport=]]
[size]
Signed-off-by: Chrysostomos Nanakos
---
block/archipelago.c | 136 +++
1 file changed, 136 insertions(+)
diff --git a/block/archipelago.c b/block/archipelago.c
index 5a678ed..d05fa2f 100644
--- a
go:my_vm_volume/mport=123
file=archipelago:my_vm_volume/mport=123:vport=1234
or
file.driver=archipelago,file.volume=my_vm_volume
file.driver=archipelago,file.volume=my_vm_volume,file.mport=123
file.driver=archipelago,file.volume=my_vm_volume,file.mport=123,
file.vport=1234
Signed-off-by: Chrysostomos Nanakos
-
On 06/02/2014 06:31 PM, Kevin Wolf wrote:
Am 02.06.2014 um 17:04 hat Chrysostomos Nanakos geschrieben:
Hi,
On 06/02/2014 05:30 PM, Eric Blake wrote:
On 06/02/2014 08:15 AM, Chrysostomos Nanakos wrote:
VM Image on Archipelago volume is specified like this:
file=archipelago:[/mport=[:vport
Hi,
On 06/02/2014 05:30 PM, Eric Blake wrote:
On 06/02/2014 08:15 AM, Chrysostomos Nanakos wrote:
VM Image on Archipelago volume is specified like this:
file=archipelago:[/mport=[:vport=]]
or
I still think that there is no need to burden the code with this
shorthand...
If everyone agrees I
copy data directly
to the destination buffer.
- Remove archipelago_aio_bh_cb() function, a full request is completed in
qemu_archipelago_complete_aio() instead.
- Resolve proposed changes from Kevin Wolf and miscellaneous style issues.
Chrysostomos Nanakos (1):
block: Support Archipelago
go:my_vm_volume/mport=123
file=archipelago:my_vm_volume/mport=123:vport=1234
or
file.driver=archipelago,file.volume=my_vm_volume
file.driver=archipelago,file.volume=my_vm_volume,file.mport=123
file.driver=archipelago,file.volume=my_vm_volume,file.mport=123,
file.vport=1234
Signed-off-by: Chrysostomos Nanakos
-
'vport' is the port number on which vlmcd is listening. This is optional
and if not specified, QEMU will make Archipelago to use the default port.
Examples:
file=archipelago:my_vm_volume
file=archipelago:my_vm_volume/mport=123
file=archipelago:my_vm_volume/mport=123:vport=1234
Signe
[2]
http://synnefo-software.blogspot.com/2013/10/archipelago-officially-open-source.html
[3] https://code.google.com/p/ganeti/
[4] http://ceph.com
[5]
http://www.gluster.org/community/documentation/index.php/Features/Archipelago_Integration
Chrysostomos Nanakos (1):
block: Support Archipelago
77 matches
Mail list logo
