Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ESP32 FG mode, stm32mp131 host, sdio mode] WiFi transmission rate #497

Open
3 tasks done
renliangli opened this issue Sep 24, 2024 · 23 comments
Open
3 tasks done

[ESP32 FG mode, stm32mp131 host, sdio mode] WiFi transmission rate #497

renliangli opened this issue Sep 24, 2024 · 23 comments

Comments

@renliangli
Copy link

Checklist

  • Checked the issue tracker for similar issues to ensure this is not a duplicate
  • Read the documentation to confirm the issue is not addressed there and your configuration is set correctly
  • Tested with the latest version to ensure the issue hasn't been fixed

How often does this bug occurs?

always

Expected behavior

WiFi transmission speed can reach the expected level

Actual behavior (suspected bug)

The actual transmission rate of WiFi downlink TCP is 1.18 Mbits/sec
image

raw transport throughput
image

Error logs or terminal output

root@sofar:/# dmesg | grep mmc
[    2.651651] mmci-pl18x 58007000.mmc: mmc0: PL180 manf 53 rev2 at 0x58007000 irq 45,0 (pio)
[    2.716092] mmc0: queuing unknown CIS tuple 0x01 [d9 01 ff] (3 bytes)
[    2.732790] mmc0: queuing unknown CIS tuple 0x1a [01 01 00 02 07] (5 bytes)
[    2.742361] mmc0: queuing unknown CIS tuple 0x1b [c1 41 30 30 ff ff ff ff] (8 bytes)
[    2.759702] mmc0: new SDIO card at address 0001
root@sofar:/#
root@sofar:/#
root@sofar:/# cat /sys/kernel/debug/mmc0/ios
clock:          25000000 Hz
actual clock:   25000000 Hz
vdd:            21 (3.3 ~ 3.4 V)
bus mode:       2 (push-pull)
chip select:    0 (don't care)
power mode:     2 (on)
bus width:      2 (4 bits)
timing spec:    0 (legacy)
signal voltage: 0 (3.30 V)
driver type:    0 (driver type B)

Steps to reproduce the behavior

1、insmod esp32_sdio.ko
2、iperf3 -s -i 1 -p 2222
3、iperf3 -c 192.168.2.144 -p 2222 -i 1 -t 30 -P 5 -R

Project release version

master

System architecture

other (details in Additional context)

Operating system

Linux

Operating system version

Linux version 5.15.67

Shell

sh

Additional context

No response

@mantriyogesh
Copy link
Collaborator

before running iperf, check issues in raw throughput and stabilise them.

Please send photo captured of your setup.

Also, please ensure that the porting guide is followed.

@renliangli
Copy link
Author

raw transport throughput
image

image

We tested it on the PCB board, and SDIO has pull-up capabilities

@mantriyogesh
Copy link
Collaborator

mantriyogesh commented Sep 24, 2024

  1. Something is wrong yet.
    Also, did you test raw throughput in opposite direction?

  2. Setup photo please? take close-up enough to possibly show the connections.

  3. Is it ESP32 or ESp32-C6 for sdio slave?
    If ESP32, eFuse burning is done (check if required or not first)?

  4. Pull-up values?

@renliangli
Copy link
Author

Host --> ESP
image

43f7cf3a9bcabe730498905d52689fb

ESP32

The pull-up resistance value is 4.7K

@renliangli
Copy link
Author

Also, when will esp_hosted_ng release a version that supports SoftAP ?

@mantriyogesh
Copy link
Collaborator

4.7K is not acceptable value, btw. check the porting guide. Try to stick with the values we use.
Minimal recommended is 10K, we use 51K. Expected on lines: CMD, DAT0,DAT1,DAT2,DAT3.

For ESP32-WROOM32, the efuse_burning is mandatory.

From above log, host tx seems good (still there is chance to be higher, but at is fair enough).
however rx not.

Can you let us know what (blue and yellow) extra wires added beside esp32?

@mantriyogesh
Copy link
Collaborator

NG supports softAP already.

@renliangli
Copy link
Author

efuse_burning has been done as required.

extra wires (blue and yellow) added beside esp32 is serial port firmware burning related cables.

@renliangli
Copy link
Author

NG supports softAP already.

Does the latest NG code on the master branch already support SoftAp? When compiling ko, just add CONFIG_AP_SUPPORT,
Is there anything that needs to be modified on the ESP side ?

@mantriyogesh
Copy link
Collaborator

Just add
CONFIG_AP_SUPPORT = y
at top.

Latest ng firmware to be used from master. It should already support softap.

@renliangli
Copy link
Author

Just add CONFIG_AP_SUPPORT = y at top.

Latest ng firmware to be used from master. It should already support softap.

NG no longer supports softap+station ?

@mantriyogesh
Copy link
Collaborator

NG doesn't yet support softap+station.
FG supports it.

@renliangli
Copy link
Author

NG doesn't yet support softap+station. FG supports it.

How long does it take for NG support softap+station

@mantriyogesh
Copy link
Collaborator

It is not currently planned. so can't commit.
If your use case is specific to station + softap, I would recommend using FG. it is supported there.

@mantriyogesh
Copy link
Collaborator

  1. Did the raw throughput test both sides worked fine in your case @renliangli ?
  2. Did you manage to change the pull-ups?

@renliangli
Copy link
Author

  1. Did the raw throughput test both sides worked fine in your case @renliangli ?
    yeah, The raw throughput test both sides worked fine, Above 20Mb.

how to enable wifi atuo reconnect after disconnected in FG station wifi mode ?

@mantriyogesh
Copy link
Collaborator

Short answer

Customise disconnected: Add hook of test_station_mode_connect() here

Long answer

  1. Doc: Control path understanding
  2. Doc: You would anyway arrive here through above link, just to point, Event Handler APIs
  3. Doc: C demo Code: C demo -> test.c I think you are already past this.
  4. Code: Event callbacks registered here
  5. Code: Event callbacks hooks:
    case CTRL_EVENT_ESP_INIT: {
    printf("%s App EVENT: ESP INIT\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE));
    break;
    } case CTRL_EVENT_HEARTBEAT: {
    printf("%s App EVENT: Heartbeat event [%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE),
    app_event->u.e_heartbeat.hb_num);
    break;
    } case CTRL_EVENT_STATION_CONNECTED_TO_AP: {
    event_sta_conn_t *p_e = &app_event->u.e_sta_conn;
    printf("%s App EVENT: STA-Connected ssid[%s] bssid[%s] channel[%d] auth[%d] aid[%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE), p_e->ssid,
    p_e->bssid, p_e->channel, p_e->authmode, p_e->aid);
    break;
    } case CTRL_EVENT_STATION_DISCONNECT_FROM_AP: {
    event_sta_disconn_t *p_e = &app_event->u.e_sta_disconn;
    printf("%s App EVENT: STA-Disconnected reason[%d] ssid[%s] bssid[%s] rssi[%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE), p_e->reason, p_e->ssid,
    p_e->bssid, p_e->rssi);
    break;
    } case CTRL_EVENT_STATION_CONNECTED_TO_ESP_SOFTAP: {
    event_softap_sta_conn_t *p_e = &app_event->u.e_softap_sta_conn;
    char *p = (char *)p_e->mac;
    if (p && strlen(p)) {
    printf("%s App EVENT: SoftAP mode: Connected MAC[%s] aid[%d] is_mesh_child[%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE),
    p, p_e->aid, p_e->is_mesh_child);
    }
    break;
    } case CTRL_EVENT_STATION_DISCONNECT_FROM_ESP_SOFTAP: {
    event_softap_sta_disconn_t *p_e = &app_event->u.e_softap_sta_disconn;
    char *p = (char *)p_e->mac;
    if (p && strlen(p)) {
    printf("%s App EVENT: SoftAP mode: Disconnect MAC[%s] reason[%d] aid[%d] is_mesh_child[%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE),
    p, p_e->reason, p_e->aid, p_e->is_mesh_child);
    }
    break;
  6. Customise disconnected: Add hook of test_station_mode_connect() here

Note

  1. Try to use async event handlers when using in event handlers.
    We faced some issues in past for 'sync' handlers when used in event handlers.
    Except event handler hooks, in general, preference should always to 'sync' APIs.

  2. Diff between sync and async:
    In Async, you register response callback. In sync, you do not register response callback.

Let us know if you face any difficulty.

@renliangli
Copy link
Author

Short answer

Customise disconnected: Add hook of test_station_mode_connect() here

Long answer

  1. Doc: Control path understanding
  2. Doc: You would anyway arrive here through above link, just to point, Event Handler APIs
  3. Doc: C demo Code: C demo -> test.c I think you are already past this.
  4. Code: Event callbacks registered here
  5. Code: Event callbacks hooks:
    case CTRL_EVENT_ESP_INIT: {
    printf("%s App EVENT: ESP INIT\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE));
    break;
    } case CTRL_EVENT_HEARTBEAT: {
    printf("%s App EVENT: Heartbeat event [%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE),
    app_event->u.e_heartbeat.hb_num);
    break;
    } case CTRL_EVENT_STATION_CONNECTED_TO_AP: {
    event_sta_conn_t *p_e = &app_event->u.e_sta_conn;
    printf("%s App EVENT: STA-Connected ssid[%s] bssid[%s] channel[%d] auth[%d] aid[%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE), p_e->ssid,
    p_e->bssid, p_e->channel, p_e->authmode, p_e->aid);
    break;
    } case CTRL_EVENT_STATION_DISCONNECT_FROM_AP: {
    event_sta_disconn_t *p_e = &app_event->u.e_sta_disconn;
    printf("%s App EVENT: STA-Disconnected reason[%d] ssid[%s] bssid[%s] rssi[%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE), p_e->reason, p_e->ssid,
    p_e->bssid, p_e->rssi);
    break;
    } case CTRL_EVENT_STATION_CONNECTED_TO_ESP_SOFTAP: {
    event_softap_sta_conn_t *p_e = &app_event->u.e_softap_sta_conn;
    char *p = (char *)p_e->mac;
    if (p && strlen(p)) {
    printf("%s App EVENT: SoftAP mode: Connected MAC[%s] aid[%d] is_mesh_child[%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE),
    p, p_e->aid, p_e->is_mesh_child);
    }
    break;
    } case CTRL_EVENT_STATION_DISCONNECT_FROM_ESP_SOFTAP: {
    event_softap_sta_disconn_t *p_e = &app_event->u.e_softap_sta_disconn;
    char *p = (char *)p_e->mac;
    if (p && strlen(p)) {
    printf("%s App EVENT: SoftAP mode: Disconnect MAC[%s] reason[%d] aid[%d] is_mesh_child[%d]\n",
    get_timestamp(ts, MIN_TIMESTAMP_STR_SIZE),
    p, p_e->reason, p_e->aid, p_e->is_mesh_child);
    }
    break;
  6. Customise disconnected: Add hook of test_station_mode_connect() here

Note

  1. Try to use async event handlers when using in event handlers.
    We faced some issues in past for 'sync' handlers when used in event handlers.
    Except event handler hooks, in general, preference should always to 'sync' APIs.
  2. Diff between sync and async:
    In Async, you register response callback. In sync, you do not register response callback.

Let us know if you face any difficulty.

Your method is to listen for the disconnection message and then restart the connection on the host side.

What I want is for the ESP32 chip station to be able to reconnect to the previously set AP hotspot by itself when rescanning the hotspot.

@mantriyogesh
Copy link
Collaborator

What I want is for the ESP32 chip station to be able to reconnect to the previously set AP hotspot by itself when rescanning the hotspot.

Just to understand this clearly, so do you mean that one every reboot, esp should auto connect to the prior AP (or router) it was connected to.
This should happen even when host is not triggering connect explicitly.

If host triggered connect with new credentials, these new credentials should become default credentials.

Is this the complete context? Anything I missed?

@renliangli
Copy link
Author

What I want is for the ESP32 chip station to be able to reconnect to the previously set AP hotspot by itself when rescanning the hotspot.

Just to understand this clearly, so do you mean that one every reboot, esp should auto connect to the prior AP (or router) it was connected to. This should happen even when host is not triggering connect explicitly.

If host triggered connect with new credentials, these new credentials should become default credentials.

Is this the complete context? Anything I missed?

In addition to reboot, there is another scenario where the station disconnects when the signal is poor, and the ESP32 chip can automatically reconnect when the signal is restored

@mantriyogesh
Copy link
Collaborator

The main intention is that host should be notified on disconnect as soon as possible.

May be we retry 1-2 times and then send the disconnect to host.

The main problem here is that host wouldn't even know that the network is disconnected.

This is a typical ownership issue. In hosted ideology, we intend to control slave from host and host should be always and ASAP consistent with esp state.
In general, host should auto-connect on receiving the disconnect. ESP cannot send disconnect notification to host and at the same time, retry connecting without host knowing about it. Host would also trigger to connect (basically both parties start to issue connect) at the same time as that of esp, then the state machines would go for toss in esp_wifi.

As workaround,
What we can do, is introduce some MACRO, AUTO_DISCONNECT_AT_SLAVE in slave_control.c to auto connect on disconnect, by default disabled. AUTO_DISCONNECT_AT_SLAVE_TIMEOUT and AUTO_DISCONNECT_AT_SLAVE_RETRY

When this macro is enabled and flashed, the slave would try to reconnect with above config, before sending the disconnect notification to host.

Note

Word of caution: Unless this disconnect is received at host, host still thinks that it is connected (which is or should be transient state, but still invalid state)

Please let me know if this covers the scenario?

@renliangli
Copy link
Author

right

@mantriyogesh
Copy link
Collaborator

We will make this change and notify you, once fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants