pico-examples icon indicating copy to clipboard operation
pico-examples copied to clipboard

Add iperf client example

Open peterharperuk opened this issue 3 years ago • 10 comments

Requires you to specify the server ip address when running cmake, e.g. -DIPERF_SERVER_IP=1.2.3.4

Fixes #255

peterharperuk avatar Aug 15 '22 11:08 peterharperuk

Perhaps the top-level README.md also wants updating to match?

lurch avatar Aug 15 '22 12:08 lurch

Perhaps the top-level README.md also wants updating to match?

ok - tweak the wording

peterharperuk avatar Aug 15 '22 14:08 peterharperuk

I know nothing about FreeRTOS, but looking at the content of the diff in this PR, does xstr need removing from picow_freertos_iperf.c too? :shrug:

lurch avatar Aug 15 '22 22:08 lurch

I know nothing about FreeRTOS, but looking at the content of the diff in this PR, does xstr need removing from picow_freertos_iperf.c too? 🤷

Oops - yes. I should test it really!

peterharperuk avatar Aug 16 '22 09:08 peterharperuk

@kilograham / @liamfraser I wonder if it might be worth modifying https://github.com/raspberrypi/pico-examples/blob/master/.github/workflows/cmake.yml to do several different builds, with different combinations of #define s set? :man_shrugging:

lurch avatar Aug 16 '22 09:08 lurch

Added iperf client to freertos and checked they work.

peterharperuk avatar Aug 16 '22 15:08 peterharperuk

@kilograham / @liamfraser I wonder if it might be worth modifying https://github.com/raspberrypi/pico-examples/blob/master/.github/workflows/cmake.yml to do several different builds, with different combinations of #define s set? 🤷‍♂️

Maybe this should be a separate issue? I don't really understand how the builds are configured but it looks a bit like pico_w examples are not built? To build everything...

-DPICO_BOARD=pico_w -DWIFI_SSID='xxx' -DWIFI_PASSWORD='xxx' -DFREERTOS_KERNEL_PATH=/source/FreeRTOS-Kernel

peterharperuk avatar Aug 16 '22 15:08 peterharperuk

Maybe this should be a separate issue?

I'll let @kilograham decide if he wants to create an issue for it, since he knows far more about the SDK (and how its builds work) than I do :wink:

lurch avatar Aug 16 '22 16:08 lurch

...although I think using -DPICO_BOARD=pico_w still wouldn't build everything, as it then wouldn't build things requiring PICO_DEFAULT_LED_PIN ? :man_shrugging: (so a multiple-builds process might be needed?)

lurch avatar Aug 16 '22 16:08 lurch

True - it should build for all boards

peterharperuk avatar Aug 16 '22 16:08 peterharperuk

I can't be bothered with this

peterharperuk avatar Feb 02 '23 18:02 peterharperuk