[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v4 3/4] dump: do not dump non-existent guest memor
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-ppc] [PATCH v4 3/4] dump: do not dump non-existent guest memory |
Date: |
Thu, 14 Sep 2017 10:48:38 +0100 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
* Laurent Vivier (address@hidden) wrote:
> On 13/09/2017 16:27, Cornelia Huck wrote:
> > On Wed, 13 Sep 2017 16:20:35 +0200
> > Laurent Vivier <address@hidden> wrote:
> >
> >> From: Cornelia Huck <address@hidden>
> >>
> >> It does not really make sense to dump memory that is not there.
> >>
> >> Moreover, that fixes a segmentation fault when calling dump-guest-memory
> >> with no filter for a machine with no memory defined.
> >>
> >> New behaviour is:
> >>
> >> (qemu) dump-guest-memory /dev/null
> >> dump: no guest memory to dump
> >> (qemu) dump-guest-memory /dev/null 0 4096
> >> dump: no guest memory to dump
> >>
> >> Signed-off-by: Cornelia Huck <address@hidden>
> >> Tested-by: Laurent Vivier <address@hidden>
> >> Reviewed-by: Laurent Vivier <address@hidden>
> >> Reviewed-by: Greg Kurz <address@hidden>
> >> Reviewed-by: Peter Xu <address@hidden>
> >> ---
> >> dump.c | 6 ++++++
> >> 1 file changed, 6 insertions(+)
> >
> > You need to supply your s-o-b as well, no?
> >
>
> I was wondering... theoretically, yes, so:
>
> Signed-off-by: Laurent Vivier <address@hidden>
Thanks.
Dave
> Thanks,
> Laurent
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [Qemu-ppc] [PATCH v4 0/4] hmp: fix "dump-quest-memory" segfault, Laurent Vivier, 2017/09/13
- [Qemu-ppc] [PATCH v4 1/4] hmp: fix "dump-quest-memory" segfault (ppc), Laurent Vivier, 2017/09/13
- [Qemu-ppc] [PATCH v4 2/4] hmp: fix "dump-quest-memory" segfault (arm), Laurent Vivier, 2017/09/13
- [Qemu-ppc] [PATCH v4 4/4] tests/hmp: test "none" machine with memory, Laurent Vivier, 2017/09/13
- Re: [Qemu-ppc] [PATCH v4 0/4] hmp: fix "dump-quest-memory" segfault, Dr. David Alan Gilbert, 2017/09/14