From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49074) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VjUiE-00061W-Hs for qemu-devel@nongnu.org; Thu, 21 Nov 2013 08:54:38 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VjUi6-00038k-4a for qemu-devel@nongnu.org; Thu, 21 Nov 2013 08:54:30 -0500 Received: from mail-qc0-x22b.google.com ([2607:f8b0:400d:c01::22b]:36184) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VjUi6-00038f-09 for qemu-devel@nongnu.org; Thu, 21 Nov 2013 08:54:22 -0500 Received: by mail-qc0-f171.google.com with SMTP id s13so6785866qcv.16 for ; Thu, 21 Nov 2013 05:54:21 -0800 (PST) Sender: Paolo Bonzini Message-ID: <528E1089.6090401@redhat.com> Date: Thu, 21 Nov 2013 14:54:17 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1385036128-8753-1-git-send-email-mst@redhat.com> <1385036128-8753-2-git-send-email-mst@redhat.com> In-Reply-To: <1385036128-8753-2-git-send-email-mst@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH for-1.7 2/2] acpi-build: fix build on glib < 2.14 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: qemu-devel@nongnu.org, Anthony Liguori , Richard Henderson Il 21/11/2013 13:17, Michael S. Tsirkin ha scritto: > g_array_get_element_size was only added in glib 2.14. > Fortunately we don't use it for any arrays where > element size is > 1, so just add an assert. > > Reported-by: Richard Henderson > Signed-off-by: Michael S. Tsirkin > --- > hw/i386/acpi-build.c | 5 ++++- > hw/i386/bios-linker-loader.c | 8 ++++---- > 2 files changed, 8 insertions(+), 5 deletions(-) > > diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c > index 59a17df..5f36e7e 100644 > --- a/hw/i386/acpi-build.c > +++ b/hw/i386/acpi-build.c > @@ -425,7 +425,10 @@ static inline void *acpi_data_push(GArray *table_data, unsigned size) > > static unsigned acpi_data_len(GArray *table) > { > - return table->len * g_array_get_element_size(table); > +#if GLIB_CHECK_VERSION(2, 14, 0) > + assert(g_array_get_element_size(table) == 1); > +#endif > + return table->len; > } > > static void acpi_align_size(GArray *blob, unsigned align) This looks good, but is the below part necessary? It's ugly! Paolo > diff --git a/hw/i386/bios-linker-loader.c b/hw/i386/bios-linker-loader.c > index 0833853..fd23611 100644 > --- a/hw/i386/bios-linker-loader.c > +++ b/hw/i386/bios-linker-loader.c > @@ -90,7 +90,7 @@ enum { > > GArray *bios_linker_loader_init(void) > { > - return g_array_new(false, true /* clear */, sizeof(BiosLinkerLoaderEntry)); > + return g_array_new(false, true /* clear */, 1); > } > > /* Free linker wrapper and return the linker array. */ > @@ -115,7 +115,7 @@ void bios_linker_loader_alloc(GArray *linker, > BIOS_LINKER_LOADER_ALLOC_ZONE_HIGH); > > /* Alloc entries must come first, so prepend them */ > - g_array_prepend_val(linker, entry); > + g_array_prepend_vals(linker, &entry, sizeof entry); > } > > void bios_linker_loader_add_checksum(GArray *linker, const char *file, > @@ -132,7 +132,7 @@ void bios_linker_loader_add_checksum(GArray *linker, const char *file, > entry.cksum.start = cpu_to_le32((uint8_t *)start - (uint8_t *)table); > entry.cksum.length = cpu_to_le32(size); > > - g_array_append_val(linker, entry); > + g_array_append_vals(linker, &entry, sizeof entry); > } > > void bios_linker_loader_add_pointer(GArray *linker, > @@ -154,5 +154,5 @@ void bios_linker_loader_add_pointer(GArray *linker, > assert(pointer_size == 1 || pointer_size == 2 || > pointer_size == 4 || pointer_size == 8); > > - g_array_append_val(linker, entry); > + g_array_append_vals(linker, &entry, sizeof entry); > } >