On Wed, Sep 11, 2013 at 02:12:50AM +0300, Sergey Senozhatsky wrote: > Dan Carpenter noted that handle_pending_slot_free() is racy with > zram_reset_device(). Take write init_lock in zram_slot_free(), thus > preventing any concurrent zram_slot_free(), zram_bvec_rw() or > zram_reset_device(). This also allows to safely check zram->init_done > in handle_pending_slot_free(). > > Initial intention was to minimze number of handle_pending_slot_free() > call from zram_bvec_rw(), which were slowing down READ requests due to > slot_free_lock spin lock. Jerome Marchand suggested to remove > handle_pending_slot_free() from zram_bvec_rw(). > > Link: https://lkml.org/lkml/2013/9/9/172 > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@xxxxxxxxx> I have multiple versions of this and the other zram patches from you, with no idea of which to accept. So, I'm going to drop them all, can you please resend what you wish to submit, and in the future, be a bit more obvious with your "vN" markings? thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
