wifi: Hostap WPA control interface configurable timeout #80098

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.

Sign up for GitHub

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

Jump to bottom

Closed

krish2718 wants to merge 2 commits into zephyrproject-rtos:main from krish2718:hostap_configurable_timeout

Contributor

krish2718 commented Oct 19, 2024

WPA control interface timeout is hardcoded to 10s, add a configuration
option to remove the hardcode, this is needed sometimes as a workaround
e.g., crypto taking too long to complete the request.

Work around for #79834, increase the default from 10 to 15s, in positive
case this will have no impact.

krish2718 added the area: Wi-Fi label

zephyrbot requested a review from jukkar

October 19, 2024 18:55

zephyrbot assigned krish2718 and jukkar

zephyrbot commented Oct 19, 2024 •

edited

Loading

The following west manifest projects have been modified in this Pull Request:

Name	Old Revision	New Revision	Diff
hostap	zephyrproject-rtos/hostap@`ac59d28`	zephyrproject-rtos/hostap@`978f1fa` (main)	zephyrproject-rtos/hostap@ac59d287..978f1fa0

Note: This message is automatically posted and updated by the Manifest GitHub Action.

zephyrbot added manifest manifest-hostap DNM labels

krish2718 requested a review from sachinthegreen

October 20, 2024 14:24

fengming-ye approved these changes

View reviewed changes

sachinthegreen previously approved these changes

View reviewed changes

jukkar previously approved these changes

View reviewed changes

krish2718 mentioned this pull request

manifest: hostap: Pull build configuration checks in crypto_alt #79971

Closed

krish2718 added 2 commits

October 21, 2024 16:50


          modules: hostap: Add a config option for WPA control timeout

cdac5fb

WPA control interface timeout is hardcoded to 10s, add a configuration
option to remove the hardcode, this is needed sometimes as a workaround
e.g., crypto taking too long to complete the request.

Work around for zephyrproject-rtos#79834, increase the default from 10 to 15s, in positive
case this will have no impact.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>


          manifest: hostap: Pull fix for WPA control timeout

675fcfa

In case a command timesout due to taking longer time (not a negative
case), use the config option as timeout.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 dismissed stale reviews from jukkar and sachinthegreen via

675fcfa

October 21, 2024 11:21

krish2718 force-pushed the hostap_configurable_timeout branch from 0e9e09c to 675fcfa Compare

October 21, 2024 11:21

zephyrbot removed the DNM label

jukkar approved these changes

View reviewed changes

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] modules: hostap: Add a config option for WPA control t…

dab3ace

…imeout

WPA control interface timeout is hardcoded to 10s, add a configuration
option to remove the hardcode, this is needed sometimes as a workaround
e.g., crypto taking too long to complete the request.

Work around for #79834, increase the default from 10 to 15s, in positive
case this will have no impact.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] manifest: hostap: Pull fix for WPA control timeout

b3b6712

In case a command timesout due to taking longer time (not a negative
case), use the config option as timeout.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] modules: hostap: Add a config option for WPA control t…

3c4c097

…imeout

WPA control interface timeout is hardcoded to 10s, add a configuration
option to remove the hardcode, this is needed sometimes as a workaround
e.g., crypto taking too long to complete the request.

Work around for #79834, increase the default from 10 to 15s, in positive
case this will have no impact.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] manifest: hostap: Pull fix for WPA control timeout

e2e1c86

In case a command timesout due to taking longer time (not a negative
case), use the config option as timeout.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] modules: hostap: Add a config option for WPA control t…

c0f5d52

…imeout

WPA control interface timeout is hardcoded to 10s, add a configuration
option to remove the hardcode, this is needed sometimes as a workaround
e.g., crypto taking too long to complete the request.

Work around for #79834, increase the default from 10 to 15s, in positive
case this will have no impact.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] manifest: hostap: Pull fix for WPA control timeout

9d81775

In case a command timesout due to taking longer time (not a negative
case), use the config option as timeout.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] modules: hostap: Add a config option for WPA control t…

9f9da59

…imeout

WPA control interface timeout is hardcoded to 10s, add a configuration
option to remove the hardcode, this is needed sometimes as a workaround
e.g., crypto taking too long to complete the request.

Work around for #79834, increase the default from 10 to 15s, in positive
case this will have no impact.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] manifest: hostap: Pull fix for WPA control timeout

85bcfe8

In case a command timesout due to taking longer time (not a negative
case), use the config option as timeout.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

jukkar mentioned this pull request

Add hostap configuration build tests #79973

Merged

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] modules: hostap: Add a config option for WPA control t…

84f9899

…imeout

WPA control interface timeout is hardcoded to 10s, add a configuration
option to remove the hardcode, this is needed sometimes as a workaround
e.g., crypto taking too long to complete the request.

Work around for #79834, increase the default from 10 to 15s, in positive
case this will have no impact.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] manifest: hostap: Pull fix for WPA control timeout

In case a command timesout due to taking longer time (not a negative
case), use the config option as timeout.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] modules: hostap: Add a config option for WPA control t…

fdcb55f

…imeout

WPA control interface timeout is hardcoded to 10s, add a configuration
option to remove the hardcode, this is needed sometimes as a workaround
e.g., crypto taking too long to complete the request.

Work around for #79834, increase the default from 10 to 15s, in positive
case this will have no impact.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] manifest: hostap: Pull fix for WPA control timeout

cd07fe3

In case a command timesout due to taking longer time (not a negative
case), use the config option as timeout.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] modules: hostap: Add a config option for WPA control t…

b596601

…imeout

WPA control interface timeout is hardcoded to 10s, add a configuration
option to remove the hardcode, this is needed sometimes as a workaround
e.g., crypto taking too long to complete the request.

Work around for #79834, increase the default from 10 to 15s, in positive
case this will have no impact.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

krish2718 added a commit to krish2718/sdk-zephyr that referenced this pull request


          [nrf fromlist] manifest: hostap: Pull fix for WPA control timeout

In case a command timesout due to taking longer time (not a negative
case), use the config option as timeout.

Upstream PR: zephyrproject-rtos/zephyr#80098

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

sachinthegreen approved these changes

View reviewed changes

jukkar added the DNM label

Member

jukkar commented Oct 22, 2024

I cherry-picked the fix from this PR to #79973, as if this is merged first, the other PR will have a merge conflict and will delay it further. So the plan is to merge #79973 asap in order not to delay other hostap related PRs.

Contributor Author

krish2718 commented Oct 23, 2024

I cherry-picked the fix from this PR to #79973, as if this is merged first, the other PR will have a merge conflict and will delay it further. So the plan is to merge #79973 asap in order not to delay other hostap related PRs.

closing this PR

krish2718 closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Wi-Fi DNM manifest manifest-hostap