<div dir="ltr"><div style="font-family:arial,sans-serif;font-size:13px">We can't optimize the strcspn(t, WHITESPACE); to a strchr(t, '\n') - t; because of the " kB" that meminfo and a few other places have, that we want to drop.</div>
<div style="font-family:arial,sans-serif;font-size:13px"><br></div><span style="font-family:arial,sans-serif;font-size:13px">get_status_field() has to be run every time we log a message, and I am starting to not like this function at all. I think we should go back to having it hand-rolled in get_process_capeff(), at least for that one in the hot-path.</span><br>
<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">patch coming...</span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Tue, Sep 17, 2013 at 4:39 PM, Zbigniew Jędrzejewski-Szmek <span dir="ltr"><<a href="mailto:zbyszek@in.waw.pl" target="_blank">zbyszek@in.waw.pl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Tue, Sep 17, 2013 at 03:48:46PM -0700, Shawn Landden wrote:<br>
> 69ab80881552d5f79ca95f6b3be48ad122ab1ec2 tried to unify<br>
> parsing of status files, but removed the logic of skipping<br>
> extra '0's when getting the effective capabilities. Restore<br>
> that logic.<br>
</div>Oh, I removed the logic on purpose, because I didn't know what<br>
it was for. I applied a different fix now, which is a bit hacky,<br>
but simplifies our hot path a bit. I've also added some comments<br>
to make it clearer why 0's are skipped.<br>
<br>
Zbyszek<br>
_______________________________________________<br>
systemd-devel mailing list<br>
<a href="mailto:systemd-devel@lists.freedesktop.org">systemd-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/systemd-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/systemd-devel</a><br>
</blockquote></div><br></div>