Ticket #6537 (new defect)

Opened 7 months ago

Last modified 11 days ago

Sharing problems with Salut

Reported by: gdesmott Owned by: morgs
Priority: normal Milestone: 8.2.1
Component: read-activity Version:
Keywords: release? Cc: Collabora, cjb
Action Needed: diagnose Verified: no
Blocked By: Blocking:

Description

On #6483 was discussed some sharing issues with Read and Salut.

I did lot of tests and sharing almost never works with Salut on XO's (but works pretty well with sugar-jhbuild with my stream tubes fix (see #6483)). Though disable Read's suspend policy (by creating this file /etc/inhibit-ebook-sleep) really improve results. So I think something goes wrong in Read's suspend code.

Would be cool if a Read developper could explain us how it's supposed to manage suspend.

Change History

Changed 7 months ago by gdesmott

See #6538 for another suspend problem with Read.

Changed 7 months ago by morgs

  • cc Collabora added; collabora removed

Changed 6 months ago by bemasc

After looking at the code, the problem is that Read suspends so aggressively that it will not even receive the "Join" notification when someone else joins the activity and requests the file. The sleep code may even interrupt the transfer. This is a major bug in Read. Fundamentally, Read must never suspend once it is shared, because it must receive multicast notifications in order for sharing to work.

Changed 6 months ago by gdesmott

Well, the real problem is in the wifi firmware which doesn't wake up when receiving multicast packets.

Changed 6 months ago by gnu

You can make the wifi firmware wake up on multicast packets (that this laptop is interested in) by doing this command: "ethtool -s eth0 wol um". The "u" in the last argument is unicast, "m" is multicast, (you can also add "b" to wake on broadcasts, though I don't recommend it). I suggest testing with this change and seeing if it is really the only problem, or whether we have other problems in Read or sharing that should also be addressed.

Changed 6 months ago by cjb

We wouldn't stay asleep for very long if we turned on wake on multicast, though, which would probably remove the point of these suspends. I agree it's still an interesting test.

Changed 6 months ago by gdesmott

I did some tests and multicast wake *does* improve things. Sharing was working fine without /etc/inhibit-ebook-sleep.

Changed 6 months ago by Blaketh

  • keywords release? added

Changed 8 weeks ago by gregorio

  • milestone deleted

Milestone Never Assigned deleted

Changed 7 weeks ago by marco

  • next_action set to diagnose
  • milestone set to 8.2.0 (was Update.2)

Changed 3 weeks ago by morgs

  • owner changed from rwh to morgs

Changed 11 days ago by morgs

  • milestone changed from 8.2.0 (was Update.2) to 8.2.1
Note: See TracTickets for help on using tickets.