[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/3] osdep: Rename qemu_{get, set}_version() to
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH 2/3] osdep: Rename qemu_{get, set}_version() to qemu_{, set_}hw_version() |
Date: |
Tue, 22 Sep 2015 16:21:47 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 |
On 09/22/2015 04:16 PM, Eduardo Habkost wrote:
> This makes the purpose of the function clearer: it is not about the
> version of QEMU that's running, but the version string exposed in the
> emulated hardware.
>
> Cc: Andrzej Zaborowski <address@hidden>
> Cc: Peter Maydell <address@hidden>
> Cc: John Snow <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> hw/arm/nseries.c | 2 +-
> hw/ide/core.c | 2 +-
> hw/scsi/scsi-bus.c | 2 +-
> hw/scsi/scsi-disk.c | 2 +-
> include/qemu/osdep.h | 4 ++--
> target-i386/cpu.c | 2 +-
> util/osdep.c | 10 +++++-----
> vl.c | 2 +-
> 8 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/hw/arm/nseries.c b/hw/arm/nseries.c
> index a659e85..ce05ffc 100644
> --- a/hw/arm/nseries.c
> +++ b/hw/arm/nseries.c
> @@ -1275,7 +1275,7 @@ static int n8x0_atag_setup(void *p, int model)
> strcpy((void *) w, "hw-build"); /* char component[12] */
> w += 6;
> strcpy((void *) w, "QEMU ");
> - pstrcat((void *) w, 12, qemu_get_version()); /* char version[12] */
> + pstrcat((void *) w, 12, qemu_hw_version()); /* char version[12] */
> w += 6;
>
> tag = (model == 810) ? "1.1.10-qemu" : "1.1.6-qemu";
> diff --git a/hw/ide/core.c b/hw/ide/core.c
> index 50449ca..c503c69 100644
> --- a/hw/ide/core.c
> +++ b/hw/ide/core.c
> @@ -2313,7 +2313,7 @@ int ide_init_drive(IDEState *s, BlockBackend *blk,
> IDEDriveKind kind,
> if (version) {
> pstrcpy(s->version, sizeof(s->version), version);
> } else {
> - pstrcpy(s->version, sizeof(s->version), qemu_get_version());
> + pstrcpy(s->version, sizeof(s->version), qemu_hw_version());
> }
>
> ide_reset(s);
> diff --git a/hw/scsi/scsi-bus.c b/hw/scsi/scsi-bus.c
> index ffac8f4..3997007 100644
> --- a/hw/scsi/scsi-bus.c
> +++ b/hw/scsi/scsi-bus.c
> @@ -453,7 +453,7 @@ static bool scsi_target_emulate_inquiry(SCSITargetReq *r)
> r->buf[7] = 0x10 | (r->req.bus->info->tcq ? 0x02 : 0); /* Sync, TCQ.
> */
> memcpy(&r->buf[8], "QEMU ", 8);
> memcpy(&r->buf[16], "QEMU TARGET ", 16);
> - pstrcpy((char *) &r->buf[32], 4, qemu_get_version());
> + pstrcpy((char *) &r->buf[32], 4, qemu_hw_version());
> }
> return true;
> }
> diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
> index bada9a7..707e734 100644
> --- a/hw/scsi/scsi-disk.c
> +++ b/hw/scsi/scsi-disk.c
> @@ -2315,7 +2315,7 @@ static void scsi_realize(SCSIDevice *dev, Error **errp)
> }
>
> if (!s->version) {
> - s->version = g_strdup(qemu_get_version());
> + s->version = g_strdup(qemu_hw_version());
> }
> if (!s->vendor) {
> s->vendor = g_strdup("QEMU");
> diff --git a/include/qemu/osdep.h b/include/qemu/osdep.h
> index ab3c876..b4ee266 100644
> --- a/include/qemu/osdep.h
> +++ b/include/qemu/osdep.h
> @@ -243,8 +243,8 @@ static inline void qemu_timersub(const struct timeval
> *val1,
>
> void qemu_set_cloexec(int fd);
>
> -void qemu_set_version(const char *);
> -const char *qemu_get_version(void);
> +void qemu_set_hw_version(const char *);
> +const char *qemu_hw_version(void);
>
> void fips_set_state(bool requested);
> bool fips_get_state(void);
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> index 7c52714..f9284d5 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -2231,7 +2231,7 @@ void x86_cpudef_setup(void)
> pstrcpy(def->model_id, sizeof(def->model_id),
> "QEMU Virtual CPU version ");
> pstrcat(def->model_id, sizeof(def->model_id),
> - qemu_get_version());
> + qemu_hw_version());
> break;
> }
> }
> diff --git a/util/osdep.c b/util/osdep.c
> index 0092bb6..80c6bfe 100644
> --- a/util/osdep.c
> +++ b/util/osdep.c
> @@ -52,7 +52,7 @@ extern int madvise(caddr_t, size_t, int);
>
> static bool fips_enabled = false;
>
> -static const char *qemu_version = QEMU_VERSION;
> +static const char *hw_version = QEMU_VERSION;
>
> int socket_set_cork(int fd, int v)
> {
> @@ -311,14 +311,14 @@ int qemu_accept(int s, struct sockaddr *addr, socklen_t
> *addrlen)
> return ret;
> }
>
> -void qemu_set_version(const char *version)
> +void qemu_set_hw_version(const char *version)
> {
> - qemu_version = version;
> + hw_version = version;
> }
>
> -const char *qemu_get_version(void)
> +const char *qemu_hw_version(void)
> {
> - return qemu_version;
> + return hw_version;
> }
>
> void fips_set_state(bool requested)
> diff --git a/vl.c b/vl.c
> index 3c6480d..204fb08 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -4078,7 +4078,7 @@ int main(int argc, char **argv, char **envp)
> cpu_exec_init_all();
>
> if (machine_class->hw_version) {
> - qemu_set_version(machine_class->hw_version);
> + qemu_set_hw_version(machine_class->hw_version);
> }
>
> /* Init CPU def lists, based on config
>
Reviewed-by: John Snow <address@hidden>
- [Qemu-devel] [PATCH 0/3] pc: Set hw_version on all machine classes, Eduardo Habkost, 2015/09/22
- [Qemu-devel] [PATCH 1/3] pc: Set hw_version on all machine classes, Eduardo Habkost, 2015/09/22
- [Qemu-devel] [PATCH 2/3] osdep: Rename qemu_{get, set}_version() to qemu_{, set_}hw_version(), Eduardo Habkost, 2015/09/22
- Re: [Qemu-devel] [PATCH 2/3] osdep: Rename qemu_{get, set}_version() to qemu_{, set_}hw_version(),
John Snow <=
- [Qemu-devel] [PATCH 3/3] megasas: Use qemu_hw_version() instead of QEMU_VERSION, Eduardo Habkost, 2015/09/22
- Re: [Qemu-devel] [PATCH 0/3] pc: Set hw_version on all machine classes, Igor Mammedov, 2015/09/23