uvideo: limit isochronous transfers to 32 frames
AcceptedPublic
Actions

Authored by guest-seuros on Tue, Jul 28, 12:14 AM.

Details

Reviewers

adrian
bapt

Summary

Raising UVIDEO_NFRAMES_MAX from 40 to 128 in 3b6f833c95eb improved
throughput on xhci but made every camera on an ehci bus fail to
stream. Integrated webcams became unusable.

Measured on a MacBookPro9,2 with two ehci(4) FaceTime HD cameras and an
xhci(4) Logitech C920:

			  128		 32
  ehci, 12 captures	  0 ok		 12 ok
  xhci 1920x1080	  5 fps		  5 fps
  xhci 1280x720		 10 fps		 10 fps

Fixes: 3b6f833c95eb

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 75199
Build 72082: arc lint + arc unit

Event Timeline

guest-seuros created this revision.Tue, Jul 28, 12:14 AM

Herald added a subscriber: imp. · View Herald TranscriptTue, Jul 28, 12:14 AM

guest-seuros requested review of this revision.Tue, Jul 28, 12:14 AM

Harbormaster completed remote builds in B75199: Diff 182874.Tue, Jul 28, 12:14 AM

guest-seuros added a parent revision: D58500: fwcamctl: remove, superseded by the video(4) interface.Tue, Jul 28, 12:14 AM

guest-seuros added a child revision: D58502: fwcam: release the IR DMA channel when starting the stream fails.Tue, Jul 28, 12:18 AM

bapt accepted this revision.Tue, Jul 28, 6:46 PM

bapt removed a subscriber: imp.

This revision is now accepted and ready to land.Tue, Jul 28, 6:46 PM

guest-seuros added projects: drivers, USB.Tue, Jul 28, 7:34 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

usb/

video/

uvideo.h

2 lines

Diff 182874

View Options

sys/dev/usb/video/uvideo.h

uvideo: limit isochronous transfers to 32 framesAcceptedPublicActions