[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH] correct rcu_unlock_domain()
>>> On 20.04.17 at 16:11, <andrew.cooper3@xxxxxxxxxx> wrote: > On 20/04/17 14:48, Jan Beulich wrote: >> Match rcu_lock_domain(), and remove the slightly misleading comment: >> This isn't just the companion to rcu_lock_domain_by_id() (and that >> latter function indeed also keeps the domain locked, not the domain >> list). >> >> No functional change, as rcu_read_{,un}lock() ignore their arguments >> anyway. I think the second half of this sentence is ... >> Reported-by: Jann Horn <jannh@xxxxxxxxxx> >> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx> > > Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, although I think > it is worth stating in the commit message that the only reason this > currently works is because rcu_read_unlock() is entirely empty. .. what you're asking for here: Whether the function is empty is irrelevant; what is relevant is whether is uses its argument. Jan _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |