On Mon, Jan 09, 2012 at 09:37:24PM +0000, Russell King - ARM Linux wrote:
On Mon, Jan 09, 2012 at 12:31:30PM -0800, Mark Brown wrote:
If it's code like the rtd devices which is reasonably sensible and stable that's one thing but wading into code we know is fragile for what's essentially a warning fix is completely disproportionate.
You're still under the impression that it's "just a warning" and not "a potential oops".
To repeat myself yet again: there are a bunch of other issues here, if we're ever in the situation where we can trigger this oops we're also likely to be running into other equally severe problems. As I keep having to say we've got code which is buggy, difficult to work with but has been there for quite some time without actually triggering issues on systems. Nothing about this is saying -rc is a good time to be wading into the code.
And I disagree with your assessment of how easy it is to fix each problem.
The rtd devices are stored in an array - this array needs breaking up into separate allocations for each rtd as the very first step in fixing that. That then needs more error checking added, etc.
There's a patch for the rtds already, they should be fine now - we just dynamically allocate the device rather than worrying about the rtd. The issue with AC'97 isn't that it's especially hard to make this specific change, it's that the code is horrible and I don't trust it not to explode underneath us if we change anything.
If you want to completely restructure the rtds as well as providng a release function for 3.3 then you've got similar issues, you're talking about restructuring which is *completely* out of scope for -rc.
On the other hand, the AC'97 code in ASoC should need this to fix it:
Which is roughly what we did for the rtds too. This won't do as-is though:
#endif kfree(codec->ac97->bus);
- kfree(codec->ac97);
- put_device(&codec->ac97->dev);
We can't have potentially live devices floating around without a bus, there's an assumption that a valid AC'97 device will be on a bus. Moving the bus free into the device release function is probably all we need to do for that.