Re: [Qemu-devel] [PATCH] memory: info mtree check mr range overflow

2017-03-15 Thread Mark Cave-Ayland
On 15/03/17 04:23, Michael S. Tsirkin wrote: Could you help elaborate in what case will we really need that 128 bit address? >>> >>> This is how memory API works. It uses 128 bit addresses (in reality >>> it typically only needs 64 bit addresses but 128 means it can do >>> math without w

Re: [Qemu-devel] [PATCH] memory: info mtree check mr range overflow

2017-03-14 Thread Michael S. Tsirkin
On Wed, Mar 15, 2017 at 12:04:27PM +0800, Peter Xu wrote: > On Wed, Mar 15, 2017 at 05:30:56AM +0200, Michael S. Tsirkin wrote: > > On Wed, Mar 15, 2017 at 11:15:50AM +0800, Peter Xu wrote: > > > On Wed, Mar 15, 2017 at 03:24:04AM +0200, Michael S. Tsirkin wrote: > > > > On Tue, Mar 14, 2017 at 08:

Re: [Qemu-devel] [PATCH] memory: info mtree check mr range overflow

2017-03-14 Thread Peter Xu
On Wed, Mar 15, 2017 at 05:30:56AM +0200, Michael S. Tsirkin wrote: > On Wed, Mar 15, 2017 at 11:15:50AM +0800, Peter Xu wrote: > > 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 regio

Re: [Qemu-devel] [PATCH] memory: info mtree check mr range overflow

2017-03-14 Thread Michael S. Tsirkin
On Wed, Mar 15, 2017 at 11:15:50AM +0800, Peter Xu wrote: > 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: > >

Re: [Qemu-devel] [PATCH] memory: info mtree check mr range overflow

2017-03-14 Thread Peter Xu
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/msg020

Re: [Qemu-devel] [PATCH] memory: info mtree check mr range overflow

2017-03-14 Thread Michael S. Tsirkin
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: M

Re: [Qemu-devel] [PATCH] memory: info mtree check mr range overflow

2017-03-14 Thread Michael S. Tsirkin
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: M

[Qemu-devel] [PATCH] memory: info mtree check mr range overflow

2017-03-14 Thread Peter Xu
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 Signed-off-by: Peter Xu --- memory.c | 21 +