On 05/17/2010 11:40 AM, Pekka Enberg wrote: > On Mon, May 17, 2010 at 8:32 AM, Nitin Gupta <ngupta@xxxxxxxxxx> wrote: >> Resending as Greg wanted. >> >> (tested on mainline but should apply to linux-next cleanly) >> >> >> * Changelog: v2 vs initial patches >> - directly add swap free callback to block_device_operations >> instead of using 'notifiers' for various swap events. >> >> ramzswap driver creates RAM based block devices which can be >> used (only) as swap disks. Pages swapped to these disks are >> compressed and stored in memory itself. >> >> However, these devices do not get any notification when a swap >> slot is freed (swap_map[i] reaches 0). So, we cannot free memory >> allocated corresponding to this swap slot. Such stale data can >> quickly accumulate in (compressed) memory defeating the whole >> purpose of such devices. >> >> To overcome this problem, we now add a callback in struct >> block_device_operations which is called as soon as a swap >> slot is freed. >> >> Nitin Gupta (3): >> Add flag to identify block swap devices >> Add swap slot free callback to block_device_operations >> ramzswap: Handler for swap slot free callback >> >> drivers/staging/ramzswap/TODO | 5 ----- >> drivers/staging/ramzswap/ramzswap_drv.c | 22 +++++++++++++--------- >> include/linux/blkdev.h | 2 ++ >> include/linux/swap.h | 1 + >> mm/swapfile.c | 5 +++++ >> 5 files changed, 21 insertions(+), 14 deletions(-) >> delete mode 100644 drivers/staging/ramzswap/TODO > > Nitin, please retain ACKs when resending patches. > > Acked-by: Pekka Enberg <penberg@xxxxxxxxxxxxxx> > > for the whole series. Sorry, I forgot to include Ack lines. Here are the remaining Acks: Acked-by: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> Acked-by: Nigel Cunningham <nigel@xxxxxxxxxxxx> Thanks you all for reviews. Nitin _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel