Hi, Indoh-san, ptdump does not support retrieving the trace data from the live system. However, the current error message does not tell so. When the crash analyses a live system, ptdump only displays the following message: crash> ptdump foo ptdump: invalid size request: 0 type: "read page for write" I think it's better to have a different message to let users know why error occurs. Therefore, I added the following error message in ptdump: crash> ptdump foo ptdump: no support on live system Please merge this patch.

--- ptdump.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ptdump.c b/ptdump.c index 8d5c367..44b19fb 100644 --- a/ptdump.c +++ b/ptdump.c @@ -491,6 +491,9 @@ cmd_ptdump(void) if (argcnt != 2) cmd_usage(pc->curcmd, SYNOPSIS); + if (ACTIVE()) + error(FATAL, "no support on live system\n"); + outdir = args[1]; if ((ret = mkdir(outdir, mode))) { fprintf(fp, "Cannot create directory %s: %d\n", outdir, ret); -- 1.8.3.1 Regards, -- Yuki Inoguchi -- Crash-utility mailing list Crash-utility(a)redhat.com https://www.redhat.com/mailman/listinfo/crash-utility

----- Original Message ----- > > > ----- Original Message ----- >> Hi, Indoh-san, >> >> ptdump does not support retrieving the trace data from the live system. >> However, the current error message does not tell so. >> When the crash analyses a live system, ptdump only displays the following >> message: >> >> crash> ptdump foo >> ptdump: invalid size request: 0 type: "read page for write" >> >> I think it's better to have a different message to let users know why error occurs. >> Therefore, I added the following error message in ptdump: >> >> crash> ptdump foo >> ptdump: no support on live system >> >> Please merge this patch. > > Hello Yugi, > > In the absence of a response from Takao, I will go ahead and update > the package next week. > > Thanks, > Dave Hi Yugi, I went ahead and created a ptdump-1.0.6 package with your patch applied: http://people.redhat.com/anderson/extensions.html#PTDUMP Thanks, Dave > >> >> --- >> ptdump.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/ptdump.c b/ptdump.c >> index 8d5c367..44b19fb 100644 >> --- a/ptdump.c >> +++ b/ptdump.c >> @@ -491,6 +491,9 @@ cmd_ptdump(void) >> if (argcnt != 2) >> cmd_usage(pc->curcmd, SYNOPSIS); >> >> + if (ACTIVE()) >> + error(FATAL, "no support on live system\n"); >> + >> outdir = args[1]; >> if ((ret = mkdir(outdir, mode))) { >> fprintf(fp, "Cannot create directory %s: %d\n", outdir, >> ret); >> -- >> 1.8.3.1 >> >> Regards, >> -- >> Yuki Inoguchi >> >> >> -- >> Crash-utility mailing list >> Crash-utility(a)redhat.com >> https://www.redhat.com/mailman/listinfo/crash-utility >> > -- Crash-utility mailing list Crash-utility(a)redhat.com https://www.redhat.com/mailman/listinfo/crash-utility