[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