-----Original Message-----
> On Tue, Jun 08, 2021 at 02:40:39AM +0000, HAGIO KAZUHITO(萩尾 一仁) wrote:
> > > Does it still make sense to check the incomplete flag for the
> > > zero_excluded case? Otherwise, the above code comment will be
> > > outdated.
> > > + } else if (0 == pd.offset) {
> > > ...
> > > - if (*diskdump_flags & ZERO_EXCLUDED) {
> > > + if (is_incomplete_dump() && (*diskdump_flags & ZERO_EXCLUDED)) {
> >
> > Given the fact that makedumpfile cannot mark a dump as incomplete every time
> > it fails, I think it might be good to be able to use zero_excluded option
> > also without the incomplete flag.
> >
>
> Hi Kazu,
>
> Yes, that was my thought too.
>
> > So I'm ok with the original change, but additionally would like to
> > - fix the comment Lianbo pointed out and help texts of zero_excluded
> > - change netdump.c as well
> >
>
> I'll correct the comment. What would you want to change in netdump.c?
I thought that if we change diskdump.c to be able to use zero_excluded
without the incomplete flag then it would be better to change netdump.c
the same, i.e. remove this is_incomplete_dump() and fix the comment in
read_netdump() too:
/*
* If the incomplete flag has been set in the header,
* first check whether zero_excluded has been set.
*/
if (is_incomplete_dump() && (read_ret >= 0) &&
(*diskdump_flags & ZERO_EXCLUDED)) {
Thanks,
Kazu
--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/crash-utility
