diff options
author | Mike Tipton <[email protected]> | 2020-04-15 16:03:27 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <[email protected]> | 2020-04-23 16:18:07 +0200 |
commit | 9c0c54a8daf465e6e6454ffe6fc999e81a22dbf9 (patch) | |
tree | 3dd4dfd0d759c43c84689182cd5c5d59b0985938 /scripts/gdb/linux | |
parent | 2d3008f658d32d82324717f44432bd4e8dc14c15 (diff) |
interconnect: qcom: Fix uninitialized tcs_cmd::wait
Currently, if tcs_cmd_gen is called with commit=false, then
tcs_cmd::wait is left uninitialized. Since the tcs_cmd structures passed
to this function aren't zero-initialized, then we're left with random
wait values. This results in waiting for completion for more commands
than is necessary, depending on what's on the stack at the time.
Removing the unnecessary if-condition fixes this, but add an explicit
memset of the tcs_cmd structure as well to ensure predictable behavior
if more tcs_cmd members are added in the future.
Fixes: 976daac4a1c5 ("interconnect: qcom: Consolidate interconnect RPMh support")
Signed-off-by: Mike Tipton <[email protected]>
Reviewed-by: Bjorn Andersson <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Georgi Djakov <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Greg Kroah-Hartman <[email protected]>
Diffstat (limited to 'scripts/gdb/linux')
0 files changed, 0 insertions, 0 deletions