On Fri, 2015-05-29 at 01:04 -0700, Narsimhulu Musini wrote: > +static const struct file_operations snic_stats_fops __read_mostly = { > + .owner = THIS_MODULE, > + .open = snic_stats_open, > + .read = seq_read, > + .llseek = seq_lseek, > + .release = single_release, > +}; Checkpatch hates this for obvious reasons: a const structure has to be read only, not read mostly, so it's a section conflict. I fixed it up, but running the code through checkpatch would have flagged it as an error. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html