Alex Elder df833050cc net: ipa: command payloads already mapped ... IPA transactions describe actions to be performed by the IPA hardware. Three cases use IPA transactions: transmitting a socket buffer; providing a page to receive packet data; and issuing an IPA immediate command. An IPA transaction contains a scatter/gather list (SGL) to hold the set of actions to be performed. We map buffers in the SGL for DMA at the time they are added to the transaction. For skb TX transactions, we fill the SGL with a call to skb_to_sgvec(). Page RX transactions involve a single page pointer, and that is recorded in the SGL with sg_set_page(). In both of these cases we then map the SGL for DMA with a call to dma_map_sg(). Immediate commands are different. The payload for an immediate command comes from a region of coherent DMA memory, which must *not* be mapped for DMA. For that reason, gsi_trans_cmd_add() sort of hand-crafts each SGL entry added to a command transaction. This patch fixes a problem with the code that crafts the SGL entry for an immediate command. Previously a portion of the SGL entry was updated using sg_set_buf(). However this is not valid because it includes a call to virt_to_page() on the buffer, but the command buffer pointer is not a linear address. Since we never actually map the SGL for command transactions, there are very few fields in the SGL we need to fill. Specifically, we only need to record the DMA address and the length, so they can be used by __gsi_trans_commit() to fill a TRE. We additionally need to preserve the SGL flags so for_each_sg() still works. For that we can simply assign a null page pointer for command SGL entries. Fixes: 9dd441e4ed ("soc: qcom: ipa: GSI transactions") Reported-by: Stephen Boyd <swboyd@chromium.org> Tested-by: Stephen Boyd <swboyd@chromium.org> Signed-off-by: Alex Elder <elder@linaro.org> Link: https://lore.kernel.org/r/20201022010029.11877-1-elder@linaro.org Signed-off-by: Jakub Kicinski <kuba@kernel.org>		2020-10-23 18:28:52 -07:00
..
gsi_private.h
gsi_reg.h
gsi_trans.c
gsi_trans.h
gsi.c
gsi.h
ipa_clock.c
ipa_clock.h
ipa_cmd.c
ipa_cmd.h
ipa_data-sc7180.c
ipa_data-sdm845.c
ipa_data.h
ipa_endpoint.c
ipa_endpoint.h
ipa_gsi.c
ipa_gsi.h
ipa_interrupt.c
ipa_interrupt.h
ipa_main.c
ipa_mem.c
ipa_mem.h
ipa_modem.c
ipa_modem.h
ipa_qmi_msg.c
ipa_qmi_msg.h
ipa_qmi.c
ipa_qmi.h
ipa_reg.c
ipa_reg.h
ipa_smp2p.c
ipa_smp2p.h
ipa_table.c
ipa_table.h
ipa_uc.c
ipa_uc.h
ipa_version.h
ipa.h
Kconfig
Makefile