[illumos-Developer] Webrev: New ARECA arcmsr driver

Albert Lee trisk at opensolaris.org
Wed Mar 23 20:26:18 PDT 2011


2437, 3321: Unnecessary casts.
"//" comments... lots of 'em, just grep.
855-857 is puzzling.
974: Seems like a weird way to say we want to allocate a packet to
store a single word?
NumToPtr/PtrToNum could be replaced.

-Albert



More information about the Developer mailing list