Re: [Qemu-devel] [PATCH] guest-agent: fix build with OpenBSD

[Blue Swirl]

On Fri, Jul 22, 2011 at 10:15 PM, Anthony Liguori <address@hidden> wrote:
> FS-Freeze only works with Linux.
>
> Signed-off-by: Anthony Liguori <address@hidden>

ACK except for the unrelated stuff.

It's strange how the messages arrive several hours out of order.

> ---
>  Makefile                   |    2 +
>  hw/pc.c                    |    3 +-
>  qemu-char.c                |   36 +++++++++++++++++++++++++
>  qemu-config.c              |    6 ++++
>  qga/guest-agent-commands.c |   63 ++++++++++++++++++++++++++++++++++++-------
>  5 files changed, 97 insertions(+), 13 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index f3a03ad..2e3231e 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -188,6 +188,8 @@ $(qapi-dir)/qga-qmp-marshal.c: 
> $(SRC_PATH)/qapi-schema-guest.json $(SRC_PATH)/sc
>  test-visitor.o: $(addprefix $(qapi-dir)/, test-qapi-types.c 
> test-qapi-types.h test-qapi-visit.c test-qapi-visit.h) $(qapi-obj-y)
>  test-visitor: test-visitor.o qfloat.o qint.o qdict.o qstring.o qlist.o 
> qbool.o $(qapi-obj-y) error.o osdep.o qemu-malloc.o $(oslib-obj-y) qjson.o 
> json-streamer.o json-lexer.o json-parser.o qerror.o qemu-error.o qemu-tool.o 
> $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o
>
> +test-linux: test-linux.o
> +
>  test-qmp-commands.o: $(addprefix $(qapi-dir)/, test-qapi-types.c 
> test-qapi-types.h test-qapi-visit.c test-qapi-visit.h test-qmp-marshal.c 
> test-qmp-commands.h) $(qapi-obj-y)
>  test-qmp-commands: test-qmp-commands.o qfloat.o qint.o qdict.o qstring.o 
> qlist.o qbool.o $(qapi-obj-y) error.o osdep.o qemu-malloc.o $(oslib-obj-y) 
> qjson.o json-streamer.o json-lexer.o json-parser.o qerror.o qemu-error.o 
> qemu-tool.o $(qapi-dir)/test-qapi-visit.o $(qapi-dir)/test-qapi-types.o 
> $(qapi-dir)/test-qmp-marshal.o module.o
>
> diff --git a/hw/pc.c b/hw/pc.c
> index a3e8539..2539372 100644
> --- a/hw/pc.c
> +++ b/hw/pc.c
> @@ -548,8 +548,7 @@ static void bochs_bios_write(void *opaque, uint32_t addr, 
> uint32_t val)
>         /* LGPL'ed VGA BIOS messages */
>     case 0x501:
>     case 0x502:
> -        fprintf(stderr, "VGA BIOS panic, line %d\n", val);
> -        exit(1);
> +        exit(1 | ((val & 0x7F) << 1));
>     case 0x500:
>     case 0x503:
>  #ifdef DEBUG_BIOS
> diff --git a/qemu-char.c b/qemu-char.c
> index fb13b28..8c18653 100644
> --- a/qemu-char.c
> +++ b/qemu-char.c
> @@ -634,6 +634,41 @@ static CharDriverState *qemu_chr_open_fd(int fd_in, int 
> fd_out)
>     return chr;
>  }
>
> +static int handle_fd_param(QemuOpts *opts, const char *key)
> +{
> +    const char *value;
> +
> +    value = qemu_opt_get(opts, key);
> +    if (value == NULL) {
> +        return -1;
> +    }
> +
> +    if (qemu_isdigit(value[0])) {
> +        return atoi(value);
> +    }
> +
> +    return monitor_get_fd(cur_mon, value);
> +}
> +
> +static CharDriverState *qemu_chr_open_fdname(QemuOpts *opts)
> +{
> +    CharDriverState *chr;
> +    FDCharDriver *s;
> +
> +    chr = qemu_mallocz(sizeof(CharDriverState));
> +    s = qemu_mallocz(sizeof(FDCharDriver));
> +    s->fd_in = handle_fd_param(opts, "fdin");
> +    s->fd_out = handle_fd_param(opts, "fdout");
> +    chr->opaque = s;
> +    chr->chr_write = fd_chr_write;
> +    chr->chr_update_read_handler = fd_chr_update_read_handler;
> +    chr->chr_close = fd_chr_close;
> +
> +    qemu_chr_generic_open(chr);
> +
> +    return chr;
> +}
> +
>  static CharDriverState *qemu_chr_open_file_out(QemuOpts *opts)
>  {
>     int fd_out;
> @@ -2483,6 +2518,7 @@ static const struct {
>     { .name = "pipe",      .open = qemu_chr_open_pipe },
>     { .name = "pty",       .open = qemu_chr_open_pty },
>     { .name = "stdio",     .open = qemu_chr_open_stdio },
> +    { .name = "fdname",    .open = qemu_chr_open_fdname },
>  #endif
>  #ifdef CONFIG_BRLAPI
>     { .name = "braille",   .open = chr_baum_init },
> diff --git a/qemu-config.c b/qemu-config.c
> index 93d20c6..ff6730d 100644
> --- a/qemu-config.c
> +++ b/qemu-config.c
> @@ -157,6 +157,12 @@ static QemuOptsList qemu_chardev_opts = {
>         },{
>             .name = "debug",
>             .type = QEMU_OPT_NUMBER,
> +        },{
> +            .name = "fdin",
> +            .type = QEMU_OPT_STRING,
> +        },{
> +            .name = "fdout",
> +            .type = QEMU_OPT_STRING,
>         },
>         { /* end of list */ }
>     },
> diff --git a/qga/guest-agent-commands.c b/qga/guest-agent-commands.c
> index 8c0d67e..e215bd3 100644
> --- a/qga/guest-agent-commands.c
> +++ b/qga/guest-agent-commands.c
> @@ -10,11 +10,17 @@
>  * See the COPYING file in the top-level directory.
>  */
>
> +#if defined(__linux__)
> +#define CONFIG_FSFREEZE
> +#endif
> +
>  #include <glib.h>
> +#if defined(CONFIG_FSFREEZE)
>  #include <mntent.h>
> +#include <linux/fs.h>
> +#endif
>  #include <sys/types.h>
>  #include <sys/ioctl.h>
> -#include <linux/fs.h>
>  #include "qga/guest-agent-core.h"
>  #include "qga-qmp-commands.h"
>  #include "qerror.h"
> @@ -22,16 +28,6 @@
>
>  static GAState *ga_state;
>
> -static void disable_logging(void)
> -{
> -    ga_disable_logging(ga_state);
> -}
> -
> -static void enable_logging(void)
> -{
> -    ga_enable_logging(ga_state);
> -}
> -
>  /* Note: in some situations, like with the fsfreeze, logging may be
>  * temporarilly disabled. if it is necessary that a command be able
>  * to log for accounting purposes, check ga_logging_enabled() beforehand,
> @@ -323,6 +319,17 @@ static void guest_file_init(void)
>     QTAILQ_INIT(&guest_file_state.filehandles);
>  }
>
> +#if defined(CONFIG_FSFREEZE)
> +static void disable_logging(void)
> +{
> +    ga_disable_logging(ga_state);
> +}
> +
> +static void enable_logging(void)
> +{
> +    ga_enable_logging(ga_state);
> +}
> +
>  typedef struct GuestFsfreezeMount {
>     char *dirname;
>     char *devtype;
> @@ -508,11 +515,45 @@ static void guest_fsfreeze_cleanup(void)
>         }
>     }
>  }
> +#else
> +/*
> + * Return status of freeze/thaw
> + */
> +GuestFsfreezeStatus qmp_guest_fsfreeze_status(Error **err)
> +{
> +    error_set(err, QERR_COMMAND_NOT_FOUND, "guest_fsfreeze_status");
> +
> +    return 0;
> +}
> +
> +/*
> + * Walk list of mounted file systems in the guest, and freeze the ones which
> + * are real local file systems.
> + */
> +int64_t qmp_guest_fsfreeze_freeze(Error **err)
> +{
> +    error_set(err, QERR_COMMAND_NOT_FOUND, "guest_fsfreeze_freeze");
> +
> +    return 0;
> +}
> +
> +/*
> + * Walk list of frozen file systems in the guest, and thaw them.
> + */
> +int64_t qmp_guest_fsfreeze_thaw(Error **err)
> +{
> +    error_set(err, QERR_COMMAND_NOT_FOUND, "guest_fsfreeze_thaw");
> +
> +    return 0;
> +}
> +#endif
>
>  /* register init/cleanup routines for stateful command groups */
>  void ga_command_state_init(GAState *s, GACommandState *cs)
>  {
>     ga_state = s;
> +#if defined(CONFIG_FSFREEZE)
>     ga_command_state_add(cs, guest_fsfreeze_init, guest_fsfreeze_cleanup);
> +#endif
>     ga_command_state_add(cs, guest_file_init, NULL);
>  }
> --
> 1.7.4.1
>
>