Olaf Hering <olaf@xxxxxxxxx> writes: > On Wed, Mar 11, K. Y. Srinivasan wrote: > >> +++ b/drivers/hv/hv_balloon.c >> @@ -652,6 +652,7 @@ static void hv_mem_hot_add(unsigned long start, unsigned long size, >> } >> has->ha_end_pfn -= HA_CHUNK; >> has->covered_end_pfn -= processed_pfn; >> + mutex_lock(&dm_device.ha_region_mutex); >> break; >> } > > Should it call the wrapper instead of doing mutex_lock directly? > Like 'acquire_region_mutex(false);'. My "Drivers: hv: hv_balloon: eliminate the trylock path in acquire/release_region_mutex" (b05d8d9ef5ef21d1b18440430f950304836e1aaa in char-misc-next) removed these wrappers. > > Olaf -- Vitaly _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
