On Thu, Feb 28, 2008 at 5:00 PM, Steven Stallion <[EMAIL PROTECTED]> wrote:
> On Thu, 28 Feb 2008 09:34:16 +0200, "Cyril Plisko"
> <[EMAIL PROTECTED]> wrote:
> > Steve,
> >
> > What catches my eye is the 7th argument to the DDI_DEFINE_STREAM_OPS()
> > macro
> > It should be a xx_getinfo() entry point. Since it is a network driver
> > and it can get away
> > without proper getinfo() it should be nodev() or ddi_no_info(). I am
> > really not sure what
> > happens if NULL is passed there.
> >
> > Well, I just checked the kernel source, and it appears that there is
> > at least one point
> > where getinfo() provided by the driver is called without checking it
> > is non-NULL beforehand.
> > Since it is called from uts/common/os/driver.c::bdev_strategy() I
> > highly doubt it has something
> > to do with your problem. Still leaving it as NULL bothers me.
>
> Gotcha. Is there any reason why I would use ddi_no_info over nodev?
>
I just checked that a little further - turns out mac_init_ops() overrides the
devo_getinfo field with dld_getinfo(). So NULL, nodev, ddi_no_info or anything
else doesn't really matter.
--
Regards,
Cyril
_______________________________________________
driver-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/driver-discuss
