From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60609) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dTUpp-0002ew-Oh for qemu-devel@nongnu.org; Fri, 07 Jul 2017 11:06:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dTUpl-0005mI-47 for qemu-devel@nongnu.org; Fri, 07 Jul 2017 11:06:21 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46270) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dTUpk-0005lt-RX for qemu-devel@nongnu.org; Fri, 07 Jul 2017 11:06:17 -0400 Date: Fri, 7 Jul 2017 16:06:12 +0100 From: "Dr. David Alan Gilbert" Message-ID: <20170707150611.GJ2451@work-vm> References: <1499429683-73361-1-git-send-email-agraf@suse.de> <20170707150011.GH2451@work-vm> <1b6bf5d1-8477-1697-16e9-091c1b762bd4@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1b6bf5d1-8477-1697-16e9-091c1b762bd4@suse.de> Subject: Re: [Qemu-devel] [PATCH] migration: Make analyze-migration script target-page-size aware List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: qemu-devel@nongnu.org, Juan Quintela * Alexander Graf (agraf@suse.de) wrote: > > > On 07.07.17 17:00, Dr. David Alan Gilbert wrote: > > * Alexander Graf (agraf@suse.de) wrote: > > > The configuration section has a new subsection to transmit the target page > > > size along with the migration stream. The analyze migration script needs > > > to learn about that to read configuration streams that were triggering > > > this subsection to get transmitted. > > > > > > With this patch applied, I can successfully analyze migration streams > > > on AArch64 again. > > > > > > Signed-off-by: Alexander Graf > > > --- > > > scripts/analyze-migration.py | 13 +++++++++++++ > > > 1 file changed, 13 insertions(+) > > > > > > diff --git a/scripts/analyze-migration.py b/scripts/analyze-migration.py > > > index 1455387..02784f2 100755 > > > --- a/scripts/analyze-migration.py > > > +++ b/scripts/analyze-migration.py > > > @@ -254,12 +254,25 @@ class HTABSection(object): > > > > (Note I'm not a particularly python person, so take lightly) > > > > > class ConfigurationSection(object): > > > + QEMU_VM_SUBSECTION = 0x05 > > > + > > > > It's odd, you already have this constant defined twice in this script. > > Yes, it lives once per class. I am not sure how to easily make it a global. > > > > > > def __init__(self, file): > > > self.file = file > > > def read(self): > > > name_len = self.file.read32() > > > name = self.file.readstr(len = name_len) > > > + oldpos = self.file.tell() > > > + if self.file.read8() == self.QEMU_VM_SUBSECTION: > > > + name = self.file.readstr() > > > + version_id = self.file.read32() > > > + if name == "configuration/target-page-bits": > > > + target_page_size = self.file.read32() > > > > All of your other references to target_page_size in the script > > are self.TARGET_PAGE_SIZE. > > Well, all other ones are actual page sizes :). This variable is never used > anywhere - we just want to stash it somewhere. oh, so you're not actually using the value read here to influence the parsing later? > > You might want to make the conditional subsection check into a function > > somewhere, but that's OK for now. > > I really hope we don't need it again. All subsections should be described > via the in-stream json description. We just missed out the configuration one > because it's not part of the object model. OK. Dave > > > Alex -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK