From owner-p4-projects@FreeBSD.ORG  Mon May 10 23:42:33 2010
Return-Path: <owner-p4-projects@FreeBSD.ORG>
Delivered-To: p4-projects@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 32767)
	id DE4A71065672; Mon, 10 May 2010 23:42:32 +0000 (UTC)
Delivered-To: perforce@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id A2CB9106566B
	for <perforce@freebsd.org>; Mon, 10 May 2010 23:42:32 +0000 (UTC)
	(envelope-from hselasky@c2i.net)
Received: from swip.net (mailfe14.swipnet.se [212.247.155.161])
	by mx1.freebsd.org (Postfix) with ESMTP id 33C5F8FC1F
	for <perforce@freebsd.org>; Mon, 10 May 2010 23:42:31 +0000 (UTC)
X-Cloudmark-Score: 0.000000 []
X-Cloudmark-Analysis: v=1.0 c=1 a=qL1Cqo4YLXoA:10 a=Q9fys5e9bTEA:10
	a=M8b_wTzEtboA:10 a=MnI1ikcADjEx7bvsp0jZvQ==:17
	a=6I5d2MoRAAAA:8 a=ALcXzfPqg_cNW9f1jGkA:9
	a=jJncDVITzPSTuolcfI9HCtGrR1sA:4 a=PUjeQqilurYA:10
Received: from [188.126.201.140] (account mc467741@c2i.net HELO
	laptop002.hselasky.homeunix.org)
	by mailfe14.swip.net (CommuniGate Pro SMTP 5.2.19)
	with ESMTPA id 622819937 for perforce@freebsd.org;
	Tue, 11 May 2010 01:42:30 +0200
From: Hans Petter Selasky <hselasky@c2i.net>
To: Perforce Change Reviews <perforce@freebsd.org>
Date: Tue, 11 May 2010 01:39:44 +0200
User-Agent: KMail/1.12.4 (FreeBSD/8.0-STABLE; KDE/4.3.4; amd64; ; )
References: <201005102335.o4ANZnb9090023@repoman.freebsd.org>
In-Reply-To: <201005102335.o4ANZnb9090023@repoman.freebsd.org>
X-Face: +~\`s("[*<S!NI[0M*"FaghA8S>|O,="7?X@L.elg*F"OA\I/3%^p8g?ab%RN'(; _IjlA:
	hGE..Ew, XAQ*o#\/M~SC=S1-f9{EzRfT'|Hhll5Q]ha5Bt-s|oTlKMusi:1e[wJl}kd}GR
	Z0adGx-x_0zGbZj'e(Y[(UNle~)8CQWXW@:DX+9)_YlB[tIccCPN$7/L'
MIME-Version: 1.0
Content-Type: Text/Plain;
  charset="iso-8859-15"
Content-Transfer-Encoding: 7bit
Message-Id: <201005110139.44220.hselasky@c2i.net>
Cc: 
Subject: Re: PERFORCE change 178075 for review
X-BeenThere: p4-projects@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: p4 projects tree changes <p4-projects.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/p4-projects>
List-Post: <mailto:p4-projects@freebsd.org>
List-Help: <mailto:p4-projects-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 10 May 2010 23:42:33 -0000

On Tuesday 11 May 2010 01:35:49 Hans Petter Selasky wrote:
> http://p4web.freebsd.org/@@178075?ac=10
> 

The following change is about computing the correct delay in the frame-list:

>  	if ((xfer->endpoint->is_synced == 0) ||
> -	    (buf_offset < ((xfer->nframes + 7) / 8))) {
> +	    (buf_offset < (((xfer->nframes << shift) + 7) / 8))) {
>  		/*
>  		 * If there is data underflow or the pipe queue is empty we
>  		 * schedule the transfer a few frames ahead of the current
> @@ -2844,7 +2855,7 @@
>  	 */
>  	xfer->isoc_time_complete =
>  	    usb_isoc_time_expand(&sc->sc_bus, nframes) + buf_offset +
> -	    ((xfer->nframes + 7) / 8);
> +	    (((xfer->nframes << shift) + 7) / 8);
> 
>  	/* get the real number of frames */
> 
>