Peter Memishian wrote: > webrev: http://cr.opensolaris.org/~meem/uv-tweaks > cscope: /net/atlantic.east/export/ws/meem/uv-tweaks/usr/src >
Just a couple of minor comments: usr/src/uts/common/io/dld/dld_drv.c * 662: should be mod_hash_key_t * 955: This comment's formatting is strange to me, and not in line with other block comments. I'd rather all lines be left-justified completely, and paragraphs be separated by a blank line. (Yes, I realize it was like that before.) * 956: s/update/updated/ usr/src/uts/common/io/dld/dld_proto.c * 1310: you could remove the spurious '.' while you're here. usr/src/uts/common/io/ib/clients/rds/rds_ioctl.c * 139,141: Question: why is this needed? -Seb
