Ticket #6537 (closed defect: fixed)

Opened 7 years ago

Last modified 3 years ago

Sharing problems with Salut

Reported by: gdesmott Owned by: morgs
Priority: normal Milestone: 11.2.0-M3
Component: read-activity Version:
Keywords: release? cjbfor9.1.0 Cc: Collabora, cjb
Action Needed: test in build Verified: no
Deployments affected: 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 years ago by gdesmott

See #6538 for another suspend problem with Read.

Changed 7 years ago by morgs

  • cc Collabora added; collabora removed

Changed 7 years 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 7 years ago by gdesmott

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

Changed 6 years 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 years 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 years ago by gdesmott

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

Changed 6 years ago by Blaketh

  • keywords release? added

Changed 6 years ago by gregorio

  • milestone deleted

Milestone Never Assigned deleted

Changed 6 years ago by marco

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

Changed 6 years ago by morgs

  • owner changed from rwh to morgs

Changed 6 years ago by morgs

  • milestone changed from 8.2.0 (was Update.2) to 8.2.1

Changed 6 years ago by mstone-xmlrpc

  • keywords cjbfor9.1.0 added
  • milestone changed from 8.2.1 to 9.1.0

Pushing out to 9.1.0, per edmcnierney's request.

Changed 3 years ago by godiard

  • next_action changed from diagnose to test in build
  • milestone changed from 9.1.0-cancelled to 11.2.0-M3

Sharing is already working and Read inhibit suspend when needed.

Changed 3 years ago by dsd

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.