Re: [PATCH] lktdm: Bring support for hardlockup, softlockup andhung task crashes

[Cyrill Gorcunov]

On Sun, May 16, 2010 at 07:39:51AM +0200, Frederic Weisbecker wrote:
...  
>  static struct jprobe lkdtm;
> @@ -320,6 +326,20 @@ static void lkdtm_do_action(enum ctype which)
>  		memset(data, 0x78, len);
>  		break;
>  	}
> +	case SOFTLOCKUP:
> +		preempt_disable();
> +		for (;;)
> +			cpu_relax();
> +		break;
> +	case HARDLOCKUP:
> +		local_irq_disable();
> +		for (;;)
> +			cpu_relax();
> +		break;
> +	case HUNG_TASK:
> +		set_current_state(TASK_UNINTERRUPTIBLE);
> +		schedule();
> +		break;
>  	case NONE:
>  	default:
>  		break;

Looks good to me. Btw perhaps we may simplify it a bit:

	case HARDLOCKUP:
		local_irq_disable();
	case SOFTLOCKUP:
		preempt_disable();
		for (;;)
			cpu_relax();
		break;

since it'll save a few bytes. What do you think? Did I miss
something?

	-- Cyrill
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/