Re: [PATCH v3 qemu 0/3] Allow dump-guest-memory to output standard kdump format

Daniel P. Berrangé <[email protected]>
Newsgroups org.kernel.vger.linux-debuggers,org.nongnu.qemu-devel
Message-ID <[email protected]>
On Tue, Sep 19, 2023 at 11:53:49AM +0400, Marc-André Lureau wrote:
> Hi
> 
> On Tue, Sep 19, 2023 at 3:32 AM Stephen Brennan
> <[email protected]> wrote:
> >
> > Hello all,
> >
> > This is the third version of the kdump patch series, the first and
> > second revisions being visible at [1] and [2] respectively. You can see
> > the history and motivation for the patch series described in the cover
> > letter of [2].
> >
> > Thank you for your continued feedback and review.
> > Stephen
> >
> > Changes from v2 to v3:
> > - Rather than use "reassembled" flag in QMP API, represent each kdump
> >   format "kdump-X" with a new enumerator "kdump-raw-X". (The HMP
> >   interface retains the -R option)
> > - Return an error if the file descriptor passed in is not seekable, yet
> >   the requested dump format is kdump-raw-*
> >
> > Changes from v1 to v2:
> > - Keep the default as the flattened format
> > - Add QMP / HMP arguments for "reassembled"
> >
> > [1]: https://lore.kernel.org/qemu-devel/[email protected]/
> > [2]: https://lore.kernel.org/qemu-devel/[email protected]/
> >
> > Stephen Brennan (3):
> >   dump: Pass DumpState to write_ functions
> >   dump: Allow directly outputting raw kdump format
> >   dump: Add command interface for kdump-raw formats
> >
> >  dump/dump-hmp-cmds.c  | 21 +++++++--
> >  dump/dump.c           | 99 +++++++++++++++++++++++++++++++------------
> >  hmp-commands.hx       |  9 +++-
> >  include/sysemu/dump.h |  3 +-
> >  qapi/dump.json        | 24 +++++++++--
> >  5 files changed, 119 insertions(+), 37 deletions(-)
> >
> > --
> > 2.39.3
> >
> 
> For the series:
> Reviewed-by: Marc-André Lureau <[email protected]>

Reviewed-by: Daniel P. Berrangé <[email protected]>


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.