From mboxrd@z Thu Jan  1 00:00:00 1970
Date: Fri,  4 Jan 2013 20:22:52 +0100
From: David du Colombier <0intro@gmail.com>
To: 9fans@9fans.net
Message-ID: <20130104202252.0b9bec10@zinc.9fans.fr>
In-Reply-To: <6e680bde0bcfb96fa20d42f2371e075f@hamnavoe.com>
References: <20130104194803.02fa087b@zinc.9fans.fr>
	<6e680bde0bcfb96fa20d42f2371e075f@hamnavoe.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Subject: Re: [9fans] 9atom
Topicbox-Message-UUID: fdf82158-ead7-11e9-9d60-3106f5b1d025

> I can't contain my curiosity: what's the line?
>
> I once worked with a colleague who had a superstition that any
> one-line change to production software was almost certain to
> introduce a bug.  (Actually what he said was "one-card change",
> which indicates how long ago this was.)   If he had to make
> a modification simple enough to be done in one line, he would
> always combine it with some other random harmless change like
> renaming a variable.

This line:

http://www.9legacy.org/9legacy/patch/pc-sdata-dma-lite.diff

It enables DMA by default in the sdata driver.
But it's not a requirement. It's just a remain
of a personal taste.

Surprisingly, I have another version of this patch
which should convince your former colleague :-)

http://www.9legacy.org/9legacy/patch/pc-sdata-dma.diff

--
David du Colombier