generic_setlease and tmpfs -- LTP fcntl failure

In debugging a LTP failure related to fcntl on tmpfs it appears that we aren't able to use fcntl(fd, F_SETLEASE, F_WRLCK). In the debugging it looks like we artificial increase the dentry->d_count and so generic_setlease() always fails with -EAGAIN since it appears dentry->d_count will be 2.

This is my first time delving into fs code so I'm not that familiar with why tmpfs is artificially increasing the d_count. I was wondering what a possible solution would be.

Maybe something like:

shmem_setlease(...) {
	if (arg == F_WRLCK) {
		dput(dentry)
		generic_setlease(...)
		dget(dentry)
	} else {
		generic_setlease(...)
	}
}

Not sure what harm doing the dput() / dget() will have.

- k
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: generic_setlease and tmpfs -- LTP fcntl failure
    • From: "J. Bruce Fields" <[email protected]>

• Prev by Date: Re: [PATCH 3/4] drivers/infiniband: Drop redundant includes of moduleparam.h
• Next by Date: Re: [PATCH 3/3] nfs: use ->mmap_prepare() to avoid an AB-BA deadlock
• Previous by thread: [PATCH] Hibernation: Fix lockdep report (v2)
• Next by thread: Re: generic_setlease and tmpfs -- LTP fcntl failure
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]