On Wed, Mar 15, 2017 at 03:24:04AM +0200, Michael S. Tsirkin wrote: > On Tue, Mar 14, 2017 at 08:56:27PM +0800, Peter Xu wrote: > > The address of memory regions might overflow when something wrong > > happened, like reported in: > > > > https://lists.gnu.org/archive/html/qemu-devel/2017-03/msg02043.html > > > > For easier debugging, let's try to detect it. > > > > Reported-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> > > Signed-off-by: Peter Xu <pet...@redhat.com> > > > After a chat with Paolo, I think the following is a more general fix > > - fix info mtree to do 128 bit math and display more than > 16 digits if necessary
Could you help elaborate in what case will we really need that 128 bit address? Btw, thanks for pointing out in the other thread that your patch wasn't printing 128 bits but 64 bits, actually I didn't notice that before... but even with that, I would still slightly prefer this one though considering readability and simplicity. > - add info about region visibility > how much info is appropriate is arguable - after all we already have info > mtree -f > we probably should report if region is not visible at all, > how about partially occluded ones? listing all windows is probably not > needed - we have the -f flag for that. For me, "info mtree" and its "-f" form work good enough. So I'll leave the discussion on this one to people who know better than me... Thanks, -- peterx