On Fri, Aug 5, 2011 at 10:45 AM, Dan Magenheimer <dan.magenheimer@xxxxxxxxxx> wrote: >> > I am fairly sure that the failed allocation is handled gracefully >> > through the remainder of the frontswap code, but will re-audit to >> > confirm. A warning might be nice though. >> >> There is a place i think maybe have problem. >> function __frontswap_flush_area() in file frontswap.c called >> memset(sis->frontswap_map, .., ..); >> But if frontswap_map allocation fail there is a null pointer access ? > > Good catch! > > I'll fix that when I submit a frontswap update in a few days. > Would you please add current patch to you frontswap update series ? So I needn't to send a Version 2 separately with only drop the allocation failed handler. Thanks. -- Regards, --Bob -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href