Athp parent dma allocate fix for #3 and #4 #8
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
for sloweer systems I have changed the ATH10K_HTC_WAIT_TIMEOUT_MSEC to 2 seconds instead.
Instead i'll implement a system that unloads the firmware when the capabilities don't come in correctly and the firmware load crashes. kldload and unload should not have to deal with internal driver issues.
…e up to 6 times. add code to force the driver to attempt to retry to load the interface up to 6 times. This should fix kldload issues, as long as the wifi card will work during boot we can automate bringing up the interface in scripts. Thats our first goal.
stopping hif and restarting hif if htc wait target doesnt finish succesfully.
trying to reset the atheros chip to to load up properly on error.
sleep a little to let the device catch up and be ready.
I decided to use pause_sig in the case a signal called us to release before we end sleeping.
need to stop the core its not being called on error. I also have set bmi.done_sent = false.
turned the probe retry in to functional code and added a retry limit to core.h called ATH10K_FW_PROBE_RETRYS
added a function to the process that allows us to cleanup any memory leaks if there are any.
pause_sig isn't being used anywhere else and pause_sig is a delay based function which does not put the thread in the sleep queue so I switched it to tsleep instead.
Grammer correction and add a null check for fwlog locking mutext variable as well as adding a longer timeout for htc connect.
used wrong case for null, woops so use to the many other programming languages with lowercase null.
added mutext checking inside of if_athp_fwlog for some reason its still trying to log on an unitialized mutex, which means somewhere something is still initialized for the first firmware failed load, need to check this out in detail, its weird.
removing annoying warnings that come out of my serial device, so i cant see or type anything.
This reverts commit ad31d5d.
This reverts commit b535d14.
Initialize commit on trying to move the hostap buffer allocation to the athp_parent instead of the add interface section.
wrong type was used.
remove extra pointers.
moved functions in the right order.
removed call to deallocate inside of allocate and the label.
ath10k_htt_rx_ring_refill does a dma allocate so no other lock can be held during this processor that can't be slept, I found that wmi_tx_beacons was holding the conf lock so I created a wait for that.
erikarn
force-pushed
the
master
branch
2 times, most recently
from
c55dc6b to
f1c00bf
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
this fixes both issue #3 and issue #4