Flent-users discussion archives
 help / color / mirror / Atom feed
* Re: [Flent-users] [tohojo/flent] tc_iterate.c is borken for newer tcs (#146)
       [not found] <tohojo/flent/issues/146@github.com>
@ 2018-08-18 23:19 ` Dave Täht
  2019-01-08 11:34 ` Toke Høiland-Jørgensen
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 4+ messages in thread
From: Dave Täht @ 2018-08-18 23:19 UTC (permalink / raw)
  To: tohojo/flent; +Cc: Subscribed


[-- Attachment #1.1: Type: text/plain, Size: 512 bytes --]

A) we add polling to tc. David liked this, Stephen didn't. I LOVE this.
B) find a way around iproute2 commit e209d4c83dc87470f259122be19afe4401407a06 - which does lookahead to see if the next command can be batched.

It's sad that it's just this past month when I had a major need that this broke. I'll find a way to fix it.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/tohojo/flent/issues/146#issuecomment-414092085

[-- Attachment #1.2: Type: text/html, Size: 3890 bytes --]

[-- Attachment #2: Type: text/plain, Size: 151 bytes --]

_______________________________________________
Flent-users mailing list
Flent-users@flent.org
http://flent.org/mailman/listinfo/flent-users_flent.org

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Flent-users] [tohojo/flent] tc_iterate.c is borken for newer tcs (#146)
       [not found] <tohojo/flent/issues/146@github.com>
  2018-08-18 23:19 ` [Flent-users] [tohojo/flent] tc_iterate.c is borken for newer tcs (#146) Dave Täht
@ 2019-01-08 11:34 ` Toke Høiland-Jørgensen
  2019-01-08 15:06 ` Toke Høiland-Jørgensen
  2019-01-19 14:40 ` Dave Täht
  3 siblings, 0 replies; 4+ messages in thread
From: Toke Høiland-Jørgensen @ 2019-01-08 11:34 UTC (permalink / raw)
  To: tohojo/flent; +Cc: Subscribed


[-- Attachment #1.1: Type: text/plain, Size: 309 bytes --]

So, finally got around to look at this. Wow, that's an annoying change for the Flent use case. Did you find a workaround?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/tohojo/flent/issues/146#issuecomment-452267712

[-- Attachment #1.2: Type: text/html, Size: 1974 bytes --]

[-- Attachment #2: Type: text/plain, Size: 151 bytes --]

_______________________________________________
Flent-users mailing list
Flent-users@flent.org
http://flent.org/mailman/listinfo/flent-users_flent.org

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Flent-users] [tohojo/flent] tc_iterate.c is borken for newer tcs (#146)
       [not found] <tohojo/flent/issues/146@github.com>
  2018-08-18 23:19 ` [Flent-users] [tohojo/flent] tc_iterate.c is borken for newer tcs (#146) Dave Täht
  2019-01-08 11:34 ` Toke Høiland-Jørgensen
@ 2019-01-08 15:06 ` Toke Høiland-Jørgensen
  2019-01-19 14:40 ` Dave Täht
  3 siblings, 0 replies; 4+ messages in thread
From: Toke Høiland-Jørgensen @ 2019-01-08 15:06 UTC (permalink / raw)
  To: tohojo/flent; +Cc: Comment, flent-users


[-- Attachment #1.1: Type: text/plain, Size: 857 bytes --]

flent-users <notifications@github.com> writes:\r
\r
> On Tue, Jan 8, 2019 at 3:34 AM Toke Høiland-Jørgensen <\r
> notifications@github.com> wrote:\r
>\r
>> So, finally got around to look at this. Wow, that's an annoying change for\r
>> the Flent use case. Did you find a workaround?\r
>>\r
>\r
> Other than ripping out the relevant commit or writing the netlnk code from\r
> scratch? No. It's a pretty crippling change. Lemmee forward where the convo\r
> stalled out.\r
\r
Right. I guess I'll go write a monitor mode for iproute2 and see if\r
it'll be accepted upstream...\r
\r
But yeah, Steven's suggestion about just adding a flag was also my first\r
thought...\r
\r
-Toke\r
\r
\r
-- \r
You are receiving this because you commented.\r
Reply to this email directly or view it on GitHub:\r
https://github.com/tohojo/flent/issues/146#issuecomment-452331033

[-- Attachment #1.2: Type: text/html, Size: 3321 bytes --]

[-- Attachment #2: Type: text/plain, Size: 151 bytes --]

_______________________________________________
Flent-users mailing list
Flent-users@flent.org
http://flent.org/mailman/listinfo/flent-users_flent.org

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Flent-users] [tohojo/flent] tc_iterate.c is borken for newer tcs (#146)
       [not found] <tohojo/flent/issues/146@github.com>
                   ` (2 preceding siblings ...)
  2019-01-08 15:06 ` Toke Høiland-Jørgensen
@ 2019-01-19 14:40 ` Dave Täht
  3 siblings, 0 replies; 4+ messages in thread
From: Dave Täht @ 2019-01-19 14:40 UTC (permalink / raw)
  To: tohojo/flent; +Cc: Comment, flent-users


[-- Attachment #1.1: Type: text/plain, Size: 1498 bytes --]

Toke Høiland-Jørgensen <notifications@github.com> writes:\r
\r
> flent-users <notifications@github.com> writes:\r
>\r
>> On Tue, Jan 8, 2019 at 3:34 AM Toke Høiland-Jørgensen <\r
>> notifications@github.com> wrote:\r
>>\r
>>> So, finally got around to look at this. Wow, that's an annoying\r
> change for\r
>>> the Flent use case. Did you find a workaround?\r
>>>\r
>>\r
>> Other than ripping out the relevant commit or writing the netlnk\r
> code from\r
>> scratch? No. It's a pretty crippling change. Lemmee forward where\r
> the convo\r
>> stalled out.\r
>\r
> Right. I guess I'll go write a monitor mode for iproute2 and see if\r
> it'll be accepted upstream...\r
>\r
> But yeah, Steven's suggestion about just adding a flag was also my\r
> first\r
> thought...\r
\r
It would be nice for this facility to work again. I have not been doing\r
any kernel-related work for a while, but will gladly test, as usual, if\r
you come up with something. I happen to prefer the idea of the generic\r
monitoring mode with high resolution timing support. Alternatively\r
updating the common "watch" utilities to have better than 1sec\r
resolution would be good.\r
\r
>\r
> -Toke\r
>\r
> —\r
> You are receiving this because you authored the thread.\r
> Reply to this email directly, view it on GitHub, or mute the thread.\r
\r
\r
-- \r
You are receiving this because you commented.\r
Reply to this email directly or view it on GitHub:\r
https://github.com/tohojo/flent/issues/146#issuecomment-455785969

[-- Attachment #1.2: Type: text/html, Size: 4908 bytes --]

[-- Attachment #2: Type: text/plain, Size: 151 bytes --]

_______________________________________________
Flent-users mailing list
Flent-users@flent.org
http://flent.org/mailman/listinfo/flent-users_flent.org

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-01-19 14:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <tohojo/flent/issues/146@github.com>
2018-08-18 23:19 ` [Flent-users] [tohojo/flent] tc_iterate.c is borken for newer tcs (#146) Dave Täht
2019-01-08 11:34 ` Toke Høiland-Jørgensen
2019-01-08 15:06 ` Toke Høiland-Jørgensen
2019-01-19 14:40 ` Dave Täht

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox