Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1343292 > unrolled thread
| Started by | Bjorn Andersson <bjorn.andersson@linaro.org> |
|---|---|
| First post | 2016-02-25 17:20 +0100 |
| Last post | 2016-03-01 18:30 +0100 |
| Articles | 4 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] soc: qcom: smp2p: Use memset_io to zero out new entries Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-02-25 17:20 +0100
Re: [PATCH] soc: qcom: smp2p: Use memset_io to zero out new entries Andy Gross <andy.gross@linaro.org> - 2016-02-25 23:10 +0100
Re: [PATCH] soc: qcom: smp2p: Use memset_io to zero out new entries Paul Walmsley <paul@pwsan.com> - 2016-03-01 18:20 +0100
Re: [PATCH] soc: qcom: smp2p: Use memset_io to zero out new entries Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-03-01 18:30 +0100
| From | Bjorn Andersson <bjorn.andersson@linaro.org> |
|---|---|
| Date | 2016-02-25 17:20 +0100 |
| Subject | [PATCH] soc: qcom: smp2p: Use memset_io to zero out new entries |
| Message-ID | <r66NY-6of-3@gated-at.bofh.it> |
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> --- drivers/soc/qcom/smp2p.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/qcom/smp2p.c b/drivers/soc/qcom/smp2p.c index f1eed7f9dd67..61df598444e6 100644 --- a/drivers/soc/qcom/smp2p.c +++ b/drivers/soc/qcom/smp2p.c @@ -379,7 +379,7 @@ static int qcom_smp2p_alloc_outbound_item(struct qcom_smp2p *smp2p) return PTR_ERR(out); } - memset(out, 0, sizeof(*out)); + memset_io(out, 0, sizeof(*out)); out->magic = SMP2P_MAGIC; out->local_pid = smp2p->local_pid; out->remote_pid = smp2p->remote_pid; -- 2.5.0
[toc] | [next] | [standalone]
| From | Andy Gross <andy.gross@linaro.org> |
|---|---|
| Date | 2016-02-25 23:10 +0100 |
| Message-ID | <r6cgG-1Uy-23@gated-at.bofh.it> |
| In reply to | #1343292 |
On Thu, Feb 25, 2016 at 08:17:37AM -0800, Bjorn Andersson wrote: > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > --- > drivers/soc/qcom/smp2p.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/soc/qcom/smp2p.c b/drivers/soc/qcom/smp2p.c > index f1eed7f9dd67..61df598444e6 100644 > --- a/drivers/soc/qcom/smp2p.c > +++ b/drivers/soc/qcom/smp2p.c > @@ -379,7 +379,7 @@ static int qcom_smp2p_alloc_outbound_item(struct qcom_smp2p *smp2p) > return PTR_ERR(out); > } > > - memset(out, 0, sizeof(*out)); > + memset_io(out, 0, sizeof(*out)); > out->magic = SMP2P_MAGIC; > out->local_pid = smp2p->local_pid; > out->remote_pid = smp2p->remote_pid; Good catch. I'll add this fix for the pull.
[toc] | [prev] | [next] | [standalone]
| From | Paul Walmsley <paul@pwsan.com> |
|---|---|
| Date | 2016-03-01 18:20 +0100 |
| Subject | Re: [PATCH] soc: qcom: smp2p: Use memset_io to zero out new entries |
| Message-ID | <r7W7M-3DI-33@gated-at.bofh.it> |
| In reply to | #1343518 |
On Thu, 25 Feb 2016, Andy Gross wrote: > On Thu, Feb 25, 2016 at 08:17:37AM -0800, Bjorn Andersson wrote: > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > > --- > > drivers/soc/qcom/smp2p.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/soc/qcom/smp2p.c b/drivers/soc/qcom/smp2p.c > > index f1eed7f9dd67..61df598444e6 100644 > > --- a/drivers/soc/qcom/smp2p.c > > +++ b/drivers/soc/qcom/smp2p.c > > @@ -379,7 +379,7 @@ static int qcom_smp2p_alloc_outbound_item(struct qcom_smp2p *smp2p) > > return PTR_ERR(out); > > } > > > > - memset(out, 0, sizeof(*out)); > > + memset_io(out, 0, sizeof(*out)); > > out->magic = SMP2P_MAGIC; > > out->local_pid = smp2p->local_pid; > > out->remote_pid = smp2p->remote_pid; > > Good catch. I'll add this fix for the pull. Folks, is qmem_smem_get() intended to return pointers to device-mapped memory, as implied by this change? If so, shouldn't the return type of qmem_smem_get() et al. be void __iomem * rather than void *? - Paul
[toc] | [prev] | [next] | [standalone]
| From | Bjorn Andersson <bjorn.andersson@linaro.org> |
|---|---|
| Date | 2016-03-01 18:30 +0100 |
| Message-ID | <r7Whs-3Ha-27@gated-at.bofh.it> |
| In reply to | #1346785 |
On Tue 01 Mar 09:10 PST 2016, Paul Walmsley wrote: > On Thu, 25 Feb 2016, Andy Gross wrote: > > > On Thu, Feb 25, 2016 at 08:17:37AM -0800, Bjorn Andersson wrote: > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > > > --- > > > drivers/soc/qcom/smp2p.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/soc/qcom/smp2p.c b/drivers/soc/qcom/smp2p.c > > > index f1eed7f9dd67..61df598444e6 100644 > > > --- a/drivers/soc/qcom/smp2p.c > > > +++ b/drivers/soc/qcom/smp2p.c > > > @@ -379,7 +379,7 @@ static int qcom_smp2p_alloc_outbound_item(struct qcom_smp2p *smp2p) > > > return PTR_ERR(out); > > > } > > > > > > - memset(out, 0, sizeof(*out)); > > > + memset_io(out, 0, sizeof(*out)); > > > out->magic = SMP2P_MAGIC; > > > out->local_pid = smp2p->local_pid; > > > out->remote_pid = smp2p->remote_pid; > > > > Good catch. I'll add this fix for the pull. > > Folks, is qmem_smem_get() intended to return pointers to device-mapped > memory, as implied by this change? > > If so, shouldn't the return type of qmem_smem_get() et al. be void > __iomem * rather than void *? > The shared memory is remapped system ram, so it should not be device mapped. I've discussed this with Andy in the past and we'll try to get the discussions settled during next week's Linaro Connect. Regards, Bjorn
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web