From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54613) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aLqnz-00047s-1N for qemu-devel@nongnu.org; Wed, 20 Jan 2016 06:20:45 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aLqmp-0001Os-Kp for qemu-devel@nongnu.org; Wed, 20 Jan 2016 06:20:02 -0500 Received: from mail-wm0-x22d.google.com ([2a00:1450:400c:c09::22d]:36947) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aLqmp-0001Om-8R for qemu-devel@nongnu.org; Wed, 20 Jan 2016 06:18:51 -0500 Received: by mail-wm0-x22d.google.com with SMTP id n5so23746642wmn.0 for ; Wed, 20 Jan 2016 03:18:51 -0800 (PST) Sender: Paolo Bonzini References: <1452761307-57200-1-git-send-email-frankja@linux.vnet.ibm.com> <1452761307-57200-4-git-send-email-frankja@linux.vnet.ibm.com> From: Paolo Bonzini Message-ID: <569F6D19.4040406@redhat.com> Date: Wed, 20 Jan 2016 12:18:49 +0100 MIME-Version: 1.0 In-Reply-To: <1452761307-57200-4-git-send-email-frankja@linux.vnet.ibm.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [RFC 3/5] scripts/dump-guest-memory.py: Improve python 3 compatibility List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Janosch Frank , qemu-devel@nongnu.org Cc: lersek@redhat.com On 14/01/2016 09:48, Janosch Frank wrote: > This commit does not make the script python 3 compatible, it is a > preparation that fixes the easy and common incompatibilities. > > Print is a function in python 3 and therefore needs braces around its > arguments. > > Range does not cast a gdb.Value object to int in python 3, we have to > do it ourselves. Would it make sense to make kvm_stat Py3-compatible too? Paolo > Signed-off-by: Janosch Frank > --- > scripts/dump-guest-memory.py | 22 +++++++++++----------- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git a/scripts/dump-guest-memory.py b/scripts/dump-guest-memory.py > index 76a6ecb..fe93135 100644 > --- a/scripts/dump-guest-memory.py > +++ b/scripts/dump-guest-memory.py > @@ -98,15 +98,15 @@ def memory_region_get_ram_ptr(mr): > > def get_guest_phys_blocks(): > guest_phys_blocks = [] > - print "guest RAM blocks:" > - print ("target_start target_end host_addr message " > - "count") > - print ("---------------- ---------------- ---------------- ------- " > - "-----") > + print("guest RAM blocks:") > + print("target_start target_end host_addr message " > + "count") > + print("---------------- ---------------- ---------------- ------- " > + "-----") > > current_map_p = gdb.parse_and_eval("address_space_memory.current_map") > current_map = current_map_p.dereference() > - for cur in range(current_map["nr"]): > + for cur in range(int(current_map["nr"])): > flat_range = (current_map["ranges"] + cur).dereference() > mr = flat_range["mr"].dereference() > > @@ -149,9 +149,9 @@ def get_guest_phys_blocks(): > predecessor["target_end"] = target_end > message = "joined" > > - print ("%016x %016x %016x %-7s %5u" % > - (target_start, target_end, host_addr.cast(UINTPTR_T), > - message, len(guest_phys_blocks))) > + print("%016x %016x %016x %-7s %5u" % > + (target_start, target_end, host_addr.cast(UINTPTR_T), > + message, len(guest_phys_blocks))) > > return guest_phys_blocks > > @@ -311,8 +311,8 @@ shape and this command should mostly work.""" > for block in self.guest_phys_blocks: > cur = block["host_addr"] > left = block["target_end"] - block["target_start"] > - print ("dumping range at %016x for length %016x" % > - (cur.cast(UINTPTR_T), left)) > + print("dumping range at %016x for length %016x" % > + (cur.cast(UINTPTR_T), left)) > while (left > 0): > chunk_size = min(TARGET_PAGE_SIZE, left) > chunk = qemu_core.read_memory(cur, chunk_size) >