Mike Christie wrote: > +static struct class_device_attribute *stgt_target_attrs[] = { > + &class_device_attr_queued_cmnds, > + NULL > +}; > + oh yeah, I am not proposing there should only be one attribute. I just added the one because I am busy or lazy.