On Sat, Dec 01, 2012 at 02:45:57AM +0900, Joonsoo Kim wrote: > @@ -614,21 +616,35 @@ static int ashmem_pin_unpin(struct ashmem_area *asma, > unsigned long cmd, > pgstart = pin.offset / PAGE_SIZE; > pgend = pgstart + (pin.len / PAGE_SIZE) - 1; > > - mutex_lock(&ashmem_mutex); > + if (asma->shared_mapping) { > + mutex_lock(&ashmem_mutex);
Wouldn't we need to hold the mutex while we check the ->shared_mapping? regards, dan carpenter -- 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/