On 10/27, Srikar Dronamraju wrote: > > Copying Oleg (since he should have been copied on this one)
Thanks ;) > Please see one comment below. > > Acked-by: Srikar Dronamraju <sri...@linux.vnet.ibm.com> > > > --- > > kernel/events/uprobes.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/kernel/events/uprobes.c b/kernel/events/uprobes.c > > index 045b649..7a9e620 100644 > > --- a/kernel/events/uprobes.c > > +++ b/kernel/events/uprobes.c > > @@ -724,7 +724,7 @@ build_map_info(struct address_space *mapping, loff_t > > offset, bool is_register) > > int more = 0; > > > > again: > > - i_mmap_lock_write(mapping); > > + i_mmap_lock_read(mapping); I too think the patch is fine. I didn't see other changes, but I hope that i_mmap_lock_write/read names provide enough info and ->i_mmap_mutex was turned into rw-lock, in this case read-lock should be enough. Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
