Re: [PATCH] extensions, ptdump: Add error message when the crash analyses a live system.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




----- 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 


> 
> ---
>  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@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/crash-utility
> 

--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility



[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux