Opinion please
Timothy R. Chavez
tinytim at us.ibm.com
Mon May 16 21:55:47 UTC 2005
On Monday 16 May 2005 11:21, Timothy R. Chavez wrote:
> Hello,
>
> Serge pointed out something to me on an internal list here and I wanted to
> poll for opinions. Which would you prefer? I think adding another
> parameter to audit_send_reply to specify GFP_ATOMIC/KERNEL would be
> simplest. But is it wise to do so?
>
> Here's what he had to say:
>
> Hey Tim,
>
> it looks like w_master is now adequately protected by the spinlock.
>
> But I'm not sure whether netlink_unicast is a problem (sure looks like
> it - it puts current on a waitqueue for one thing) or what you should do
> about it. You might just need to build a list of things to send under
> the rcu_read_lock(), then send each element once you rcu_read_unlock().
Ok, I took this advice, and just went ahead and implemented something very
similar. Good idea Serge. Thanks. I'm going to do some more extensive
testing here before I release a patch since people are apparently having
problems with the code (plus I'm going to patch up to rc4-mm2).
-tim
More information about the Linux-audit
mailing list