<div dir="ltr">Probably the best is revert <a href="https://github.com/sugarlabs/sugar/commit/9c37cff7c8bc7d6cfc55f53e4c55926f555713ff">https://github.com/sugarlabs/sugar/commit/9c37cff7c8bc7d6cfc55f53e4c55926f555713ff</a><div>or try to move the logic to jarabe/model/network.py,</div><div>but I don't know if that will solve the performance regression. </div><div><br><div>I will look at the other changes too.</div></div><div><br></div><div>Gonzalo</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Apr 12, 2015 at 7:14 PM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">G'day,<br>
<br>
In recent OLPC builds I've been carrying a patch to About My Computer<br>
which is mostly XO laptop specific.<br>
<br>
Some of my changes are to support manufacturing verification; to<br>
ensure that the components in the system are as planned.<br>
<br>
I've not proposed the patch upstream, because I'm not sure what parts<br>
are useful to others.<br>
<br>
Now, I'm asking for feedback from Sugar developers.<br>
<br>
The patch does these things:<br>
<br>
- show model and SKU for an XO laptop by reading manufacturing data,<br>
<br>
- show EC firmware version,<br>
<br>
- show wireless driver, card, and capability in terms of IEEE 802.11,<br>
  (though it is missing XO-1 support, and makes capability assumptions<br>
  based on firmware and driver),<br>
<br>
- simplify access to device tree data, factoring redundant code,<br>
<br>
- fix a performance regression; it was taking several seconds to start<br>
  and display, and this was reduced to a quarter second, by backing<br>
  out an earlier change, see <a href="http://dev.laptop.org/ticket/12856" target="_blank">http://dev.laptop.org/ticket/12856</a><br>
<br>
The patch cannot be taken as is because:<br>
<br>
- it will display incorrect data on systems other than XO laptops.<br>
<br>
You can find the patch in the builder:<br>
<br>
<a href="http://dev.laptop.org/git/projects/olpc-os-builder/tree/modules/sugar/kspost.91.patch.inc?id=d38d9608d96a34b768c2cce88a0f618d1143c485" target="_blank">http://dev.laptop.org/git/projects/olpc-os-builder/tree/modules/sugar/kspost.91.patch.inc?id=d38d9608d96a34b768c2cce88a0f618d1143c485</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Gonzalo Odiard<br><br><div>SugarLabs - Software for children learning <br></div></div></div>
</div>