Subject: Re: Worrisome IDE PIO transfers...
From: Christophe Saout <christophe@saout.de>
To: Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>
Cc: Jeff Garzik <jgarzik@pobox.com>, Matt Mackall <mpm@selenic.com>,
       Jens Axboe <axboe@suse.de>, Geert Uytterhoeven <geert@linux-m68k.org>,
       Linux Kernel <linux-kernel@vger.kernel.org>
In-Reply-To: <200403010147.47808.bzolnier@elka.pw.edu.pl>
References: <4041232C.7030305@pobox.com> <20040229015041.GQ3883@waste.org>
	 <40415152.8040205@pobox.com>  <200403010147.47808.bzolnier@elka.pw.edu.pl>
Content-Type: text/plain
Message-Id: <1078147381.7497.15.camel@leto.cs.pocnet.net>
Mime-Version: 1.0
Date: Mon, 01 Mar 2004 14:23:02 +0100
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1070
Lines: 26

Am Mo, den 01.03.2004 schrieb Bartlomiej Zolnierkiewicz um 01:47:

> http://www.kernel.org/pub/linux/kernel/people/bart/dm-byteswap-2.6.4-rc1.patch
> 
> Guess what's this? :)

The thieves... they've stolen my precioussss. ;)

> It is simply a stripped down dm-crypt.c, so all credits go to Christophe.
> I have tested it quickly with loop device and it seems to work.

Yes, it's not that complicated. Looks good.
BTW: You don't need the km_types voodoo as the conversion routine is
never called from a softirq context and you are allowed (but should try
to avoid it) to sleep. You could add a conditional reschedule after
kunmapping the buffers to keep the latency low on non-preempt kernels.

BTW: I've got some cleanups and a small fix in Andrew's latest tree
(using a #define for the log prefix and I bvec array thingy).


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