[PATCH 7/7] session: Let autoconnect handle service connecting

Patrik Flykt patrik.flykt at linux.intel.com
Fri Nov 2 04:18:15 PDT 2012


On Fri, 2012-11-02 at 11:06 +0100, Daniel Wagner wrote:
> With this removal (which should remove the comple function btw)

Yep, I just had to stop at the minimum so that the patch set would be
readable. As part II there is quite a lot more that can be removed.

> you will 
> break the unit tests. When setting SessionMode to TRUE via D-Bus the 
> caller will get the result after the all service are disconnected.
> This 
> is is necessary pre conditions for the unit test.

Right, I have seem to forget that autoconnect is disabled when
SessionMode is enabled. Need to fix that.

If I remember correctly, we speculated what the purpose of a SessionMode
really is. Sessions must be notified also when SessionMode is set to
FALSE, and that part is working today.

The purpose of this patch set was to integrate the connection logic into
the autoconnect core. This in order to resolve the dilemma of
contradicting default route setups and service connection between
multiple sessions.

The point of all this is that sessions should not locally optimize what
service is used, instead it should be handled on a ConnMan wide scale.

Cheers,

	Patrik





More information about the connman mailing list