Re: [PATCH] staging: vchiq: rework remove_event handling

From: Nicolas Saenz Julienne
Date: Tue Dec 11 2018 - 07:31:05 EST


On Tue, 2018-12-11 at 13:07 +0300, Dan Carpenter wrote:
> On Mon, Dec 10, 2018 at 10:11:58PM +0100, Arnd Bergmann wrote:
> > Note that I'm changing it back to 'killable' after the previous
> > patch
> > changed 'killable' to 'interruptible', apparently based on a
> > misunderstanding
> > of the subtle down_interruptible() macro override in
> > vchiq_killable.h.
>
> Oh wow... The non standard down_interruptible() macro is ugly.
>

Same reaction here, *_*. I wasn't even aware of it.

Arnd, as long as it you're not doing it already. I'll prepare some
extra fixes for the rest of completions and get rid of that file
altogether.

Regards,
Nicolas

Attachment: signature.asc
Description: This is a digitally signed message part