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 > <stephen.s.bren...@oracle.com> 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/20230717163855.7383-1-stephen.s.bren...@oracle.com/ > > [2]: > > https://lore.kernel.org/qemu-devel/20230914010315.945705-1-stephen.s.bren...@oracle.com/ > > > > 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 <marcandre.lur...@redhat.com>
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> 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 :|
