lkml.org 
[lkml]   [2005]   [Oct]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC][PATCH 5/6] swsusp: move swap-handling functions to separate file
Hi!

> This patch is to show that the swap-handling part of swsusp is really independent
> and it can be moved entirely to a separate file. It introduces the file swap.c
> containing all of the swap-handling code.
>
> After the change swsusp.c contains the functions that in my opinion do not
> belong to either the snapshot-handling part or the swap-handling part
> (swsusp_suspend(), swsusp_resume() and the functions related to highmem).

Highmem handling should go to snapshot.c. Other parts do not need to know about
it.


linux-2.6.14-rc5-mm1/kernel/power/swap.c
> ===================================================================
> --- /dev/null 1970-01-01 00:00:00.000000000 +0000
> +++ linux-2.6.14-rc5-mm1/kernel/power/swap.c 2005-10-29 13:26:26.000000000 +0200
> @@ -0,0 +1,915 @@
> +/*
> + * linux/kernel/power/snapshot.c

wrong name.

> +static void dump_info(void)
> +{
> + pr_debug(" swsusp: Version: %u\n",swsusp_info.version_code);
> + pr_debug(" swsusp: Num Pages: %ld\n",swsusp_info.num_physpages);
> + pr_debug(" swsusp: UTS Sys: %s\n",swsusp_info.uts.sysname);
> + pr_debug(" swsusp: UTS Node: %s\n",swsusp_info.uts.nodename);
> + pr_debug(" swsusp: UTS Release: %s\n",swsusp_info.uts.release);
> + pr_debug(" swsusp: UTS Version: %s\n",swsusp_info.uts.version);
> + pr_debug(" swsusp: UTS Machine: %s\n",swsusp_info.uts.machine);
> + pr_debug(" swsusp: UTS Domain: %s\n",swsusp_info.uts.domainname);
> + pr_debug(" swsusp: CPUs: %d\n",swsusp_info.cpus);
> + pr_debug(" swsusp: Image: %ld Pages\n",swsusp_info.image_pages);
> + pr_debug(" swsusp: Total: %ld Pages\n", swsusp_info.pages);
> +}

I'd rather get rid of this, or at least made it *way* more terse.

Ok, that probably belongs to separate patch.

Pavel
--
64 bytes from 195.113.31.123: icmp_seq=28 ttl=51 time=448769.1 ms

-
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/

\
 
 \ /
  Last update: 2005-10-30 20:35    [W:0.105 / U:3.696 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site