Hi,
On Thu, 2010-02-18 at 16:04 -0500, David Teigland wrote:
On Thu, Feb 18, 2010 at 09:16:03AM +, Steven Whitehouse wrote:
I'm not sure what more I can say here this is a sysfs file store
function and one of the reasons for using it is that sysfs looks after
the ref counting for
Hi,
On Wed, 2010-02-17 at 15:12 -0500, David Teigland wrote:
On Wed, Feb 17, 2010 at 09:41:35AM +, Steven Whitehouse wrote:
We don't need to look up the lockspace in this particular
case since we already have a pointer to it (which was being
dereferenced in order to do the lookup in
On Thu, Feb 18, 2010 at 09:16:03AM +, Steven Whitehouse wrote:
I'm not sure what more I can say here this is a sysfs file store
function and one of the reasons for using it is that sysfs looks after
the ref counting for you.
Even aside from that, if you don't have a reference to the
We don't need to look up the lockspace in this particular
case since we already have a pointer to it (which was being
dereferenced in order to do the lookup in the first place).
Signed-off-by: Steven Whitehouse swhit...@redhat.com
---
fs/dlm/lockspace.c |6 +-
1 files changed, 1
On Wed, Feb 17, 2010 at 09:41:35AM +, Steven Whitehouse wrote:
We don't need to look up the lockspace in this particular
case since we already have a pointer to it (which was being
dereferenced in order to do the lookup in the first place).
It'll take more to convince me that that