[illumos-Developer] Webrev: New ARECA arcmsr driver

Garrett D'Amore garrett at damore.org
Thu Mar 24 11:21:29 PDT 2011


On Thu, 2011-03-24 at 14:20 -0400, Albert Lee wrote:
> On Thu, Mar 24, 2011 at 1:45 PM, Garrett D'Amore <garrett at damore.org> wrote:
> > On Thu, 2011-03-24 at 13:41 -0400, Dan McDonald wrote:
> >> On Thu, Mar 24, 2011 at 09:57:55AM -0700, Garrett D'Amore wrote:
> >> > All of these changes are small, and hopefully low risk changes.
> >> >
> >> > The new webrev is here:
> >> >
> >> > http://mexico.purplecow.org/gdamore/webrev/arcmsr-delta/
> >>
> 
> Last nit:
> 1005 +                arcmsr_cdb->Context = (uintptr_t)arcmsr_cdb;
> 
> Use the new PtrToNum?
> 
> -Albert

Nah.  I hate that macro, actually. :-)

	- Garrett





More information about the Developer mailing list