Request for patch approval (Re: svn commit: r225458 - in
stable/8/sys: dev/usb dev/usb/quirk dev/usb/storage sys)
kostikbel at gmail.com
Sat Sep 10 14:28:21 UTC 2011
On Sat, Sep 10, 2011 at 03:40:16PM +0200, Hans Petter Selasky wrote:
> > Right -- exactly my point. If this change breaks third-party compiled USB
> > device drivers, then our current approach to device driver KBIs does not
> > allow it to be MFC'd in this form. Are there ways you can reformulate the
> > change to avoid breaking those drivers? Sometimes this can be done by
> > adding new symbols, rather than replacing currently symbols, although
> > mileage varies.
> Here is my proposal:
> Implement test for automatic quirks in function which has access to the USB
> device structure. This decouples the structure change in "struct
> The only structure which needs change is "struct usb_device". In 9-current
> this structure will be kept as is. In 8-stable the new element will be moved
> to the end of the structure like suggested, and then there shouldn't be any
> Please find patches attached.
> Commit message:
> Refactor auto-quirk solution so that we break as few external
> drivers as possible.
> PR: usb/160299
> Approved by: re (kib)
> Suggested by: rwatson
> MFC after: 0 days
First, can you, please, regenerate the diff for stable/8 against the
code before r225458 ? I want to read diff to see ABI change, assuming
r225458 was not done at all.
Second, you cannot decrement __FreeBSD_version. In fact, you shall increment
it once more in the patch for stable/8.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 196 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/svn-src-stable-8/attachments/20110910/cd9bf9e3/attachment.pgp
More information about the svn-src-stable-8