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 <srikar@xxxxxxxxxxxxxxxxxx> > > > --- > > 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, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>