try_module_get in hfi1

[Jason Gunthorpe <jgg@xxxxxxxxxxxx>]

Intel Guys,

I'm looking at code in hfi1:

static int __i2c_debugfs_open(struct inode *in, struct file *fp, u32 target)
{
	struct hfi1_pportdata *ppd;
	int ret;

	if (!try_module_get(THIS_MODULE))
		return -ENODEV;

Seems like nonsense to me.

I think it should be:

--- a/drivers/infiniband/hw/hfi1/debugfs.c
+++ b/drivers/infiniband/hw/hfi1/debugfs.c
@@ -1155,6 +1155,7 @@ static int exprom_wp_debugfs_release(struct inode *in, struct file *fp)
 { \
        .name = nm, \
        .ops = { \
+               .owner = THIS_MODULE, \
                .read = readroutine, \
                .write = writeroutine, \
                .llseek = generic_file_llseek, \
@@ -1165,6 +1166,7 @@ static int exprom_wp_debugfs_release(struct inode *in, struct file *fp)
 { \
        .name = nm, \
        .ops = { \
+               .owner = THIS_MODULE, \
                .read = readf, \
                .write = writef, \
                .llseek = generic_file_llseek, \


Can you fix it??

Jason