Re: i2c-remove-redundant-i2c_client-list.patch

[Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>]

On Wed, 9 Jan 2008 13:21:28 -0800, David Brownell wrote:
> Right now I'm thinking that we'll need a multi-phase approach:
> 
> (a) Start phasing out users of i2c_client.list and its lock, ASAP;
>     merging those DVB driver patches, and some i2c-core changes.

Fine with me. Can you please send the i2c-core patch? It would include
i2c_verify_client(), then I can deal with the V4L drivers patch and
these 2 patches can be pushed to -mm quickly.

> (b) But don't remove that list from the deletion path until ...
> 
> (c) ... We have a solution that removes that wait_for_completion()
>     and its infrastructure.  (Note the similar i2c_adapter logic
>     too, sigh.)
> 
> (d) Meanwhile, come up with a different solution to the deadlock
>     observed with i2c_adapter.clist ... which for some unknown
>     reason has *NOT* shown up for me with lockdep.
> 
> Of course, if (c) happens soon soon, this problem simplifies.  And
> maybe someone will come up with a non-invasive solution to that
> problem ... but if nobody does so before, say, Monday, I'm thinking
> that (d) becomes a priority.

I won't be able to help with this as I don't understand it fully,
sorry. But I can do any amount of testing if someone sends patches.

-- 
Jean Delvare