Messages in this thread Patch in this message | | | Date | Fri, 10 Sep 2004 10:57:40 +0200 | From | Pavel Machek <> | Subject | swsusp: another simplifycation |
| |
Hi!
verify is not really descriptive name of function. Fortunately its insides are self-documenting which makes it easy to fix. Please apply,
Pavel
--- clean-mm/kernel/power/swsusp.c 2004-09-07 21:12:33.000000000 +0200 +++ linux-mm/kernel/power/swsusp.c 2004-09-09 08:56:20.000000000 +0200 @@ -1130,17 +1108,6 @@ return error; } - -int __init verify(void) -{ - int error; - - if (!(error = check_sig())) - error = check_header(); - return error; -} - - /** * swsusp_read_data - Read image pages from swap. * @@ -1202,13 +1171,14 @@ { int error = 0; - if ((error = verify())) + if ((error = check_sig())) + return error; + if ((error = check_header())) return error; if ((error = read_pagedir())) return error; - if ((error = data_read())) { - free_pages((unsigned long)pagedir_nosave,pagedir_order); - } + if ((error = data_read())) + free_pages((unsigned long)pagedir_nosave, pagedir_order); return error; } - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |