svn commit: r234723 - in head/sys: amd64/amd64 amd64/include i386/i386 i386/include pc98/pc98 sparc64/include sparc64/sparc64 x86/x86

John Baldwin jhb at freebsd.org
Fri Apr 27 14:21:01 UTC 2012


On Friday, April 27, 2012 9:55:41 am Attilio Rao wrote:
> 2012/4/27, John Baldwin <jhb at freebsd.org>:
> > On Thursday, April 26, 2012 4:24:25 pm Attilio Rao wrote:
> >> Author: attilio
> >> Date: Thu Apr 26 20:24:25 2012
> >> New Revision: 234723
> >> URL: http://svn.freebsd.org/changeset/base/234723
> >>
> >> Log:
> >>   Clean up the intr* MD KPI from the SMP dependency, removing a cause of
> >>   discrepancy between modules and kernel, but deal with SMP differences
> >>   within the functions themselves.
> >>
> >>   As an added bonus this also helps in terms of code readability.
> >
> > Hmm, this should not have affected anything with modules as no modules
> > should have ever called this.  Also, making intr_bind() available for UP
> > kernels on x86 is largely pointless.  It's only caller is already
> > conditional
> > in sys/x86/x86/nexus.c:
> 
> It doesn't mean that there couldn't be an user in the future of this
> KPI in terms of thirdy part modules.
> These functions are part of the public KPI and we speak a lot about
> making our KPI as less as dependent by compiling options so I don't
> really see why you are not happy about this.
> 
> If the function (intr_bind() included) is part of the public KPI it
> can be called by thirdy part modules too.

No!  intr_bind() is _NOT_ part of the public KPI.  The public KPI is
bus_intr_bind() which accepts a struct resource, which is the approved
handle for interrupt resources in drivers and other modules.

-- 
John Baldwin


More information about the svn-src-head mailing list