Date: Thu, 25 May 2006 07:52:57 -0700
From: Andrew Morton <akpm@osdl.org>
To: Anssi Hannula <anssi@mandriva.org>
Cc: dtor_core@ameritech.net, linux-joystick@atrey.karlin.mff.cuni.cz,
       linux-kernel@vger.kernel.org
Subject: Re: [patch 03/11] input: new force feedback interface
Message-Id: <20060525075257.3f2963a9.akpm@osdl.org>
In-Reply-To: <4475C2F2.7090207@mandriva.org>
References: <20060515211229.521198000@gmail.com>
	<20060515211506.783939000@gmail.com>
	<20060517222007.2b606b1b.akpm@osdl.org>
	<44757246.9010300@mandriva.org>
	<20060525070017.16344c97.akpm@osdl.org>
	<4475C2F2.7090207@mandriva.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1397
Lines: 33

Anssi Hannula <anssi@mandriva.org> wrote:
>
> > Generally we use file descriptors (and driver-specific state at
>  > file.f_private) to manage things like that.  But I'd imagine that we
>  > couldn't retain the existing semantics with any such scheme.
>  > 
>  > A pragmatic approach would be to put a big fat comment in there explaining
>  > how it all works and leave it at that.
> 
>  As I don't see this could break any existing applications, I would very
>  much like to change the behaviour so that the effects are file
>  descriptor specific.

ooh, that's always risky - we just don't know what people are doing out
there.  They do the damnedest things.

Is it possible to implement the new behaviour while retaining the old
behaviour as well?  And to detect when an app is using the old behaviour
and to drop a printk("stop doing this")?  So we can kill the old behaviour
in a year or so?

> What should I use to differentiate the descriptors?
>  Can I just compare the "struct file*"? (it seems to work well, I just
>  modified the code so)

Depends what you're trying to do.  Different threads in the same process
can share the same file*'s.

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