> Hi Dave > > I see your point. Sorry for all the confusion. > I now discard using THIS_KERNEL_VERSION(), and > use VALID_MEMBER() instead. > > Sending along the revised patch. Kindly review. > > Thanks > Rachita Hi Rachita, This looks good -- the only thing I'll change is to print out the offsets unconditionally in dump_offset_table() -- I actually want to see everything in the table, including the -1's for the invalid members. Red Hat's on a break until January 3rd, so I probably won't get to it unti then. Thanks, Dave