提交 36f21329 编写于 作者: S Sarah Sharp 提交者: Greg Kroah-Hartman

USB: ehci: Fix IST boundary checking interval math.

When the EHCI driver falls behind in its scheduling, the active stream's
first empty microframe may be in the past with respect to the current
microframe.  The code attempts to move the starting microframe ("start") N
number of microframes forward, where N is the interval of endpoint.
However, stream->interval is a copy of the endpoint's bInterval, which is
designated in frames for FS devices, and microframes for HS devices.
Convert stream->interval to microframes before using it to move the
starting microframe forward.
Acked-by: NAlan Stern <stern@rowland.harvard.edu>
Signed-off-by: NSarah Sharp <sarah.a.sharp@linux.intel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
上级 12148da6
...@@ -1400,6 +1400,10 @@ iso_stream_schedule ( ...@@ -1400,6 +1400,10 @@ iso_stream_schedule (
goto fail; goto fail;
} }
period = urb->interval;
if (!stream->highspeed)
period <<= 3;
now = ehci_readl(ehci, &ehci->regs->frame_index) % mod; now = ehci_readl(ehci, &ehci->regs->frame_index) % mod;
/* when's the last uframe this urb could start? */ /* when's the last uframe this urb could start? */
...@@ -1417,8 +1421,8 @@ iso_stream_schedule ( ...@@ -1417,8 +1421,8 @@ iso_stream_schedule (
/* Fell behind (by up to twice the slop amount)? */ /* Fell behind (by up to twice the slop amount)? */
if (start >= max - 2 * 8 * SCHEDULE_SLOP) if (start >= max - 2 * 8 * SCHEDULE_SLOP)
start += stream->interval * DIV_ROUND_UP( start += period * DIV_ROUND_UP(
max - start, stream->interval) - mod; max - start, period) - mod;
/* Tried to schedule too far into the future? */ /* Tried to schedule too far into the future? */
if (unlikely((start + sched->span) >= max)) { if (unlikely((start + sched->span) >= max)) {
...@@ -1441,10 +1445,6 @@ iso_stream_schedule ( ...@@ -1441,10 +1445,6 @@ iso_stream_schedule (
/* NOTE: assumes URB_ISO_ASAP, to limit complexity/bugs */ /* NOTE: assumes URB_ISO_ASAP, to limit complexity/bugs */
period = urb->interval;
if (!stream->highspeed)
period <<= 3;
/* find a uframe slot with enough bandwidth */ /* find a uframe slot with enough bandwidth */
for (; start < (stream->next_uframe + period); start++) { for (; start < (stream->next_uframe + period); start++) {
int enough_space; int enough_space;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册