Markus Armbruster <[email protected]> writes: > Peter Maydell <[email protected]> writes: > >> On Fri, 3 Nov 2023 at 07:02, <[email protected]> wrote: >>> >>> From: Stephen Brennan <[email protected]> >>> >>> The QMP dump API represents the dump format as an enumeration. Add three >>> new enumerators, one for each supported kdump compression, each named >>> "kdump-raw-*". >>> >>> For the HMP command line, rather than adding a new flag corresponding to >>> each format, it seems more human-friendly to add a single flag "-R" to >>> switch the kdump formats to "raw" mode. The choice of "-R" also >>> correlates nicely to the "makedumpfile -R" option, which would serve to >>> reassemble a flattened vmcore. >>> >>> Signed-off-by: Stephen Brennan <[email protected]> >>> Reviewed-by: Daniel P. Berrangé <[email protected]> >>> [ Marc-André: replace loff_t with off_t, indent fixes ] >>> Reviewed-by: Marc-André Lureau <[email protected]> >>> Message-Id: <[email protected]> >> >> Hi; Coverity points out some issues in this commit: > > Proposed fixes: > > Subject: [PATCH 0/2] dump: Fix issues flagged by Coverity > Message-ID: <[email protected]>
Closing the loop on this, these fixes were exactly right. Sorry for the extra work here folks. Stephen
