[openib-general] Re: Re: [Andrew Morton] inappropriate use of in_atomic()

Michael S. Tsirkin mst at mellanox.co.il
Sun Mar 20 10:12:42 PST 2005


Quoting r. Libor Michalek <libor at topspin.com>:
> Subject: Re: Re: [Andrew Morton] inappropriate use of in_atomic()
> 
> On Fri, Mar 11, 2005 at 09:31:08AM +0200, Michael S. Tsirkin wrote:
> > 
> > Sdp also has a couple of uses.
> > Maybe we can use the atomic branch in all cases here, as well?
> > Libor?
> 
>   Yes, the case in sdp_iocb.c can probably always take the atomic
> path. The kmap/kunmap cases really only care whether we're in an
> interrupt, so switching to in_interrupt() should be sufficient.
> 
> -Libor
> 

Recent comments by Andrew indicate that it is better to always
use kmap_atomic/kunmap_atomic if possible. This will also
let us get rid of the wrapper function, which is good.

Why do you think we need to kmap?

-- 
MST - Michael S. Tsirkin



More information about the general mailing list