Date: Mon, 21 Nov 2005 17:33:48 +0100 (CET)
From: Bodo Eggert <7eggert@gmx.de>
To: Con Kolivas <kernel@kolivas.org>
cc: Daniel =?iso-8859-1?q?Marjam=E4ki?= <daniel.marjamaki@comhem.se>,
       linux-kernel@vger.kernel.org, 7eggert@gmx.de, nish.aravamudan@gmail.com
Subject: Re: I made a patch and would like feedback/testers (drivers/cdrom/aztcd.c)
In-Reply-To: <200511211919.11429.kernel@kolivas.org>
Message-ID: <Pine.LNX.4.58.0511211729510.2082@be1.lrz>
References: <5aZsv-3CJ-17@gated-at.bofh.it> <E1EdwGs-0000qv-NL@be1.lrz>
 <438180C0.2030503@comhem.se> <200511211919.11429.kernel@kolivas.org>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=iso-8859-1
Content-Transfer-Encoding: 8BIT
X-be10.7eggert.dyndns.org-MailScanner-Information: See www.mailscanner.info for information
X-be10.7eggert.dyndns.org-MailScanner: Found to be clean
X-be10.7eggert.dyndns.org-MailScanner-From: 7eggert@web.de
Sender: linux-kernel-owner@vger.kernel.org
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 879
Lines: 22

On Mon, 21 Nov 2005, Con Kolivas wrote:
> On Mon, 21 Nov 2005 19:09, Daniel Marjam?ki wrote:

> > The way I do it:
> > All systems will give me a delay for at least a few ms.
> > I get the shortest timeout possible on each computer.
> 
> Convention in the kernel would be 
> 	aztTimeOut =  HZ / 100 ? : 1;
> to be at least one tick (works for HZ even below 100) and is at least 10ms.

You have to add one to the timeout, since you might be just before the end 
of the timeslice when the delay starts. aztTimeOut = (HZ / 100 ? : 1) + 1;

-- 
Top 100 things you don't want the sysadmin to say:
11. Can you get VMS for this Sparc 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/