From: Mikael Abrahamsson <swmike@swm.pp.se>
To: Sebastian Moeller <moeller0@gmx.de>
Cc: "cerowrt-devel@lists.bufferbloat.net"
<cerowrt-devel@lists.bufferbloat.net>
Subject: Re: [Cerowrt-devel] performance numbers from WRT1200AC (Re: Latest build test - new sqm-scripts seem to work; "cake overhead 40" didn't)
Date: Mon, 29 Jun 2015 09:54:37 +0200 (CEST) [thread overview]
Message-ID: <alpine.DEB.2.02.1506290952050.9487@uplift.swm.pp.se> (raw)
In-Reply-To: <F3088E9D-E022-4CFF-AE1A-D3A0934FDDAF@gmx.de>
[-- Attachment #1: Type: TEXT/PLAIN, Size: 610 bytes --]
On Mon, 29 Jun 2015, Sebastian Moeller wrote:
> Good work-around; not a real solution as sqm_logger() will also
> add "SQM:”. I think the solution most likely is to remove line 155
> completely. But weirdly this seems to work with cerowrt…
May I suggest a wrapper around the logger part that handles this case, ie
if the string is empty? Because when the string is empty and it gets
stuck, luci just waits and waits and waits. I mean, this specific case can
be fixed, but for the future it would be more robust if this error case
can't happen at all.
--
Mikael Abrahamsson email: swmike@swm.pp.se
next prev parent reply other threads:[~2015-06-29 7:54 UTC|newest]
Thread overview: 119+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-19 14:44 [Cerowrt-devel] Latest build test - new sqm-scripts seem to work; "cake overhead 40" didn't Alan Jenkins
2015-06-19 16:41 ` Sebastian Moeller
2015-06-19 17:35 ` Alan Jenkins
2015-06-19 20:12 ` Dave Taht
2015-06-19 20:40 ` Alan Jenkins
2015-06-19 20:51 ` Dave Taht
2015-06-19 20:57 ` Dave Taht
2015-06-19 20:57 ` Alan Jenkins
2015-06-19 21:06 ` Dave Taht
2015-06-19 21:24 ` Dave Taht
2015-06-19 21:52 ` Luis E. Garcia
2015-06-19 23:32 ` Dave Taht
2015-06-20 5:52 ` Sebastian Moeller
2015-06-23 2:41 ` Jim Reisert AD1C
2015-06-23 7:20 ` Sebastian Moeller
2015-06-23 12:55 ` [Cerowrt-devel] performance numbers from WRT1200AC (Re: Latest build test - new sqm-scripts seem to work; "cake overhead 40" didn't) Mikael Abrahamsson
2015-06-23 14:09 ` Dave Taht
2015-06-23 17:25 ` Sebastian Moeller
2015-06-23 18:15 ` Jonathan Morton
2015-06-24 5:21 ` Mikael Abrahamsson
2015-06-24 5:19 ` Mikael Abrahamsson
2015-06-24 11:31 ` Mikael Abrahamsson
2015-06-24 16:32 ` Dave Taht
2015-06-25 1:53 ` Aaron Wood
2015-06-25 3:07 ` Mikael Abrahamsson
2015-06-25 3:32 ` Aaron Wood
2015-06-25 9:12 ` Mikael Abrahamsson
2015-06-25 10:26 ` Mikael Abrahamsson
2015-06-25 20:13 ` Dave Taht
2015-06-25 20:16 ` Dave Taht
2015-06-25 20:24 ` Toke Høiland-Jørgensen
2015-06-25 22:14 ` Dave Taht
2015-06-26 6:58 ` Mikael Abrahamsson
2015-06-26 7:12 ` Mikael Abrahamsson
2015-06-26 9:46 ` Sebastian Moeller
2015-06-26 12:26 ` Mikael Abrahamsson
2015-06-26 14:17 ` Sebastian Moeller
2015-06-26 14:49 ` Mikael Abrahamsson
2015-06-26 16:18 ` Jonathan Morton
2015-06-26 16:31 ` Mikael Abrahamsson
2015-06-26 16:35 ` Jonathan Morton
2015-06-26 17:04 ` Dave Taht
2015-06-26 18:24 ` Dave Taht
2015-06-26 18:38 ` Mikael Abrahamsson
2015-06-26 18:58 ` Dave Taht
2015-06-26 18:59 ` Dave Taht
2015-06-26 19:11 ` Mikael Abrahamsson
2015-06-26 19:13 ` Dave Taht
2015-06-27 5:03 ` Mikael Abrahamsson
2015-06-27 5:18 ` Dave Taht
2015-06-27 5:50 ` Mikael Abrahamsson
2015-06-27 17:59 ` Dave Taht
2015-06-27 18:23 ` Mikael Abrahamsson
2015-06-27 18:52 ` Dave Taht
2015-06-27 23:13 ` Sebastian Moeller
2015-06-28 7:06 ` Mikael Abrahamsson
2015-06-28 8:23 ` Sebastian Moeller
2015-06-28 10:29 ` Mikael Abrahamsson
2015-06-28 17:04 ` Sebastian Moeller
2015-06-28 17:32 ` Mikael Abrahamsson
2015-06-28 17:58 ` Jonathan Morton
2015-06-28 18:04 ` Dave Taht
2015-06-28 18:55 ` Sebastian Moeller
2015-06-28 19:17 ` Mikael Abrahamsson
2015-06-28 19:24 ` Sebastian Moeller
2015-06-28 20:48 ` Mikael Abrahamsson
2015-06-28 21:23 ` Sebastian Moeller
2015-06-29 4:58 ` Mikael Abrahamsson
2015-06-29 5:11 ` Mikael Abrahamsson
2015-06-29 7:46 ` Sebastian Moeller
2015-06-29 7:54 ` Mikael Abrahamsson [this message]
2015-06-29 7:56 ` Sebastian Moeller
2015-06-29 8:10 ` Sebastian Moeller
2015-06-29 8:17 ` Mikael Abrahamsson
2015-06-29 8:24 ` Sebastian Moeller
2015-06-29 7:44 ` Sebastian Moeller
2015-06-29 8:09 ` Mikael Abrahamsson
2015-06-29 8:34 ` Sebastian Moeller
2015-06-29 8:42 ` Sebastian Moeller
2015-06-29 9:12 ` Mikael Abrahamsson
2015-06-29 10:09 ` Toke Høiland-Jørgensen
2015-06-29 13:00 ` Mikael Abrahamsson
2015-06-29 13:34 ` Sebastian Moeller
2015-06-29 13:46 ` dpreed
2015-06-29 16:45 ` Jonathan Morton
2015-06-30 13:58 ` [Cerowrt-devel] Build instructions for regular OpenWRT with Ceropackages Mikael Abrahamsson
2015-06-30 16:20 ` dpreed
2015-06-30 19:58 ` Mikael Abrahamsson
2015-07-01 8:23 ` David Lang
2015-07-01 10:32 ` Mikael Abrahamsson
2015-07-01 11:55 ` Sebastian Moeller
2015-07-01 15:37 ` dpreed
2015-06-29 13:42 ` [Cerowrt-devel] performance numbers from WRT1200AC (Re: Latest build test - new sqm-scripts seem to work; "cake overhead 40" didn't) Sebastian Moeller
2015-06-29 16:44 ` Dave Taht
2015-06-29 18:24 ` Sebastian Moeller
2015-06-29 22:15 ` Mikael Abrahamsson
2015-06-29 22:49 ` Mikael Abrahamsson
2015-06-30 8:00 ` Mikael Abrahamsson
2015-06-30 9:40 ` Mikael Abrahamsson
2015-07-02 15:33 ` Mikael Abrahamsson
2015-07-02 15:39 ` Toke Høiland-Jørgensen
2015-07-02 15:43 ` Mikael Abrahamsson
2015-07-02 15:47 ` Toke Høiland-Jørgensen
2015-07-02 16:06 ` dpreed
2015-07-02 19:12 ` Mikael Abrahamsson
2015-07-07 1:07 ` David Lang
2015-07-02 16:09 ` Rich Brown
2015-07-02 16:12 ` Toke Høiland-Jørgensen
2015-07-03 11:38 ` Mikael Abrahamsson
2015-06-29 6:12 ` Mikael Abrahamsson
2015-06-28 18:48 ` Sebastian Moeller
2015-06-26 16:34 ` Dave Taht
2015-06-26 16:27 ` Sebastian Moeller
2015-06-26 16:36 ` Mikael Abrahamsson
2015-06-26 16:43 ` Dave Taht
2015-06-26 17:01 ` Mikael Abrahamsson
2015-06-23 14:35 ` [Cerowrt-devel] Latest build test - new sqm-scripts seem to work; "cake overhead 40" didn't Jim Reisert AD1C
2015-06-23 14:40 ` Dave Taht
2015-06-19 20:37 ` Sebastian Moeller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://lists.bufferbloat.net/postorius/lists/cerowrt-devel.lists.bufferbloat.net/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.2.02.1506290952050.9487@uplift.swm.pp.se \
--to=swmike@swm.pp.se \
--cc=cerowrt-devel@lists.bufferbloat.net \
--cc=moeller0@gmx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox