Re: Override endpoint attributes (was: RE: [PATCH] zd1211rw: change endpoint types of EP_REGS_OUT and EP_INT_IN from interrupt to bulk)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, 6 Feb 2011, Sujith wrote:

> Alan Stern wrote:
> > Why do you want to?  Why does changing the endpoint type from interrupt
> > to bulk improve performance?  Yes, the description says that this
> > causes CPU usage to go from 10% to below 1%, but _why_ does it do that?
> > 
> > Maybe the same effect can be achieved simply by changing the interrupt
> > interval instead.
> 
> Well, when the device is plugged in, the USB core recognizes the endpoints
> as interrupt type. Later, in the probe() callback, the FW is uploaded to the
> target and it patches the endpoints' type to bulk. Please correct me if am
> wrong, but I think this mismatch between the host and the target would be
> the cause for the high CPU usage, when the transmission load for that
> endpoint is high.

(Actually, bulk transfers generally cause a higher CPU load than 
interrupt transfers.  But never mind that...)

The problem is that the kernel doesn't know the firmware has been
changed.  The kernel has to be told about this, one way or another.  
Typically the device would disconnect itself from the USB bus briefly,
when starting to run the new firmware.  Or the kernel would be told to
reset the device after the firmware was transferred, after which the
device would start running the new firmware.

Apparently the driver needs to call usb_reset_device() when the
firmware has been transferred.  Following the reset, the kernel will
see that the descriptors have changed and it will reload all of them.  
Then it will know that the endpoint is bulk rather than interrupt.

Alan Stern

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux