Re: [RFC][PATCH -mmotm 1/4] Add static function calls of pstore tokexec path

From: Matthew Garrett
Date: Tue Jul 19 2011 - 15:28:08 EST


On Tue, Jul 19, 2011 at 03:14:22PM -0400, Seiji Aguchi wrote:
>
> >And how does that handle the case where we're halfway through a pstore
> >access when the NMI arrives? ERST, at least, has a complex state
> >machine. You can't guarantee what starting one transaction without
> >completing one that was in process will do.
>
> As for ERST, write access is protected by raw_spin_trylock_irqsave(&erst_lock).
> Are there anything I'm missing?

If there's already locking involved, what benefit does removing the lock
in the pstore code give? You'll just hang when you hit the erst code
instead of the pstore code.

--
Matthew Garrett | mjg59@xxxxxxxxxxxxx
--
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/