* [Qemu-devel] [PULL 0/6] Block patches
@ 2014-06-23 9:31 Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 1/6] block: m25p80: sync_page(): Deindent function body Stefan Hajnoczi
` (6 more replies)
0 siblings, 7 replies; 18+ messages in thread
From: Stefan Hajnoczi @ 2014-06-23 9:31 UTC (permalink / raw)
To: qemu-devel; +Cc: Peter Maydell, Stefan Hajnoczi
The following changes since commit 427e1750a0b98a72cad424327604f51e993dcc5f:
gt64xxx_pci: Add VMStateDescription (2014-06-20 23:40:16 +0200)
are available in the git repository at:
git://github.com/stefanha/qemu.git tags/block-pull-request
for you to fetch changes up to 2bd3bce8efebe86b031beab5c0e3b9bbaec0b502:
block: asynchronously stop the VM on I/O errors (2014-06-23 16:36:13 +0800)
----------------------------------------------------------------
Block pull request
----------------------------------------------------------------
Chunyan Liu (1):
QemuOpts: check NULL opts in qemu_opt_get functions
Liu Yuan (1):
sheepdog: fix NULL dereference in sd_create
Paolo Bonzini (2):
vl: allow other threads to do qemu_system_vmstop_request
block: asynchronously stop the VM on I/O errors
Peter Crosthwaite (2):
block: m25p80: sync_page(): Deindent function body.
block: m25p80: Support read only bdrvs.
block.c | 21 ++++++++++--
block/sheepdog.c | 1 +
cpus.c | 1 +
docs/qmp/qmp-events.txt | 2 +-
hw/block/m25p80.c | 30 ++++++++---------
include/sysemu/sysemu.h | 1 +
stubs/vm-stop.c | 7 +++-
target-lm32/op_helper.c | 2 +-
util/qemu-option.c | 28 +++++++++++++---
vl.c | 85 +++++++++++++++++++++++++++++++------------------
10 files changed, 122 insertions(+), 56 deletions(-)
--
1.9.3
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 1/6] block: m25p80: sync_page(): Deindent function body.
2014-06-23 9:31 [Qemu-devel] [PULL 0/6] Block patches Stefan Hajnoczi
@ 2014-06-23 9:31 ` Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 2/6] block: m25p80: Support read only bdrvs Stefan Hajnoczi
` (5 subsequent siblings)
6 siblings, 0 replies; 18+ messages in thread
From: Stefan Hajnoczi @ 2014-06-23 9:31 UTC (permalink / raw)
To: qemu-devel; +Cc: Peter Maydell, Peter Crosthwaite, Stefan Hajnoczi
From: Peter Crosthwaite <peter.crosthwaite@xilinx.com>
sync_page() was conditionalizing it's whole fn body on the bdrv being
non-null. Just return for the function immediately on NULL brdv and
get rid of the big if.
Makes implementation consistent with flash_zynq_area().
Signed-off-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
hw/block/m25p80.c | 24 +++++++++++++-----------
1 file changed, 13 insertions(+), 11 deletions(-)
diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c
index 4076114..e4ef733 100644
--- a/hw/block/m25p80.c
+++ b/hw/block/m25p80.c
@@ -288,18 +288,20 @@ static void bdrv_sync_complete(void *opaque, int ret)
static void flash_sync_page(Flash *s, int page)
{
- if (s->bdrv) {
- int bdrv_sector, nb_sectors;
- QEMUIOVector iov;
-
- bdrv_sector = (page * s->pi->page_size) / BDRV_SECTOR_SIZE;
- nb_sectors = DIV_ROUND_UP(s->pi->page_size, BDRV_SECTOR_SIZE);
- qemu_iovec_init(&iov, 1);
- qemu_iovec_add(&iov, s->storage + bdrv_sector * BDRV_SECTOR_SIZE,
- nb_sectors * BDRV_SECTOR_SIZE);
- bdrv_aio_writev(s->bdrv, bdrv_sector, &iov, nb_sectors,
- bdrv_sync_complete, NULL);
+ int bdrv_sector, nb_sectors;
+ QEMUIOVector iov;
+
+ if (!s->bdrv) {
+ return;
}
+
+ bdrv_sector = (page * s->pi->page_size) / BDRV_SECTOR_SIZE;
+ nb_sectors = DIV_ROUND_UP(s->pi->page_size, BDRV_SECTOR_SIZE);
+ qemu_iovec_init(&iov, 1);
+ qemu_iovec_add(&iov, s->storage + bdrv_sector * BDRV_SECTOR_SIZE,
+ nb_sectors * BDRV_SECTOR_SIZE);
+ bdrv_aio_writev(s->bdrv, bdrv_sector, &iov, nb_sectors, bdrv_sync_complete,
+ NULL);
}
static inline void flash_sync_area(Flash *s, int64_t off, int64_t len)
--
1.9.3
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 2/6] block: m25p80: Support read only bdrvs.
2014-06-23 9:31 [Qemu-devel] [PULL 0/6] Block patches Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 1/6] block: m25p80: sync_page(): Deindent function body Stefan Hajnoczi
@ 2014-06-23 9:31 ` Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 3/6] QemuOpts: check NULL opts in qemu_opt_get functions Stefan Hajnoczi
` (4 subsequent siblings)
6 siblings, 0 replies; 18+ messages in thread
From: Stefan Hajnoczi @ 2014-06-23 9:31 UTC (permalink / raw)
To: qemu-devel; +Cc: Peter Maydell, Peter Crosthwaite, Stefan Hajnoczi
From: Peter Crosthwaite <peter.crosthwaite@xilinx.com>
By just never doing write-backs. This is completely invisible to the
guest, as the entire storage area is implemented as device state (at
realize time the entire drive is read in).
Signed-off-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
hw/block/m25p80.c | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c
index e4ef733..5893773 100644
--- a/hw/block/m25p80.c
+++ b/hw/block/m25p80.c
@@ -291,7 +291,7 @@ static void flash_sync_page(Flash *s, int page)
int bdrv_sector, nb_sectors;
QEMUIOVector iov;
- if (!s->bdrv) {
+ if (!s->bdrv || bdrv_is_read_only(s->bdrv)) {
return;
}
@@ -309,7 +309,7 @@ static inline void flash_sync_area(Flash *s, int64_t off, int64_t len)
int64_t start, end, nb_sectors;
QEMUIOVector iov;
- if (!s->bdrv) {
+ if (!s->bdrv || bdrv_is_read_only(s->bdrv)) {
return;
}
@@ -627,10 +627,6 @@ static int m25p80_init(SSISlave *ss)
if (dinfo && dinfo->bdrv) {
DB_PRINT_L(0, "Binding to IF_MTD drive\n");
s->bdrv = dinfo->bdrv;
- if (bdrv_is_read_only(s->bdrv)) {
- fprintf(stderr, "Can't use a read-only drive");
- return 1;
- }
/* FIXME: Move to late init */
if (bdrv_read(s->bdrv, 0, s->storage, DIV_ROUND_UP(s->size,
--
1.9.3
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 3/6] QemuOpts: check NULL opts in qemu_opt_get functions
2014-06-23 9:31 [Qemu-devel] [PULL 0/6] Block patches Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 1/6] block: m25p80: sync_page(): Deindent function body Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 2/6] block: m25p80: Support read only bdrvs Stefan Hajnoczi
@ 2014-06-23 9:31 ` Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 4/6] sheepdog: fix NULL dereference in sd_create Stefan Hajnoczi
` (3 subsequent siblings)
6 siblings, 0 replies; 18+ messages in thread
From: Stefan Hajnoczi @ 2014-06-23 9:31 UTC (permalink / raw)
To: qemu-devel; +Cc: Peter Maydell, Chunyan Liu, Stefan Hajnoczi
From: Chunyan Liu <cyliu@suse.com>
Some places will call bdrv_create_file(filename, NULL, &local_err), where
opts is NULL. Check NULL in qemu_opt_get and qemu_opt_get_*_del functions,
to avoid extra effort of checking opts before calling them every time.
Signed-off-by: Chunyan Liu <cyliu@suse.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
util/qemu-option.c | 28 ++++++++++++++++++++++++----
1 file changed, 24 insertions(+), 4 deletions(-)
diff --git a/util/qemu-option.c b/util/qemu-option.c
index 836055a..43de3ad 100644
--- a/util/qemu-option.c
+++ b/util/qemu-option.c
@@ -310,8 +310,13 @@ static void qemu_opt_del_all(QemuOpts *opts, const char *name)
const char *qemu_opt_get(QemuOpts *opts, const char *name)
{
- QemuOpt *opt = qemu_opt_find(opts, name);
+ QemuOpt *opt;
+ if (opts == NULL) {
+ return NULL;
+ }
+
+ opt = qemu_opt_find(opts, name);
if (!opt) {
const QemuOptDesc *desc = find_desc_by_name(opts->list->desc, name);
if (desc && desc->def_value_str) {
@@ -364,9 +369,14 @@ bool qemu_opt_has_help_opt(QemuOpts *opts)
static bool qemu_opt_get_bool_helper(QemuOpts *opts, const char *name,
bool defval, bool del)
{
- QemuOpt *opt = qemu_opt_find(opts, name);
+ QemuOpt *opt;
bool ret = defval;
+ if (opts == NULL) {
+ return ret;
+ }
+
+ opt = qemu_opt_find(opts, name);
if (opt == NULL) {
const QemuOptDesc *desc = find_desc_by_name(opts->list->desc, name);
if (desc && desc->def_value_str) {
@@ -395,9 +405,14 @@ bool qemu_opt_get_bool_del(QemuOpts *opts, const char *name, bool defval)
static uint64_t qemu_opt_get_number_helper(QemuOpts *opts, const char *name,
uint64_t defval, bool del)
{
- QemuOpt *opt = qemu_opt_find(opts, name);
+ QemuOpt *opt;
uint64_t ret = defval;
+ if (opts == NULL) {
+ return ret;
+ }
+
+ opt = qemu_opt_find(opts, name);
if (opt == NULL) {
const QemuOptDesc *desc = find_desc_by_name(opts->list->desc, name);
if (desc && desc->def_value_str) {
@@ -427,9 +442,14 @@ uint64_t qemu_opt_get_number_del(QemuOpts *opts, const char *name,
static uint64_t qemu_opt_get_size_helper(QemuOpts *opts, const char *name,
uint64_t defval, bool del)
{
- QemuOpt *opt = qemu_opt_find(opts, name);
+ QemuOpt *opt;
uint64_t ret = defval;
+ if (opts == NULL) {
+ return ret;
+ }
+
+ opt = qemu_opt_find(opts, name);
if (opt == NULL) {
const QemuOptDesc *desc = find_desc_by_name(opts->list->desc, name);
if (desc && desc->def_value_str) {
--
1.9.3
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 4/6] sheepdog: fix NULL dereference in sd_create
2014-06-23 9:31 [Qemu-devel] [PULL 0/6] Block patches Stefan Hajnoczi
` (2 preceding siblings ...)
2014-06-23 9:31 ` [Qemu-devel] [PULL 3/6] QemuOpts: check NULL opts in qemu_opt_get functions Stefan Hajnoczi
@ 2014-06-23 9:31 ` Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 5/6] vl: allow other threads to do qemu_system_vmstop_request Stefan Hajnoczi
` (2 subsequent siblings)
6 siblings, 0 replies; 18+ messages in thread
From: Stefan Hajnoczi @ 2014-06-23 9:31 UTC (permalink / raw)
To: qemu-devel; +Cc: Kevin Wolf, Peter Maydell, Stefan Hajnoczi, Liu Yuan
From: Liu Yuan <namei.unix@gmail.com>
Following command
qemu-img create -f qcow2 sheepdog:test 20g
will cause core dump because aio_context is NULL in sd_create. We should
initialize it by qemu_get_aio_context() to avoid NULL dereference.
Cc: qemu-devel@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Liu Yuan <namei.unix@gmail.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
block/sheepdog.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/block/sheepdog.c b/block/sheepdog.c
index 2dcc595..8d9350c 100644
--- a/block/sheepdog.c
+++ b/block/sheepdog.c
@@ -1756,6 +1756,7 @@ static int sd_create(const char *filename, QemuOpts *opts,
bdrv_unref(bs);
}
+ s->aio_context = qemu_get_aio_context();
ret = do_sd_create(s, &vid, 0, errp);
if (ret) {
goto out;
--
1.9.3
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 5/6] vl: allow other threads to do qemu_system_vmstop_request
2014-06-23 9:31 [Qemu-devel] [PULL 0/6] Block patches Stefan Hajnoczi
` (3 preceding siblings ...)
2014-06-23 9:31 ` [Qemu-devel] [PULL 4/6] sheepdog: fix NULL dereference in sd_create Stefan Hajnoczi
@ 2014-06-23 9:31 ` Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 6/6] block: asynchronously stop the VM on I/O errors Stefan Hajnoczi
2014-06-23 12:18 ` [Qemu-devel] [PULL 0/6] Block patches Peter Maydell
6 siblings, 0 replies; 18+ messages in thread
From: Stefan Hajnoczi @ 2014-06-23 9:31 UTC (permalink / raw)
To: qemu-devel; +Cc: Peter Maydell, Stefan Hajnoczi, Paolo Bonzini
From: Paolo Bonzini <pbonzini@redhat.com>
There patch protects vmstop_requested with a lock and introduces
qemu_system_vmstop_request_prepare.
Together with the new call to qemu_vmstop_requested in vm_start,
qemu_system_vmstop_request_prepare avoids a race where the VM could remain
stopped even though the iostatus of a block device has already been set
(for example).
qemu_system_vmstop_request_prepare however also lets the caller thread
delay observation of the state change until it has itself communicated
that change to the user. This delay avoids any possibility of a wrong
reordering of the BLOCK_IO_ERROR event and the subsequent STOP event.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
cpus.c | 1 +
include/sysemu/sysemu.h | 1 +
target-lm32/op_helper.c | 2 +-
vl.c | 85 +++++++++++++++++++++++++++++++------------------
4 files changed, 57 insertions(+), 32 deletions(-)
diff --git a/cpus.c b/cpus.c
index 1ec3a9e..06da4e7 100644
--- a/cpus.c
+++ b/cpus.c
@@ -1206,6 +1206,7 @@ void cpu_stop_current(void)
int vm_stop(RunState state)
{
if (qemu_in_vcpu_thread()) {
+ qemu_system_vmstop_request_prepare();
qemu_system_vmstop_request(state);
/*
* FIXME: should not return to device code in case
diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
index 277230d..6b4cc13 100644
--- a/include/sysemu/sysemu.h
+++ b/include/sysemu/sysemu.h
@@ -62,6 +62,7 @@ void qemu_system_powerdown_request(void);
void qemu_register_powerdown_notifier(Notifier *notifier);
void qemu_system_debug_request(void);
void qemu_system_vmstop_request(RunState reason);
+void qemu_system_vmstop_request_prepare(void);
int qemu_shutdown_requested_get(void);
int qemu_reset_requested_get(void);
void qemu_system_killed(int signal, pid_t pid);
diff --git a/target-lm32/op_helper.c b/target-lm32/op_helper.c
index 308742a..61209c1 100644
--- a/target-lm32/op_helper.c
+++ b/target-lm32/op_helper.c
@@ -42,7 +42,7 @@ void HELPER(ill)(CPULM32State *env)
fprintf(stderr, "VM paused due to illegal instruction. "
"Connect a debugger or switch to the monitor console "
"to find out more.\n");
- qemu_system_vmstop_request(RUN_STATE_PAUSED);
+ vm_stop(RUN_STATE_PAUSED);
cs->halted = 1;
raise_exception(env, EXCP_HALTED);
#endif
diff --git a/vl.c b/vl.c
index 54b4627..ab8f152 100644
--- a/vl.c
+++ b/vl.c
@@ -574,6 +574,10 @@ static int default_driver_check(QemuOpts *opts, void *opaque)
static RunState current_run_state = RUN_STATE_PRELAUNCH;
+/* We use RUN_STATE_MAX but any invalid value will do */
+static RunState vmstop_requested = RUN_STATE_MAX;
+static QemuMutex vmstop_lock;
+
typedef struct {
RunState from;
RunState to;
@@ -650,10 +654,11 @@ static void runstate_init(void)
const RunStateTransition *p;
memset(&runstate_valid_transitions, 0, sizeof(runstate_valid_transitions));
-
for (p = &runstate_transitions_def[0]; p->from != RUN_STATE_MAX; p++) {
runstate_valid_transitions[p->from][p->to] = true;
}
+
+ qemu_mutex_init(&vmstop_lock);
}
/* This function will abort() on invalid state transitions */
@@ -693,6 +698,54 @@ StatusInfo *qmp_query_status(Error **errp)
return info;
}
+static bool qemu_vmstop_requested(RunState *r)
+{
+ qemu_mutex_lock(&vmstop_lock);
+ *r = vmstop_requested;
+ vmstop_requested = RUN_STATE_MAX;
+ qemu_mutex_unlock(&vmstop_lock);
+ return *r < RUN_STATE_MAX;
+}
+
+void qemu_system_vmstop_request_prepare(void)
+{
+ qemu_mutex_lock(&vmstop_lock);
+}
+
+void qemu_system_vmstop_request(RunState state)
+{
+ vmstop_requested = state;
+ qemu_mutex_unlock(&vmstop_lock);
+ qemu_notify_event();
+}
+
+void vm_start(void)
+{
+ RunState requested;
+
+ qemu_vmstop_requested(&requested);
+ if (runstate_is_running() && requested == RUN_STATE_MAX) {
+ return;
+ }
+
+ /* Ensure that a STOP/RESUME pair of events is emitted if a
+ * vmstop request was pending. The BLOCK_IO_ERROR event, for
+ * example, according to documentation is always followed by
+ * the STOP event.
+ */
+ if (runstate_is_running()) {
+ monitor_protocol_event(QEVENT_STOP, NULL);
+ } else {
+ cpu_enable_ticks();
+ runstate_set(RUN_STATE_RUNNING);
+ vm_state_notify(1, RUN_STATE_RUNNING);
+ resume_all_vcpus();
+ }
+
+ monitor_protocol_event(QEVENT_RESUME, NULL);
+}
+
+
/***********************************************************/
/* real time host monotonic timer */
@@ -1658,17 +1711,6 @@ void vm_state_notify(int running, RunState state)
}
}
-void vm_start(void)
-{
- if (!runstate_is_running()) {
- cpu_enable_ticks();
- runstate_set(RUN_STATE_RUNNING);
- vm_state_notify(1, RUN_STATE_RUNNING);
- resume_all_vcpus();
- monitor_protocol_event(QEVENT_RESUME, NULL);
- }
-}
-
/* reset/shutdown handler */
typedef struct QEMUResetEntry {
@@ -1693,7 +1735,6 @@ static NotifierList suspend_notifiers =
static NotifierList wakeup_notifiers =
NOTIFIER_LIST_INITIALIZER(wakeup_notifiers);
static uint32_t wakeup_reason_mask = ~(1 << QEMU_WAKEUP_REASON_NONE);
-static RunState vmstop_requested = RUN_STATE_MAX;
int qemu_shutdown_requested_get(void)
{
@@ -1761,18 +1802,6 @@ static int qemu_debug_requested(void)
return r;
}
-/* We use RUN_STATE_MAX but any invalid value will do */
-static bool qemu_vmstop_requested(RunState *r)
-{
- if (vmstop_requested < RUN_STATE_MAX) {
- *r = vmstop_requested;
- vmstop_requested = RUN_STATE_MAX;
- return true;
- }
-
- return false;
-}
-
void qemu_register_reset(QEMUResetHandler *func, void *opaque)
{
QEMUResetEntry *re = g_malloc0(sizeof(QEMUResetEntry));
@@ -1922,12 +1951,6 @@ void qemu_system_debug_request(void)
qemu_notify_event();
}
-void qemu_system_vmstop_request(RunState state)
-{
- vmstop_requested = state;
- qemu_notify_event();
-}
-
static bool main_loop_should_exit(void)
{
RunState r;
--
1.9.3
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 6/6] block: asynchronously stop the VM on I/O errors
2014-06-23 9:31 [Qemu-devel] [PULL 0/6] Block patches Stefan Hajnoczi
` (4 preceding siblings ...)
2014-06-23 9:31 ` [Qemu-devel] [PULL 5/6] vl: allow other threads to do qemu_system_vmstop_request Stefan Hajnoczi
@ 2014-06-23 9:31 ` Stefan Hajnoczi
2014-06-23 12:18 ` [Qemu-devel] [PULL 0/6] Block patches Peter Maydell
6 siblings, 0 replies; 18+ messages in thread
From: Stefan Hajnoczi @ 2014-06-23 9:31 UTC (permalink / raw)
To: qemu-devel; +Cc: Peter Maydell, Stefan Hajnoczi, Paolo Bonzini
From: Paolo Bonzini <pbonzini@redhat.com>
With virtio-blk dataplane, I/O errors might occur while QEMU is
not in the main I/O thread. However, it's invalid to call vm_stop
when we're neither in a VCPU thread nor in the main I/O thread,
even if we were to take the iothread mutex around it.
To avoid this problem, we can raise a request to the main I/O thread,
similar to what QEMU does when vm_stop is called from a CPU thread.
We know that bdrv_error_action is called from an AIO callback, and
the moment at which the callback will fire is not well-defined; it
depends on the moment at which the disk or OS finishes the operation,
which can happen at any time. Note that QEMU is certainly not in a CPU
thread and we do not need to call cpu_stop_current() like vm_stop() does.
However, we need to ensure that any action taken by management will
result in correct detection of the error _and_ a running VM. In particular:
- the event must be raised after the iostatus has been set, so that
"info block" will return an iostatus that matches the event.
- the VM must be stopped after the iostatus has been set, so that
"info block" will return an iostatus that matches the runstate.
The ordering between the STOP and BLOCK_IO_ERROR events is preserved;
BLOCK_IO_ERROR is documented to come first.
This makes bdrv_error_action() thread safe (assuming QMP events are,
which is attacked by a separate series).
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
block.c | 21 +++++++++++++++++++--
docs/qmp/qmp-events.txt | 2 +-
stubs/vm-stop.c | 7 ++++++-
3 files changed, 26 insertions(+), 4 deletions(-)
diff --git a/block.c b/block.c
index 43abe96..ff44e76 100644
--- a/block.c
+++ b/block.c
@@ -3626,10 +3626,27 @@ void bdrv_error_action(BlockDriverState *bs, BlockErrorAction action,
bool is_read, int error)
{
assert(error >= 0);
- bdrv_emit_qmp_error_event(bs, QEVENT_BLOCK_IO_ERROR, action, is_read);
+
if (action == BDRV_ACTION_STOP) {
- vm_stop(RUN_STATE_IO_ERROR);
+ /* First set the iostatus, so that "info block" returns an iostatus
+ * that matches the events raised so far (an additional error iostatus
+ * is fine, but not a lost one).
+ */
bdrv_iostatus_set_err(bs, error);
+
+ /* Then raise the request to stop the VM and the event.
+ * qemu_system_vmstop_request_prepare has two effects. First,
+ * it ensures that the STOP event always comes after the
+ * BLOCK_IO_ERROR event. Second, it ensures that even if management
+ * can observe the STOP event and do a "cont" before the STOP
+ * event is issued, the VM will not stop. In this case, vm_start()
+ * also ensures that the STOP/RESUME pair of events is emitted.
+ */
+ qemu_system_vmstop_request_prepare();
+ bdrv_emit_qmp_error_event(bs, QEVENT_BLOCK_IO_ERROR, action, is_read);
+ qemu_system_vmstop_request(RUN_STATE_IO_ERROR);
+ } else {
+ bdrv_emit_qmp_error_event(bs, QEVENT_BLOCK_IO_ERROR, action, is_read);
}
}
diff --git a/docs/qmp/qmp-events.txt b/docs/qmp/qmp-events.txt
index 019db53..22fea58 100644
--- a/docs/qmp/qmp-events.txt
+++ b/docs/qmp/qmp-events.txt
@@ -62,7 +62,7 @@ Data:
- "action": action that has been taken, it's one of the following (json-string):
"ignore": error has been ignored
"report": error has been reported to the device
- "stop": error caused VM to be stopped
+ "stop": the VM is going to stop because of the error
Example:
diff --git a/stubs/vm-stop.c b/stubs/vm-stop.c
index f82c897..69fd86b 100644
--- a/stubs/vm-stop.c
+++ b/stubs/vm-stop.c
@@ -1,7 +1,12 @@
#include "qemu-common.h"
#include "sysemu/sysemu.h"
-int vm_stop(RunState state)
+void qemu_system_vmstop_request_prepare(void)
+{
+ abort();
+}
+
+void qemu_system_vmstop_request(RunState state)
{
abort();
}
--
1.9.3
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [Qemu-devel] [PULL 0/6] Block patches
2014-06-23 9:31 [Qemu-devel] [PULL 0/6] Block patches Stefan Hajnoczi
` (5 preceding siblings ...)
2014-06-23 9:31 ` [Qemu-devel] [PULL 6/6] block: asynchronously stop the VM on I/O errors Stefan Hajnoczi
@ 2014-06-23 12:18 ` Peter Maydell
6 siblings, 0 replies; 18+ messages in thread
From: Peter Maydell @ 2014-06-23 12:18 UTC (permalink / raw)
To: Stefan Hajnoczi; +Cc: QEMU Developers
On 23 June 2014 10:31, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> The following changes since commit 427e1750a0b98a72cad424327604f51e993dcc5f:
>
> gt64xxx_pci: Add VMStateDescription (2014-06-20 23:40:16 +0200)
>
> are available in the git repository at:
>
> git://github.com/stefanha/qemu.git tags/block-pull-request
>
> for you to fetch changes up to 2bd3bce8efebe86b031beab5c0e3b9bbaec0b502:
>
> block: asynchronously stop the VM on I/O errors (2014-06-23 16:36:13 +0800)
>
> ----------------------------------------------------------------
> Block pull request
Applied, thanks.
-- PMM
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [Qemu-devel] [PULL 0/6] Block patches
2018-03-05 9:40 Stefan Hajnoczi
2018-03-05 9:52 ` no-reply
@ 2018-03-05 18:55 ` Peter Maydell
1 sibling, 0 replies; 18+ messages in thread
From: Peter Maydell @ 2018-03-05 18:55 UTC (permalink / raw)
To: Stefan Hajnoczi; +Cc: QEMU Developers, Qemu-block
On 5 March 2018 at 09:40, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> The following changes since commit 136c67e07869227b21b3f627316e03679ce7b738:
>
> Merge remote-tracking branch 'remotes/bkoppelmann/tags/pull-tricore-2018-03-02' into staging (2018-03-02 16:56:20 +0000)
>
> are available in the Git repository at:
>
> git://github.com/stefanha/qemu.git tags/block-pull-request
>
> for you to fetch changes up to 23500c6a9409efc80d696aede0629bfbe7556a90:
>
> README: Document 'git-publish' workflow (2018-03-05 09:03:17 +0000)
>
> ----------------------------------------------------------------
> Pull request
>
> Mostly patches that are only indirectly related to the block layer, but I've
> reviewed them and there is no maintainer.
>
> ----------------------------------------------------------------
>
> Fam Zheng (2):
> Add a git-publish configuration file
> README: Document 'git-publish' workflow
>
> Su Hang (3):
> util/uri.c: Coding style check, Only whitespace involved
> util/uri.c: remove brackets that wrap `return` statement's content.
> util/uri.c: wrap single statement blocks with braces {}
>
> Thomas Huth (1):
> tests/libqos: Check for valid dev pointer when looking for PCI devices
>
> tests/libqos/virtio-pci.c | 4 +-
> util/uri.c | 1733 ++++++++++++++++++++++++---------------------
> .gitpublish | 51 ++
> README | 31 +-
> 4 files changed, 1014 insertions(+), 805 deletions(-)
> create mode 100644 .gitpublish
Applied, thanks.
-- PMM
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [Qemu-devel] [PULL 0/6] Block patches
2018-03-05 9:40 Stefan Hajnoczi
@ 2018-03-05 9:52 ` no-reply
2018-03-05 18:55 ` Peter Maydell
1 sibling, 0 replies; 18+ messages in thread
From: no-reply @ 2018-03-05 9:52 UTC (permalink / raw)
To: stefanha; +Cc: famz, qemu-devel, peter.maydell, qemu-block
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180305094006.21446-1-stefanha@redhat.com
Subject: [Qemu-devel] [PULL 0/6] Block patches
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total=$(git log --oneline $BASE.. | wc -l)
failed=0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
commits="$(git log --format=%H --reverse $BASE..)"
for c in $commits; do
echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..."
if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then
failed=1
echo
fi
n=$((n+1))
done
exit $failed
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
136c67e078..4f51e1d386 master -> master
* [new tag] patchew/1520243276-30197-1-git-send-email-thuth@redhat.com -> patchew/1520243276-30197-1-git-send-email-thuth@redhat.com
* [new tag] patchew/20180305094006.21446-1-stefanha@redhat.com -> patchew/20180305094006.21446-1-stefanha@redhat.com
Switched to a new branch 'test'
20eeab9795 README: Document 'git-publish' workflow
333283c56b Add a git-publish configuration file
3a9752c776 tests/libqos: Check for valid dev pointer when looking for PCI devices
6863406cb9 util/uri.c: wrap single statement blocks with braces {}
906ebc14e5 util/uri.c: remove brackets that wrap `return` statement's content.
d0e12d0916 util/uri.c: Coding style check, Only whitespace involved
=== OUTPUT BEGIN ===
Checking PATCH 1/6: util/uri.c: Coding style check, Only whitespace involved...
ERROR: return is not a function, parentheses are not required
#171: FILE: util/uri.c:215:
+ return (-1);
ERROR: return is not a function, parentheses are not required
#176: FILE: util/uri.c:219:
+ return (2);
ERROR: return is not a function, parentheses are not required
#190: FILE: util/uri.c:229:
+ return (0);
ERROR: braces {} are necessary for all arms of this statement
#212: FILE: util/uri.c:262:
+ if (uri->cleanup & 2)
[...]
+ else
[...]
ERROR: braces {} are necessary for all arms of this statement
#276: FILE: util/uri.c:359:
+ if (uri->cleanup & 2)
[...]
+ else
[...]
ERROR: return is not a function, parentheses are not required
#282: FILE: util/uri.c:365:
+ return (0);
ERROR: return is not a function, parentheses are not required
#285: FILE: util/uri.c:367:
+ return (1);
ERROR: return is not a function, parentheses are not required
#305: FILE: util/uri.c:389:
+ return (1);
ERROR: braces {} are necessary for all arms of this statement
#306: FILE: util/uri.c:390:
+ if (!ISA_DIGIT(cur + 1))
[...]
+ else if ((*cur != '0') && (ISA_DIGIT(cur + 1)) && (!ISA_DIGIT(cur + 2)))
[...]
else if ((*cur == '1') && (ISA_DIGIT(cur + 1)) && (ISA_DIGIT(cur + 2)))
[...]
- else if ((*cur == '2') && (*(cur + 1) >= '0') &&
[...]
- else if ((*cur == '2') && (*(cur + 1) == '5') &&
[...]
+ cur += 3;
[...]
ERROR: braces {} are necessary for all arms of this statement
#308: FILE: util/uri.c:392:
+ else if ((*cur != '0') && (ISA_DIGIT(cur + 1)) && (!ISA_DIGIT(cur + 2)))
[...]
else if ((*cur == '1') && (ISA_DIGIT(cur + 1)) && (ISA_DIGIT(cur + 2)))
[...]
- else if ((*cur == '2') && (*(cur + 1) >= '0') &&
[...]
- else if ((*cur == '2') && (*(cur + 1) == '5') &&
[...]
+ cur += 3;
[...]
ERROR: braces {} are necessary for all arms of this statement
#319: FILE: util/uri.c:396:
+ else if ((*cur == '2') && (*(cur + 1) >= '0') && (*(cur + 1) <= '4') &&
[...]
+ else if ((*cur == '2') && (*(cur + 1) == '5') && (*(cur + 2) >= '0') &&
[...]
else
[...]
ERROR: braces {} are necessary for all arms of this statement
#322: FILE: util/uri.c:399:
+ else if ((*cur == '2') && (*(cur + 1) == '5') && (*(cur + 2) >= '0') &&
[...]
else
[...]
ERROR: return is not a function, parentheses are not required
#327: FILE: util/uri.c:403:
+ return (1);
ERROR: return is not a function, parentheses are not required
#330: FILE: util/uri.c:405:
+ return (0);
ERROR: braces {} are necessary even for single statement blocks
#354: FILE: util/uri.c:433:
+ while ((*cur != ']') && (*cur != 0))
+ cur++;
ERROR: braces {} are necessary for all arms of this statement
#356: FILE: util/uri.c:435:
+ if (*cur != ']')
[...]
ERROR: return is not a function, parentheses are not required
#357: FILE: util/uri.c:436:
+ return (1);
ERROR: braces {} are necessary for all arms of this statement
#371: FILE: util/uri.c:446:
+ if (*cur != '.')
[...]
ERROR: braces {} are necessary for all arms of this statement
#379: FILE: util/uri.c:451:
+ if (*cur != '.')
[...]
ERROR: braces {} are necessary for all arms of this statement
#386: FILE: util/uri.c:455:
+ if (*cur != '.')
[...]
ERROR: braces {} are necessary for all arms of this statement
#412: FILE: util/uri.c:473:
+ if (cur != host) {
[...]
+ } else
[...]
ERROR: braces {} are necessary for all arms of this statement
#413: FILE: util/uri.c:474:
+ if (uri->cleanup & 2)
[...]
+ else
[...]
ERROR: return is not a function, parentheses are not required
#422: FILE: util/uri.c:482:
+ return (0);
ERROR: braces {} are necessary for all arms of this statement
#441: FILE: util/uri.c:512:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#442: FILE: util/uri.c:513:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#447: FILE: util/uri.c:517:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#448: FILE: util/uri.c:518:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#452: FILE: util/uri.c:521:
+ return (0);
ERROR: return is not a function, parentheses are not required
#471: FILE: util/uri.c:547:
+ return (0);
ERROR: return is not a function, parentheses are not required
#472: FILE: util/uri.c:548:
+ return (1);
ERROR: return is not a function, parentheses are not required
#494: FILE: util/uri.c:579:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#513: FILE: util/uri.c:616:
+ return (1);
ERROR: return is not a function, parentheses are not required
#526: FILE: util/uri.c:624:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#546: FILE: util/uri.c:662:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#547: FILE: util/uri.c:663:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#554: FILE: util/uri.c:668:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#573: FILE: util/uri.c:705:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#574: FILE: util/uri.c:706:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#581: FILE: util/uri.c:711:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#606: FILE: util/uri.c:753:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#607: FILE: util/uri.c:754:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#609: FILE: util/uri.c:756:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#610: FILE: util/uri.c:757:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#612: FILE: util/uri.c:759:
+ return (0);
ERROR: braces {} are necessary for all arms of this statement
#616: FILE: util/uri.c:762:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#617: FILE: util/uri.c:763:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#621: FILE: util/uri.c:766:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#622: FILE: util/uri.c:767:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#653: FILE: util/uri.c:802:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#654: FILE: util/uri.c:803:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#656: FILE: util/uri.c:805:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#657: FILE: util/uri.c:806:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#662: FILE: util/uri.c:809:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#663: FILE: util/uri.c:810:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#667: FILE: util/uri.c:813:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#668: FILE: util/uri.c:814:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#687: FILE: util/uri.c:826:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#688: FILE: util/uri.c:827:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#696: FILE: util/uri.c:832:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#697: FILE: util/uri.c:833:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#703: FILE: util/uri.c:837:
+ return (1);
ERROR: return is not a function, parentheses are not required
#706: FILE: util/uri.c:839:
+ return (0);
ERROR: braces {} are necessary for all arms of this statement
#725: FILE: util/uri.c:859:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#726: FILE: util/uri.c:860:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#729: FILE: util/uri.c:862:
+ return (1);
ERROR: braces {} are necessary for all arms of this statement
#734: FILE: util/uri.c:866:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#735: FILE: util/uri.c:867:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#742: FILE: util/uri.c:871:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#743: FILE: util/uri.c:872:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#751: FILE: util/uri.c:877:
+ if (ret != 0)
[...]
ERROR: return is not a function, parentheses are not required
#752: FILE: util/uri.c:878:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#758: FILE: util/uri.c:882:
+ return (1);
ERROR: return is not a function, parentheses are not required
#761: FILE: util/uri.c:884:
+ return (0);
ERROR: return is not a function, parentheses are not required
#777: FILE: util/uri.c:904:
+ return (-1);
ERROR: return is not a function, parentheses are not required
#794: FILE: util/uri.c:917:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#798: FILE: util/uri.c:920:
+ return (0);
ERROR: return is not a function, parentheses are not required
#815: FILE: util/uri.c:939:
+ return (NULL);
ERROR: return is not a function, parentheses are not required
#821: FILE: util/uri.c:944:
+ return (NULL);
ERROR: return is not a function, parentheses are not required
#824: FILE: util/uri.c:946:
+ return (uri);
ERROR: return is not a function, parentheses are not required
#837: FILE: util/uri.c:963:
+ return (rfc3986_parse_uri_reference(uri, str));
ERROR: return is not a function, parentheses are not required
#854: FILE: util/uri.c:983:
+ return (NULL);
ERROR: return is not a function, parentheses are not required
#863: FILE: util/uri.c:991:
+ return (NULL);
ERROR: return is not a function, parentheses are not required
#866: FILE: util/uri.c:993:
+ return (uri);
ERROR: return is not a function, parentheses are not required
#891: FILE: util/uri.c:1014:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#910: FILE: util/uri.c:1031:
+ return (temp);
ERROR: braces {} are necessary for all arms of this statement
#930: FILE: util/uri.c:1050:
+ if (uri == NULL)
[...]
ERROR: return is not a function, parentheses are not required
#931: FILE: util/uri.c:1051:
+ return (NULL);
ERROR: braces {} are necessary for all arms of this statement
#983: FILE: util/uri.c:1079:
+ if (IS_RESERVED(*(p)) || IS_UNRESERVED(*(p)))
[...]
+ else {
[...]
ERROR: braces {} are necessary for all arms of this statement
#1033: FILE: util/uri.c:1104:
+ if ((IS_UNRESERVED(*(p))) || ((*(p) == ';')) ||
[...]
+ else {
[...]
ERROR: braces {} are necessary for all arms of this statement
#1117: FILE: util/uri.c:1150:
+ if ((IS_UNRESERVED(*(p))) || ((*(p) == '$')) ||
[...]
+ else {
[...]
ERROR: braces {} are necessary for all arms of this statement
#1187: FILE: util/uri.c:1194:
+ if ((IS_UNRESERVED(*(p))) || ((*(p) == '/')) ||
[...]
- else {
[...]
ERROR: braces {} are necessary for all arms of this statement
#1269: FILE: util/uri.c:1236:
+ if ((IS_UNRESERVED(*(p))) || (IS_RESERVED(*(p))))
[...]
+ else {
[...]
ERROR: return is not a function, parentheses are not required
#1286: FILE: util/uri.c:1252:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#1299: FILE: util/uri.c:1263:
+ if (uri == NULL)
[...]
ERROR: return is not a function, parentheses are not required
#1338: FILE: util/uri.c:1318:
+ return (-1);
ERROR: return is not a function, parentheses are not required
#1349: FILE: util/uri.c:1327:
+ return (0);
ERROR: braces {} are necessary even for single statement blocks
#1375: FILE: util/uri.c:1343:
+ while (cur[0] == '/')
+ cur++;
ERROR: braces {} are necessary for all arms of this statement
#1390: FILE: util/uri.c:1352:
+ if ((cur[0] == '.') && (cur[1] == '\0'))
[...]
ERROR: braces {} are necessary even for single statement blocks
#1408: FILE: util/uri.c:1362:
+ while ((cur[0] == '/') && (cur[1] == '/'))
+ cur++;
ERROR: return is not a function, parentheses are not required
#1424: FILE: util/uri.c:1375:
+ return (0);
ERROR: braces {} are necessary even for single statement blocks
#1517: FILE: util/uri.c:1482:
+ while (cur[0] != '\0')
+ (out++)[0] = (cur++)[0];
ERROR: return is not a function, parentheses are not required
#1524: FILE: util/uri.c:1488:
+ return (0);
ERROR: braces {} are necessary for all arms of this statement
#1532: FILE: util/uri.c:1493:
+ if (((c >= '0') && (c <= '9')) || ((c >= 'a') && (c <= 'f')) ||
[...]
ERROR: return is not a function, parentheses are not required
#1536: FILE: util/uri.c:1495:
+ return (1);
ERROR: return is not a function, parentheses are not required
#1537: FILE: util/uri.c:1496:
+ return (0);
ERROR: return is not a function, parentheses are not required
#1559: FILE: util/uri.c:1519:
+ return (NULL);
ERROR: braces {} are necessary for all arms of this statement
#1560: FILE: util/uri.c:1520:
+ if (len <= 0)
[...]
ERROR: braces {} are necessary for all arms of this statement
#1562: FILE: util/uri.c:1522:
+ if (len < 0)
[...]
ERROR: return is not a function, parentheses are not required
#1563: FILE: util/uri.c:1523:
+ return (NULL);
ERROR: braces {} are necessary for all arms of this statement
#1599: FILE: util/uri.c:1534:
+ if ((*in >= '0') && (*in <= '9'))
[...]
+ else if ((*in >= 'a') && (*in <= 'f'))
[...]
+ else if ((*in >= 'A') && (*in <= 'F'))
[...]
ERROR: braces {} are necessary for all arms of this statement
#1601: FILE: util/uri.c:1536:
+ else if ((*in >= 'a') && (*in <= 'f'))
[...]
+ else if ((*in >= 'A') && (*in <= 'F'))
[...]
ERROR: braces {} are necessary for all arms of this statement
#1603: FILE: util/uri.c:1538:
+ else if ((*in >= 'A') && (*in <= 'F'))
[...]
ERROR: braces {} are necessary for all arms of this statement
#1606: FILE: util/uri.c:1541:
+ if ((*in >= '0') && (*in <= '9'))
[...]
+ else if ((*in >= 'a') && (*in <= 'f'))
[...]
+ else if ((*in >= 'A') && (*in <= 'F'))
[...]
ERROR: braces {} are necessary for all arms of this statement
#1608: FILE: util/uri.c:1543:
+ else if ((*in >= 'a') && (*in <= 'f'))
[...]
+ else if ((*in >= 'A') && (*in <= 'F'))
[...]
ERROR: braces {} are necessary for all arms of this statement
#1610: FILE: util/uri.c:1545:
+ else if ((*in >= 'A') && (*in <= 'F'))
[...]
ERROR: return is not a function, parentheses are not required
#1622: FILE: util/uri.c:1556:
+ return (ret);
ERROR: return is not a function, parentheses are not required
#1641: FILE: util/uri.c:1577:
+ return (NULL);
ERROR: return is not a function, parentheses are not required
#1644: FILE: util/uri.c:1579:
+ return (g_strdup(str));
ERROR: braces {} are necessary for all arms of this statement
#1647: FILE: util/uri.c:1581:
+ if (!(len > 0))
[...]
ERROR: return is not a function, parentheses are not required
#1648: FILE: util/uri.c:1582:
+ return (NULL);
ERROR: braces {} are necessary for all arms of this statement
#1689: FILE: util/uri.c:1600:
+ if (val <= 9)
[...]
+ else
[...]
ERROR: braces {} are necessary for all arms of this statement
#1694: FILE: util/uri.c:1605:
+ if (val <= 9)
[...]
+ else
[...]
ERROR: return is not a function, parentheses are not required
#1705: FILE: util/uri.c:1615:
+ return (ret);
ERROR: braces {} are necessary for all arms of this statement
#1742: FILE: util/uri.c:1658:
+ if (*uri) {
[...]
+ } else
[...]
ERROR: braces {} are necessary for all arms of this statement
#1776: FILE: util/uri.c:1680:
+ if (ref)
[...]
ERROR: braces {} are necessary for all arms of this statement
#1806: FILE: util/uri.c:1710:
+ if (bas->authority != NULL)
[...]
+ else if (bas->server != NULL) {
[...]
ERROR: braces {} are necessary for all arms of this statement
#1846: FILE: util/uri.c:1750:
+ if (ref->authority != NULL)
[...]
+ else {
[...]
ERROR: braces {} are necessary even for single statement blocks
#1905: FILE: util/uri.c:1802:
+ while ((bas->path[cur] != 0) && (bas->path[cur] != '/'))
+ cur++;
ERROR: braces {} are necessary for all arms of this statement
#1907: FILE: util/uri.c:1804:
+ if (bas->path[cur] == 0)
[...]
ERROR: braces {} are necessary for all arms of this statement
#1942: FILE: util/uri.c:1825:
+ if ((out == 0) && (bas->server != NULL))
[...]
ERROR: return is not a function, parentheses are not required
#1963: FILE: util/uri.c:1854:
+ return (val);
ERROR: braces {} are necessary for all arms of this statement
#1995: FILE: util/uri.c:1912:
+ if (ret != 0)
[...]
ERROR: braces {} are necessary for all arms of this statement
#2017: FILE: util/uri.c:1927:
+ if (ret != 0)
[...]
ERROR: braces {} are necessary for all arms of this statement
#2073: FILE: util/uri.c:1965:
+ if (*uptr == '/')
[...]
ERROR: braces {} are necessary for all arms of this statement
#2089: FILE: util/uri.c:1975:
+ if (bptr[ix] == '/')
[...]
ERROR: braces {} are necessary for all arms of this statement
#2102: FILE: util/uri.c:1984:
+ if ((ref->path[pos] == '.') && (ref->path[pos + 1] == '/'))
[...]
ERROR: braces {} are necessary for all arms of this statement
#2105: FILE: util/uri.c:1986:
+ if ((*bptr == '.') && (bptr[1] == '/'))
[...]
- else if ((*bptr == '/') && (ref->path[pos] != '/'))
[...]
ERROR: braces {} are necessary for all arms of this statement
#2111: FILE: util/uri.c:1988:
+ else if ((*bptr == '/') && (ref->path[pos] != '/'))
[...]
ERROR: braces {} are necessary even for single statement blocks
#2113: FILE: util/uri.c:1990:
+ while ((bptr[pos] == ref->path[pos]) && (bptr[pos] != 0))
+ pos++;
ERROR: braces {} are necessary for all arms of this statement
#2149: FILE: util/uri.c:2003:
+ if ((ref->path[ix] == '/') && (ix > 0))
[...]
+ else if ((ref->path[ix] == 0) && (ix > 1) && (ref->path[ix - 1] == '/'))
[...]
ERROR: braces {} are necessary for all arms of this statement
#2151: FILE: util/uri.c:2005:
+ else if ((ref->path[ix] == 0) && (ix > 1) && (ref->path[ix - 1] == '/'))
[...]
ERROR: braces {} are necessary for all arms of this statement
#2154: FILE: util/uri.c:2008:
+ if (ref->path[ix] == '/')
[...]
ERROR: braces {} are necessary for all arms of this statement
#2179: FILE: util/uri.c:2023:
+ if (bptr[ix] == '/')
[...]
ERROR: braces {} are necessary for all arms of this statement
#2191: FILE: util/uri.c:2031:
+ if (uptr != NULL)
[...]
ERROR: braces {} are necessary for all arms of this statement
#2272: FILE: util/uri.c:2096:
+ if (init_alloc <= 0)
[...]
ERROR: braces {} are necessary for all arms of this statement
#2341: FILE: util/uri.c:2159:
+ if (eq && eq >= end)
[...]
ERROR: braces {} are necessary for all arms of this statement
#2383: FILE: util/uri.c:2199:
+ if (*query)
[...]
total: 147 errors, 0 warnings, 2299 lines checked
Your patch has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Checking PATCH 2/6: util/uri.c: remove brackets that wrap `return` statement's content....
Checking PATCH 3/6: util/uri.c: wrap single statement blocks with braces {}...
Checking PATCH 4/6: tests/libqos: Check for valid dev pointer when looking for PCI devices...
Checking PATCH 5/6: Add a git-publish configuration file...
Checking PATCH 6/6: README: Document 'git-publish' workflow...
=== OUTPUT END ===
Test command exited with code: 1
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@freelists.org
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 0/6] Block patches
@ 2018-03-05 9:40 Stefan Hajnoczi
2018-03-05 9:52 ` no-reply
2018-03-05 18:55 ` Peter Maydell
0 siblings, 2 replies; 18+ messages in thread
From: Stefan Hajnoczi @ 2018-03-05 9:40 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-block, Peter Maydell, Stefan Hajnoczi
The following changes since commit 136c67e07869227b21b3f627316e03679ce7b738:
Merge remote-tracking branch 'remotes/bkoppelmann/tags/pull-tricore-2018-03-02' into staging (2018-03-02 16:56:20 +0000)
are available in the Git repository at:
git://github.com/stefanha/qemu.git tags/block-pull-request
for you to fetch changes up to 23500c6a9409efc80d696aede0629bfbe7556a90:
README: Document 'git-publish' workflow (2018-03-05 09:03:17 +0000)
----------------------------------------------------------------
Pull request
Mostly patches that are only indirectly related to the block layer, but I've
reviewed them and there is no maintainer.
----------------------------------------------------------------
Fam Zheng (2):
Add a git-publish configuration file
README: Document 'git-publish' workflow
Su Hang (3):
util/uri.c: Coding style check, Only whitespace involved
util/uri.c: remove brackets that wrap `return` statement's content.
util/uri.c: wrap single statement blocks with braces {}
Thomas Huth (1):
tests/libqos: Check for valid dev pointer when looking for PCI devices
tests/libqos/virtio-pci.c | 4 +-
util/uri.c | 1733 ++++++++++++++++++++++++---------------------
.gitpublish | 51 ++
README | 31 +-
4 files changed, 1014 insertions(+), 805 deletions(-)
create mode 100644 .gitpublish
--
2.14.3
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 0/6] Block patches
@ 2013-12-13 17:48 Kevin Wolf
0 siblings, 0 replies; 18+ messages in thread
From: Kevin Wolf @ 2013-12-13 17:48 UTC (permalink / raw)
To: anthony; +Cc: kwolf, qemu-devel
The following changes since commit 47acdd63a33a5966bf4fc94a6ac835d72a70c555:
target-microblaze: Use the new qemu_ld/st opcodes (2013-12-13 10:00:42 +1000)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git tags/for-anthony
for you to fetch changes up to c547e5640d5b0993cdfb252331065c1a1d813bd8:
blkdebug: Use QLIST_FOREACH_SAFE to resume IO (2013-12-13 17:11:19 +0100)
----------------------------------------------------------------
Block patches
----------------------------------------------------------------
Fam Zheng (1):
blkdebug: Use QLIST_FOREACH_SAFE to resume IO
Kevin Wolf (1):
qapi-schema.json: Change 1.8 reference to 2.0
Liu Yuan (1):
sheepdog: check if '-o redundancy' is passed from user
Peter Lieven (2):
block: expect get_block_status errors in bdrv_make_zero
qemu-img: make progress output more accurate during convert
Stefan Weil (1):
block/vvfat: Fix compiler warnings for OpenBSD
block.c | 5 +++++
block/blkdebug.c | 8 ++++----
block/sheepdog.c | 8 +++++---
block/vvfat.c | 43 ++++++++++++++++++++++---------------------
qapi-schema.json | 2 +-
qemu-img.c | 26 +++++++++++++++++++++-----
6 files changed, 58 insertions(+), 34 deletions(-)
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [Qemu-devel] [PULL 0/6] Block patches
2013-01-14 10:01 Kevin Wolf
@ 2013-01-20 14:47 ` Anthony Liguori
0 siblings, 0 replies; 18+ messages in thread
From: Anthony Liguori @ 2013-01-20 14:47 UTC (permalink / raw)
To: Kevin Wolf, anthony; +Cc: qemu-devel
Pulled. Thanks.
Regards,
Anthony Liguori
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 0/6] Block patches
@ 2013-01-14 10:01 Kevin Wolf
2013-01-20 14:47 ` Anthony Liguori
0 siblings, 1 reply; 18+ messages in thread
From: Kevin Wolf @ 2013-01-14 10:01 UTC (permalink / raw)
To: anthony; +Cc: kwolf, qemu-devel
The following changes since commit 63fb2590839162afdf14d7c0ee02d460766c0956:
Merge branch 'target-arm.next' of git://git.linaro.org/people/pmaydell/qemu-arm (2013-01-12 12:47:07 +0000)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git for-anthony
Liu Yuan (1):
sheepdog: implement direct write semantics
Paolo Bonzini (2):
block: do not probe zero-sized disks
raw-posix: fix bdrv_aio_ioctl
Stefan Hajnoczi (3):
block: make qiov_is_aligned() public
dataplane: extract virtio-blk read/write processing into do_rdwr_cmd()
dataplane: handle misaligned virtio-blk requests
block.c | 18 +++++++++++-
block/raw-posix.c | 28 +----------------
block/sheepdog.c | 70 +++++++++++++++++++++++++-------------------
hw/dataplane/virtio-blk.c | 71 ++++++++++++++++++++++++++++++++++++++-------
include/block/block.h | 1 +
5 files changed, 120 insertions(+), 68 deletions(-)
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [Qemu-devel] [PULL 0/6] Block patches
2011-05-03 11:08 Kevin Wolf
@ 2011-05-03 12:44 ` Anthony Liguori
0 siblings, 0 replies; 18+ messages in thread
From: Anthony Liguori @ 2011-05-03 12:44 UTC (permalink / raw)
To: Kevin Wolf; +Cc: qemu-devel
On 05/03/2011 06:08 AM, Kevin Wolf wrote:
> The following changes since commit 57aa265d462a64a06268be26d49020729cff56c1:
>
> lm32: add Milkymist Minimac2 support (2011-05-03 10:48:40 +0200)
>
> are available in the git repository at:
> git://repo.or.cz/qemu/kevin.git for-anthony
Pulled. Thanks.
Regards,
Anthony Liguori
> Alon Levy (1):
> ide/atapi: fix set but unused
>
> Amit Shah (2):
> atapi: Move comment to proper place
> atapi: Explain why we need a 'media not present' state
>
> Jes Sorensen (1):
> qemu-progress.c: printf isn't signal safe
>
> Kevin Wolf (1):
> qemu-img resize: Fix option parsing
>
> Nick Thomas (1):
> NBD: Avoid leaking a couple of strings when the NBD device is closed
>
> block/nbd.c | 4 ++++
> hw/ide/atapi.c | 14 +++++++++-----
> qemu-img.c | 13 +++++++++++--
> qemu-progress.c | 7 ++++++-
> 4 files changed, 30 insertions(+), 8 deletions(-)
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 0/6] Block patches
@ 2011-05-03 11:08 Kevin Wolf
2011-05-03 12:44 ` Anthony Liguori
0 siblings, 1 reply; 18+ messages in thread
From: Kevin Wolf @ 2011-05-03 11:08 UTC (permalink / raw)
To: anthony; +Cc: kwolf, qemu-devel
The following changes since commit 57aa265d462a64a06268be26d49020729cff56c1:
lm32: add Milkymist Minimac2 support (2011-05-03 10:48:40 +0200)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git for-anthony
Alon Levy (1):
ide/atapi: fix set but unused
Amit Shah (2):
atapi: Move comment to proper place
atapi: Explain why we need a 'media not present' state
Jes Sorensen (1):
qemu-progress.c: printf isn't signal safe
Kevin Wolf (1):
qemu-img resize: Fix option parsing
Nick Thomas (1):
NBD: Avoid leaking a couple of strings when the NBD device is closed
block/nbd.c | 4 ++++
hw/ide/atapi.c | 14 +++++++++-----
qemu-img.c | 13 +++++++++++--
qemu-progress.c | 7 ++++++-
4 files changed, 30 insertions(+), 8 deletions(-)
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [Qemu-devel] [PULL 0/6] Block patches
2010-08-03 14:44 Kevin Wolf
@ 2010-08-09 13:41 ` Anthony Liguori
0 siblings, 0 replies; 18+ messages in thread
From: Anthony Liguori @ 2010-08-09 13:41 UTC (permalink / raw)
To: Kevin Wolf; +Cc: qemu-devel
On 08/03/2010 09:44 AM, Kevin Wolf wrote:
> The following changes since commit 5933e8a96ab9c59cb6b6c80c9db385364a68c959:
>
> fix last cpu timer initialization (2010-08-02 18:49:13 +0000)
>
Pulled, thanks.
Regards,
Anthony Liguori
> are available in the git repository at:
> git://repo.or.cz/qemu/kevin.git for-anthony
>
> Andrea Arcangeli (1):
> ide: Avoid canceling IDE DMA
>
> Kevin Wolf (2):
> block: Change bdrv_commit to handle multiple sectors at once
> block: Fix bdrv_has_zero_init
>
> Markus Armbruster (1):
> block: Change bdrv_eject() not to drop the image
>
> Miguel Di Ciurcio Filho (1):
> loadvm: improve tests before bdrv_snapshot_goto()
>
> Yoshiaki Tamura (1):
> block migration: replace tabs by spaces.
>
> block-migration.c | 12 ++++----
> block.c | 50 ++++++++++++++++++------------------
> block/raw-posix.c | 13 ++++++---
> block/raw-win32.c | 6 ++++
> block/raw.c | 6 ++++
> block_int.h | 8 ++++-
> hw/ide/pci.c | 23 +++++++++++++++-
> monitor.c | 3 +-
> savevm.c | 71 +++++++++++++++++++++++++---------------------------
> 9 files changed, 115 insertions(+), 77 deletions(-)
>
>
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* [Qemu-devel] [PULL 0/6] Block patches
@ 2010-08-03 14:44 Kevin Wolf
2010-08-09 13:41 ` Anthony Liguori
0 siblings, 1 reply; 18+ messages in thread
From: Kevin Wolf @ 2010-08-03 14:44 UTC (permalink / raw)
To: anthony; +Cc: kwolf, qemu-devel
The following changes since commit 5933e8a96ab9c59cb6b6c80c9db385364a68c959:
fix last cpu timer initialization (2010-08-02 18:49:13 +0000)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git for-anthony
Andrea Arcangeli (1):
ide: Avoid canceling IDE DMA
Kevin Wolf (2):
block: Change bdrv_commit to handle multiple sectors at once
block: Fix bdrv_has_zero_init
Markus Armbruster (1):
block: Change bdrv_eject() not to drop the image
Miguel Di Ciurcio Filho (1):
loadvm: improve tests before bdrv_snapshot_goto()
Yoshiaki Tamura (1):
block migration: replace tabs by spaces.
block-migration.c | 12 ++++----
block.c | 50 ++++++++++++++++++------------------
block/raw-posix.c | 13 ++++++---
block/raw-win32.c | 6 ++++
block/raw.c | 6 ++++
block_int.h | 8 ++++-
hw/ide/pci.c | 23 +++++++++++++++-
monitor.c | 3 +-
savevm.c | 71 +++++++++++++++++++++++++---------------------------
9 files changed, 115 insertions(+), 77 deletions(-)
^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2018-03-05 18:55 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-23 9:31 [Qemu-devel] [PULL 0/6] Block patches Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 1/6] block: m25p80: sync_page(): Deindent function body Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 2/6] block: m25p80: Support read only bdrvs Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 3/6] QemuOpts: check NULL opts in qemu_opt_get functions Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 4/6] sheepdog: fix NULL dereference in sd_create Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 5/6] vl: allow other threads to do qemu_system_vmstop_request Stefan Hajnoczi
2014-06-23 9:31 ` [Qemu-devel] [PULL 6/6] block: asynchronously stop the VM on I/O errors Stefan Hajnoczi
2014-06-23 12:18 ` [Qemu-devel] [PULL 0/6] Block patches Peter Maydell
-- strict thread matches above, loose matches on Subject: below --
2018-03-05 9:40 Stefan Hajnoczi
2018-03-05 9:52 ` no-reply
2018-03-05 18:55 ` Peter Maydell
2013-12-13 17:48 Kevin Wolf
2013-01-14 10:01 Kevin Wolf
2013-01-20 14:47 ` Anthony Liguori
2011-05-03 11:08 Kevin Wolf
2011-05-03 12:44 ` Anthony Liguori
2010-08-03 14:44 Kevin Wolf
2010-08-09 13:41 ` Anthony Liguori
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.