[PATCH] Use external tool for creating backtraces on crashes if available.

Barry Scott barry.scott at onelan.co.uk
Tue Dec 7 09:42:06 PST 2010


On Friday 08 October 2010 17:25:12 Matthias Hopf wrote:
> This calls /usr/bin/xorg-backtrace to create reasonable commented backtraces
> with gdb. On errors it falls back to the generic method.
> 
> Signed-off-by: Matthias Hopf <mhopf at suse.de>
> ---

This is a very useful patch, thanks for developing it.

...

> +    status = 0;
> +    while (fgets (buf, 256, f)) {
> +	status++;
> +	ErrorF("%s", buf);
> +    }
> +    fclose (f);
> +    if (status < 10 && ret == 0) {
> +	ErrorF ("%s only produced %d lines of output\n", xorg_backtrace, status);
> +	return 1;
> +    }


We are using this patch with a different usage. In our environment the stdout
file from Xorg does not last long enough, its gone after the user reboots.

Our xorg-backtrace script writes the analysis into a new log file.

It would match our usage better to have this check for atleast 1 line rather
then 10. Our one line says where we wrote the analysis too.

Barry


More information about the xorg-devel mailing list