Date: Fri, 26 Nov 2004 00:55:27 +0100
From: Pavel Machek <pavel@ucw.cz>
To: Nigel Cunningham <ncunningham@linuxmail.org>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Suspend 2 merge: 48/51: Swapwriter
Message-ID: <20041125235527.GH2909@elf.ucw.cz>
References: <1101292194.5805.180.camel@desktop.cunninghams> <1101300479.5805.389.camel@desktop.cunninghams>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1101300479.5805.389.camel@desktop.cunninghams>
User-Agent: Mutt/1.5.6+20040722i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1010
Lines: 23

Hi!

> This is the swapwriter. It is forms the glue between the highlevel I/O
> routines in io.c and the blockwriter routines in block_io.c. It is
> responsible for allocating storage, translating the requests for pages
> within pagesets into devices and blocks and the like. It is abstracted
> from the block writer because the plan is that we'll eventually have a
> generic file writer (ie not using swapspace, but a simple file,
> possibly

This file alone is bigger than whole swsusp1. That strongly suggests
you have too many layers of abstraction in there. Planning for future
is nice, but not at this cost.

								Pavel
-- 
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
-
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/