* [PATCH v2] i2c: at91: use dma safe buffers
@ 2022-04-07 15:08 Michael Walle
2022-05-02 21:36 ` Michael Walle
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Michael Walle @ 2022-04-07 15:08 UTC (permalink / raw)
To: Codrin Ciubotariu, Nicolas Ferre, Alexandre Belloni,
Claudiu Beznea
Cc: linux-i2c, linux-arm-kernel, linux-kernel, Michael Walle
The supplied buffer might be on the stack and we get the following error
message:
[ 3.312058] at91_i2c e0070600.i2c: rejecting DMA map of vmalloc memory
Use i2c_{get,put}_dma_safe_msg_buf() to get a DMA-able memory region if
necessary.
Fixes: 60937b2cdbf9 ("i2c: at91: add dma support")
Signed-off-by: Michael Walle <michael@walle.cc>
Reviewed-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
---
changes since v1:
- remove extra empty line
- add fixes tag as suggested by Codrin
drivers/i2c/busses/i2c-at91-master.c | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/drivers/i2c/busses/i2c-at91-master.c b/drivers/i2c/busses/i2c-at91-master.c
index b0eae94909f4..5eca3b3bb609 100644
--- a/drivers/i2c/busses/i2c-at91-master.c
+++ b/drivers/i2c/busses/i2c-at91-master.c
@@ -656,6 +656,7 @@ static int at91_twi_xfer(struct i2c_adapter *adap, struct i2c_msg *msg, int num)
unsigned int_addr_flag = 0;
struct i2c_msg *m_start = msg;
bool is_read;
+ u8 *dma_buf;
dev_dbg(&adap->dev, "at91_xfer: processing %d messages:\n", num);
@@ -703,7 +704,17 @@ static int at91_twi_xfer(struct i2c_adapter *adap, struct i2c_msg *msg, int num)
dev->msg = m_start;
dev->recv_len_abort = false;
+ if (dev->use_dma) {
+ dma_buf = i2c_get_dma_safe_msg_buf(m_start, 1);
+ if (!dma_buf) {
+ ret = -ENOMEM;
+ goto out;
+ }
+ dev->buf = dma_buf;
+ }
+
ret = at91_do_twi_transfer(dev);
+ i2c_put_dma_safe_msg_buf(dma_buf, m_start, !ret);
ret = (ret < 0) ? ret : num;
out:
--
2.30.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH v2] i2c: at91: use dma safe buffers 2022-04-07 15:08 [PATCH v2] i2c: at91: use dma safe buffers Michael Walle @ 2022-05-02 21:36 ` Michael Walle 2022-05-04 20:23 ` Wolfram Sang 2022-05-05 14:51 ` Nathan Chancellor 2 siblings, 0 replies; 7+ messages in thread From: Michael Walle @ 2022-05-02 21:36 UTC (permalink / raw) To: Codrin Ciubotariu, Nicolas Ferre, Alexandre Belloni, Claudiu Beznea, wsa Cc: linux-i2c, linux-arm-kernel, linux-kernel Am 2022-04-07 17:08, schrieb Michael Walle: > The supplied buffer might be on the stack and we get the following > error > message: > [ 3.312058] at91_i2c e0070600.i2c: rejecting DMA map of vmalloc > memory > > Use i2c_{get,put}_dma_safe_msg_buf() to get a DMA-able memory region if > necessary. > > Fixes: 60937b2cdbf9 ("i2c: at91: add dma support") > Signed-off-by: Michael Walle <michael@walle.cc> > Reviewed-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> Wolfgang, will you pick this one? -michael _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] i2c: at91: use dma safe buffers 2022-04-07 15:08 [PATCH v2] i2c: at91: use dma safe buffers Michael Walle 2022-05-02 21:36 ` Michael Walle @ 2022-05-04 20:23 ` Wolfram Sang 2022-05-04 20:48 ` Michael Walle 2022-05-05 14:51 ` Nathan Chancellor 2 siblings, 1 reply; 7+ messages in thread From: Wolfram Sang @ 2022-05-04 20:23 UTC (permalink / raw) To: Michael Walle Cc: Alexandre Belloni, linux-kernel, linux-i2c, Codrin Ciubotariu, Claudiu Beznea, linux-arm-kernel [-- Attachment #1.1: Type: text/plain, Size: 561 bytes --] On Thu, Apr 07, 2022 at 05:08:28PM +0200, Michael Walle wrote: > The supplied buffer might be on the stack and we get the following error > message: > [ 3.312058] at91_i2c e0070600.i2c: rejecting DMA map of vmalloc memory > > Use i2c_{get,put}_dma_safe_msg_buf() to get a DMA-able memory region if > necessary. > > Fixes: 60937b2cdbf9 ("i2c: at91: add dma support") > Signed-off-by: Michael Walle <michael@walle.cc> > Reviewed-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> Applied to for-next, thanks! BTW, my name is Wolfram. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] i2c: at91: use dma safe buffers 2022-05-04 20:23 ` Wolfram Sang @ 2022-05-04 20:48 ` Michael Walle 0 siblings, 0 replies; 7+ messages in thread From: Michael Walle @ 2022-05-04 20:48 UTC (permalink / raw) To: Michael Walle Cc: Alexandre Belloni, linux-kernel, linux-i2c, Codrin Ciubotariu, Claudiu Beznea, linux-arm-kernel Am 2022-05-04 22:23, schrieb Wolfram Sang: > BTW, my name is Wolfram. Sorry :/ -michael _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] i2c: at91: use dma safe buffers 2022-04-07 15:08 [PATCH v2] i2c: at91: use dma safe buffers Michael Walle 2022-05-02 21:36 ` Michael Walle 2022-05-04 20:23 ` Wolfram Sang @ 2022-05-05 14:51 ` Nathan Chancellor 2022-05-05 15:11 ` Michael Walle 2 siblings, 1 reply; 7+ messages in thread From: Nathan Chancellor @ 2022-05-05 14:51 UTC (permalink / raw) To: Michael Walle Cc: Alexandre Belloni, llvm, linux-kernel, linux-i2c, Codrin Ciubotariu, Claudiu Beznea, linux-arm-kernel On Thu, Apr 07, 2022 at 05:08:28PM +0200, Michael Walle wrote: > The supplied buffer might be on the stack and we get the following error > message: > [ 3.312058] at91_i2c e0070600.i2c: rejecting DMA map of vmalloc memory > > Use i2c_{get,put}_dma_safe_msg_buf() to get a DMA-able memory region if > necessary. > > Fixes: 60937b2cdbf9 ("i2c: at91: add dma support") > Signed-off-by: Michael Walle <michael@walle.cc> > Reviewed-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> > --- > changes since v1: > - remove extra empty line > - add fixes tag as suggested by Codrin > > drivers/i2c/busses/i2c-at91-master.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/drivers/i2c/busses/i2c-at91-master.c b/drivers/i2c/busses/i2c-at91-master.c > index b0eae94909f4..5eca3b3bb609 100644 > --- a/drivers/i2c/busses/i2c-at91-master.c > +++ b/drivers/i2c/busses/i2c-at91-master.c > @@ -656,6 +656,7 @@ static int at91_twi_xfer(struct i2c_adapter *adap, struct i2c_msg *msg, int num) > unsigned int_addr_flag = 0; > struct i2c_msg *m_start = msg; > bool is_read; > + u8 *dma_buf; > > dev_dbg(&adap->dev, "at91_xfer: processing %d messages:\n", num); > > @@ -703,7 +704,17 @@ static int at91_twi_xfer(struct i2c_adapter *adap, struct i2c_msg *msg, int num) > dev->msg = m_start; > dev->recv_len_abort = false; > > + if (dev->use_dma) { > + dma_buf = i2c_get_dma_safe_msg_buf(m_start, 1); > + if (!dma_buf) { > + ret = -ENOMEM; > + goto out; > + } > + dev->buf = dma_buf; > + } > + > ret = at91_do_twi_transfer(dev); > + i2c_put_dma_safe_msg_buf(dma_buf, m_start, !ret); > > ret = (ret < 0) ? ret : num; > out: > -- > 2.30.2 > > This change as commit 03fbb903c8bf ("i2c: at91: use dma safe buffers") causes the following clang warning: drivers/i2c/busses/i2c-at91-master.c:707:6: error: variable 'dma_buf' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized] if (dev->use_dma) { ^~~~~~~~~~~~ drivers/i2c/busses/i2c-at91-master.c:717:27: note: uninitialized use occurs here i2c_put_dma_safe_msg_buf(dma_buf, m_start, !ret); ^~~~~~~ drivers/i2c/busses/i2c-at91-master.c:707:2: note: remove the 'if' if its condition is always true if (dev->use_dma) { ^~~~~~~~~~~~~~~~~~ drivers/i2c/busses/i2c-at91-master.c:659:13: note: initialize the variable 'dma_buf' to silence this warning u8 *dma_buf; ^ = NULL 1 error generated. Should this variable be initialized or should the call to i2c_put_dma_safe_msg_buf() be moved into the if statement? Cheers, Nathan _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] i2c: at91: use dma safe buffers 2022-05-05 14:51 ` Nathan Chancellor @ 2022-05-05 15:11 ` Michael Walle 2022-05-05 15:18 ` Nathan Chancellor 0 siblings, 1 reply; 7+ messages in thread From: Michael Walle @ 2022-05-05 15:11 UTC (permalink / raw) To: Nathan Chancellor Cc: Alexandre Belloni, llvm, linux-kernel, Wolfram Sang, linux-i2c, Codrin Ciubotariu, Claudiu Beznea, linux-arm-kernel [+ Wolfram] Am 2022-05-05 16:51, schrieb Nathan Chancellor: > On Thu, Apr 07, 2022 at 05:08:28PM +0200, Michael Walle wrote: >> The supplied buffer might be on the stack and we get the following >> error >> message: >> [ 3.312058] at91_i2c e0070600.i2c: rejecting DMA map of vmalloc >> memory >> >> Use i2c_{get,put}_dma_safe_msg_buf() to get a DMA-able memory region >> if >> necessary. >> >> Fixes: 60937b2cdbf9 ("i2c: at91: add dma support") >> Signed-off-by: Michael Walle <michael@walle.cc> >> Reviewed-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> >> --- >> changes since v1: >> - remove extra empty line >> - add fixes tag as suggested by Codrin >> >> drivers/i2c/busses/i2c-at91-master.c | 11 +++++++++++ >> 1 file changed, 11 insertions(+) >> >> diff --git a/drivers/i2c/busses/i2c-at91-master.c >> b/drivers/i2c/busses/i2c-at91-master.c >> index b0eae94909f4..5eca3b3bb609 100644 >> --- a/drivers/i2c/busses/i2c-at91-master.c >> +++ b/drivers/i2c/busses/i2c-at91-master.c >> @@ -656,6 +656,7 @@ static int at91_twi_xfer(struct i2c_adapter *adap, >> struct i2c_msg *msg, int num) >> unsigned int_addr_flag = 0; >> struct i2c_msg *m_start = msg; >> bool is_read; >> + u8 *dma_buf; >> >> dev_dbg(&adap->dev, "at91_xfer: processing %d messages:\n", num); >> >> @@ -703,7 +704,17 @@ static int at91_twi_xfer(struct i2c_adapter >> *adap, struct i2c_msg *msg, int num) >> dev->msg = m_start; >> dev->recv_len_abort = false; >> >> + if (dev->use_dma) { >> + dma_buf = i2c_get_dma_safe_msg_buf(m_start, 1); >> + if (!dma_buf) { >> + ret = -ENOMEM; >> + goto out; >> + } >> + dev->buf = dma_buf; >> + } >> + >> ret = at91_do_twi_transfer(dev); >> + i2c_put_dma_safe_msg_buf(dma_buf, m_start, !ret); >> >> ret = (ret < 0) ? ret : num; >> out: >> -- >> 2.30.2 >> >> > > This change as commit 03fbb903c8bf ("i2c: at91: use dma safe buffers") > causes the following clang warning: Ugh.. > drivers/i2c/busses/i2c-at91-master.c:707:6: error: variable 'dma_buf' > is used uninitialized whenever 'if' condition is false > [-Werror,-Wsometimes-uninitialized] > if (dev->use_dma) { > ^~~~~~~~~~~~ > drivers/i2c/busses/i2c-at91-master.c:717:27: note: uninitialized use > occurs here > i2c_put_dma_safe_msg_buf(dma_buf, m_start, !ret); > ^~~~~~~ > drivers/i2c/busses/i2c-at91-master.c:707:2: note: remove the 'if' if > its condition is always true > if (dev->use_dma) { > ^~~~~~~~~~~~~~~~~~ > drivers/i2c/busses/i2c-at91-master.c:659:13: note: initialize the > variable 'dma_buf' to silence this warning > u8 *dma_buf; > ^ > = NULL > 1 error generated. > > Should this variable be initialized or should the call to > i2c_put_dma_safe_msg_buf() be moved into the if statement? Initializing dma_buf with NULL should to the trick because i2c_put_dma_safe_msg_buf() is handling NULL gracefully. No need for an extra "if (dma_buf)". Will you take care or should I send a patch? -michael _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] i2c: at91: use dma safe buffers 2022-05-05 15:11 ` Michael Walle @ 2022-05-05 15:18 ` Nathan Chancellor 0 siblings, 0 replies; 7+ messages in thread From: Nathan Chancellor @ 2022-05-05 15:18 UTC (permalink / raw) To: Michael Walle Cc: Alexandre Belloni, llvm, linux-kernel, Wolfram Sang, linux-i2c, Codrin Ciubotariu, Claudiu Beznea, linux-arm-kernel On Thu, May 05, 2022 at 05:11:30PM +0200, Michael Walle wrote: > [+ Wolfram] > > Am 2022-05-05 16:51, schrieb Nathan Chancellor: > > On Thu, Apr 07, 2022 at 05:08:28PM +0200, Michael Walle wrote: > > > The supplied buffer might be on the stack and we get the following > > > error > > > message: > > > [ 3.312058] at91_i2c e0070600.i2c: rejecting DMA map of vmalloc > > > memory > > > > > > Use i2c_{get,put}_dma_safe_msg_buf() to get a DMA-able memory region > > > if > > > necessary. > > > > > > Fixes: 60937b2cdbf9 ("i2c: at91: add dma support") > > > Signed-off-by: Michael Walle <michael@walle.cc> > > > Reviewed-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> > > > --- > > > changes since v1: > > > - remove extra empty line > > > - add fixes tag as suggested by Codrin > > > > > > drivers/i2c/busses/i2c-at91-master.c | 11 +++++++++++ > > > 1 file changed, 11 insertions(+) > > > > > > diff --git a/drivers/i2c/busses/i2c-at91-master.c > > > b/drivers/i2c/busses/i2c-at91-master.c > > > index b0eae94909f4..5eca3b3bb609 100644 > > > --- a/drivers/i2c/busses/i2c-at91-master.c > > > +++ b/drivers/i2c/busses/i2c-at91-master.c > > > @@ -656,6 +656,7 @@ static int at91_twi_xfer(struct i2c_adapter > > > *adap, struct i2c_msg *msg, int num) > > > unsigned int_addr_flag = 0; > > > struct i2c_msg *m_start = msg; > > > bool is_read; > > > + u8 *dma_buf; > > > > > > dev_dbg(&adap->dev, "at91_xfer: processing %d messages:\n", num); > > > > > > @@ -703,7 +704,17 @@ static int at91_twi_xfer(struct i2c_adapter > > > *adap, struct i2c_msg *msg, int num) > > > dev->msg = m_start; > > > dev->recv_len_abort = false; > > > > > > + if (dev->use_dma) { > > > + dma_buf = i2c_get_dma_safe_msg_buf(m_start, 1); > > > + if (!dma_buf) { > > > + ret = -ENOMEM; > > > + goto out; > > > + } > > > + dev->buf = dma_buf; > > > + } > > > + > > > ret = at91_do_twi_transfer(dev); > > > + i2c_put_dma_safe_msg_buf(dma_buf, m_start, !ret); > > > > > > ret = (ret < 0) ? ret : num; > > > out: > > > -- > > > 2.30.2 > > > > > > > > > > This change as commit 03fbb903c8bf ("i2c: at91: use dma safe buffers") > > causes the following clang warning: > > Ugh.. > > > drivers/i2c/busses/i2c-at91-master.c:707:6: error: variable 'dma_buf' > > is used uninitialized whenever 'if' condition is false > > [-Werror,-Wsometimes-uninitialized] > > if (dev->use_dma) { > > ^~~~~~~~~~~~ > > drivers/i2c/busses/i2c-at91-master.c:717:27: note: uninitialized use > > occurs here > > i2c_put_dma_safe_msg_buf(dma_buf, m_start, !ret); > > ^~~~~~~ > > drivers/i2c/busses/i2c-at91-master.c:707:2: note: remove the 'if' if > > its condition is always true > > if (dev->use_dma) { > > ^~~~~~~~~~~~~~~~~~ > > drivers/i2c/busses/i2c-at91-master.c:659:13: note: initialize the > > variable 'dma_buf' to silence this warning > > u8 *dma_buf; > > ^ > > = NULL > > 1 error generated. > > > > Should this variable be initialized or should the call to > > i2c_put_dma_safe_msg_buf() be moved into the if statement? > > Initializing dma_buf with NULL should to the trick because > i2c_put_dma_safe_msg_buf() is handling NULL gracefully. No > need for an extra "if (dma_buf)". > > Will you take care or should I send a patch? I can send a patch, thank you for the input! Cheers, Nathan _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-05-05 15:19 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-04-07 15:08 [PATCH v2] i2c: at91: use dma safe buffers Michael Walle 2022-05-02 21:36 ` Michael Walle 2022-05-04 20:23 ` Wolfram Sang 2022-05-04 20:48 ` Michael Walle 2022-05-05 14:51 ` Nathan Chancellor 2022-05-05 15:11 ` Michael Walle 2022-05-05 15:18 ` Nathan Chancellor
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).