Flent-users discussion archives
 help / color / mirror / Atom feed
* Re: [Flent-users] [tohojo/flent] Adding netperf send size params (-m/-M) (#195)
       [not found] <tohojo/flent/issues/195@github.com>
@ 2020-02-06 12:38 ` Toke Høiland-Jørgensen
  2020-02-06 13:14 ` Toke Høiland-Jørgensen
  2020-02-06 15:30 ` Toke Høiland-Jørgensen
  2 siblings, 0 replies; 3+ messages in thread
From: Toke Høiland-Jørgensen @ 2020-02-06 12:38 UTC (permalink / raw)
  To: tohojo/flent; +Cc: Subscribed


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

Very nice find! Hmm, we quite recently added the ability to set UDP packet size (in 00eda5b44f1d), and I guess this is similar.

So maybe a global switch that sets -m for netperf, and also sets the default UDP send size (overridable by the test parameter)? And maybe make --send-size also allow comma-separated values for per-flow setting?

-- 
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/195#issuecomment-582887022

[-- Attachment #1.2: Type: text/html, Size: 2151 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] 3+ messages in thread

* Re: [Flent-users] [tohojo/flent] Adding netperf send size params (-m/-M) (#195)
       [not found] <tohojo/flent/issues/195@github.com>
  2020-02-06 12:38 ` [Flent-users] [tohojo/flent] Adding netperf send size params (-m/-M) (#195) Toke Høiland-Jørgensen
@ 2020-02-06 13:14 ` Toke Høiland-Jørgensen
  2020-02-06 15:30 ` Toke Høiland-Jørgensen
  2 siblings, 0 replies; 3+ messages in thread
From: Toke Høiland-Jørgensen @ 2020-02-06 13:14 UTC (permalink / raw)
  To: tohojo/flent; +Cc: Subscribed


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

Pete Heist <notifications@github.com> writes:

> Ok, it sounds good allowing the comma separated values for setting it
> per-flow, but I'm not sure about sharing it with the UDP setting.
> People usually specify a UDP packet size that's something below the
> MTU right? That would cause pretty high CPU on the stream test. I
> expect when people are lowering this setting to use something like
> 4-32k or so.

Well, we don't have any tests that have both UDP and TCP flows, so for
now that won't be an issue. It may be if we add some, I suppose, but
there's always the test parameter to override the UDP size? We can also
just make it TCP-only, though...


-- 
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/195#issuecomment-582900042

[-- Attachment #1.2: Type: text/html, Size: 2292 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] 3+ messages in thread

* Re: [Flent-users] [tohojo/flent] Adding netperf send size params (-m/-M) (#195)
       [not found] <tohojo/flent/issues/195@github.com>
  2020-02-06 12:38 ` [Flent-users] [tohojo/flent] Adding netperf send size params (-m/-M) (#195) Toke Høiland-Jørgensen
  2020-02-06 13:14 ` Toke Høiland-Jørgensen
@ 2020-02-06 15:30 ` Toke Høiland-Jørgensen
  2 siblings, 0 replies; 3+ messages in thread
From: Toke Høiland-Jørgensen @ 2020-02-06 15:30 UTC (permalink / raw)
  To: tohojo/flent; +Cc: Subscribed


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

Pete Heist <notifications@github.com> writes:

> Sent a pull request but only for setting it globally for TCP so far,
> as setting it per-flow and for the UDP packet size was more than I
> bargained for. :)

No worries, I can add that while merging :)

> The reason I'm probably seeing this more than most people is because I
> have non-default settings for `net.ipv4.tcp_wmem` for doing tests on
> high BDPs. Being able to set this send size as a test parameter would
> be useful, because when I do lower and higher bandwidth tests in the
> same batch, a setting that works well at low bandwidths may be too
> expensive at higher bandwidths. Also why it would be nice if this
> "just worked" in netperf.

Yeah, netperf auto-tuning would be better, of course. But guess we can't
have it all...

> I'll also be trying plots with delivery rate instead, since that comes
> from the kernel, so I may add some more plots to flent with that in
> it.

Sure, go right ahead!

-Toke


-- 
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/195#issuecomment-582959606

[-- Attachment #1.2: Type: text/html, Size: 2686 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] 3+ messages in thread

end of thread, other threads:[~2020-02-06 15:31 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <tohojo/flent/issues/195@github.com>
2020-02-06 12:38 ` [Flent-users] [tohojo/flent] Adding netperf send size params (-m/-M) (#195) Toke Høiland-Jørgensen
2020-02-06 13:14 ` Toke Høiland-Jørgensen
2020-02-06 15:30 ` Toke Høiland-Jørgensen

Flent-users discussion archives

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://lists.flent.org/flent-users

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V1 flent-users flent-users/ https://lists.flent.org/flent-users \
		flent-users@flent.org
	public-inbox-index flent-users

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git