From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40975) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WNLIu-0001mx-L0 for qemu-devel@nongnu.org; Tue, 11 Mar 2014 07:57:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WNLIm-0002hp-7g for qemu-devel@nongnu.org; Tue, 11 Mar 2014 07:57:04 -0400 Received: from mail-ea0-x233.google.com ([2a00:1450:4013:c01::233]:50948) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WNLIl-0002hF-VU for qemu-devel@nongnu.org; Tue, 11 Mar 2014 07:56:56 -0400 Received: by mail-ea0-f179.google.com with SMTP id q10so4296937ead.38 for ; Tue, 11 Mar 2014 04:56:54 -0700 (PDT) Message-ID: <1394539033.3981.27.camel@localhost.localdomain> From: Marcel Apfelbaum Date: Tue, 11 Mar 2014 13:57:13 +0200 In-Reply-To: <1394388381-22813-1-git-send-email-mst@redhat.com> References: <1394388381-22813-1-git-send-email-mst@redhat.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 1/3] loader: rename in_ram/has_mr Reply-To: marcel.a@redhat.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: jeff.nelson@redhat.com, qemu-devel@nongnu.org, dgilbert@redhat.com, Anthony Liguori , pbonzini@redhat.com, lersek@redhat.com On Sun, 2014-03-09 at 20:06 +0200, Michael S. Tsirkin wrote: > we put copy of ROMs in MR for migration. > but the name rom_in_ram makes one think we > load it in guest RAM. > Rename has_mr to make intent clearer. > > Signed-off-by: Michael S. Tsirkin > --- > include/hw/loader.h | 2 +- > hw/core/loader.c | 6 +++--- > hw/i386/pc_piix.c | 2 +- > hw/i386/pc_q35.c | 2 +- > 4 files changed, 6 insertions(+), 6 deletions(-) Hi, Series Reviewed-by: Marcel Apfelbaum Thanks, Marcel > > diff --git a/include/hw/loader.h b/include/hw/loader.h > index aaf08c3..3dc5b94 100644 > --- a/include/hw/loader.h > +++ b/include/hw/loader.h > @@ -49,7 +49,7 @@ void pstrcpy_targphys(const char *name, > hwaddr dest, int buf_size, > const char *source); > > -extern bool rom_file_in_ram; > +extern bool rom_file_has_mr; > > int rom_add_file(const char *file, const char *fw_dir, > hwaddr addr, int32_t bootindex); > diff --git a/hw/core/loader.c b/hw/core/loader.c > index b323c0c..13e98d8 100644 > --- a/hw/core/loader.c > +++ b/hw/core/loader.c > @@ -54,7 +54,7 @@ > > #include > > -bool rom_file_in_ram = true; > +bool rom_file_has_mr = true; > > static int roms_loaded; > > @@ -694,7 +694,7 @@ int rom_add_file(const char *file, const char *fw_dir, > basename); > snprintf(devpath, sizeof(devpath), "/rom@%s", fw_file_name); > > - if (rom_file_in_ram) { > + if (rom_file_has_mr) { > data = rom_set_mr(rom, OBJECT(fw_cfg), devpath); > } else { > data = rom->data; > @@ -738,7 +738,7 @@ void *rom_add_blob(const char *name, const void *blob, size_t len, > > snprintf(devpath, sizeof(devpath), "/rom@%s", fw_file_name); > > - if (rom_file_in_ram) { > + if (rom_file_has_mr) { > data = rom_set_mr(rom, OBJECT(fw_cfg), devpath); > } else { > data = rom->data; > diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c > index ae1699d..fb2d636 100644 > --- a/hw/i386/pc_piix.c > +++ b/hw/i386/pc_piix.c > @@ -272,7 +272,7 @@ static void pc_compat_1_6(QEMUMachineInitArgs *args) > { > pc_compat_1_7(args); > has_pci_info = false; > - rom_file_in_ram = false; > + rom_file_has_mr = false; > has_acpi_build = false; > } > > diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c > index a7f6260..eb55ae4 100644 > --- a/hw/i386/pc_q35.c > +++ b/hw/i386/pc_q35.c > @@ -250,7 +250,7 @@ static void pc_compat_1_6(QEMUMachineInitArgs *args) > { > pc_compat_1_7(args); > has_pci_info = false; > - rom_file_in_ram = false; > + rom_file_has_mr = false; > has_acpi_build = false; > } >