* [PATCH] dma-mapping: Fix sizeof() mismatch on tsk allocation
@ 2020-11-25 14:05 Colin King
2020-11-25 18:26 ` Song Bao Hua (Barry Song)
0 siblings, 1 reply; 5+ messages in thread
From: Colin King @ 2020-11-25 14:05 UTC (permalink / raw)
To: Barry Song, Christoph Hellwig, Marek Szyprowski, Robin Murphy, iommu
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
An incorrect sizeof() is being used, sizeof(tsk) is not correct, it should
be sizeof(*tsk). Fix it.
Addresses-Coverity: ("Sizeof not portable (SIZEOF_MISMATCH)")
Fixes: bfd2defed94d ("dma-mapping: add benchmark support for streaming DMA APIs")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
kernel/dma/map_benchmark.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/dma/map_benchmark.c b/kernel/dma/map_benchmark.c
index e1e37603d01b..b1496e744c68 100644
--- a/kernel/dma/map_benchmark.c
+++ b/kernel/dma/map_benchmark.c
@@ -121,7 +121,7 @@ static int do_map_benchmark(struct map_benchmark_data *map)
int ret = 0;
int i;
- tsk = kmalloc_array(threads, sizeof(tsk), GFP_KERNEL);
+ tsk = kmalloc_array(threads, sizeof(*tsk), GFP_KERNEL);
if (!tsk)
return -ENOMEM;
--
2.29.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [PATCH] dma-mapping: Fix sizeof() mismatch on tsk allocation
2020-11-25 14:05 [PATCH] dma-mapping: Fix sizeof() mismatch on tsk allocation Colin King
@ 2020-11-25 18:26 ` Song Bao Hua (Barry Song)
2020-11-25 18:29 ` Christoph Hellwig
0 siblings, 1 reply; 5+ messages in thread
From: Song Bao Hua (Barry Song) @ 2020-11-25 18:26 UTC (permalink / raw)
To: Colin King, Christoph Hellwig, Marek Szyprowski, Robin Murphy, iommu
Cc: kernel-janitors, linux-kernel
DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogQ29saW4gS2luZyBbbWFp
bHRvOmNvbGluLmtpbmdAY2Fub25pY2FsLmNvbV0NCj4gU2VudDogVGh1cnNkYXksIE5vdmVtYmVy
IDI2LCAyMDIwIDM6MDUgQU0NCj4gVG86IFNvbmcgQmFvIEh1YSAoQmFycnkgU29uZykgPHNvbmcu
YmFvLmh1YUBoaXNpbGljb24uY29tPjsgQ2hyaXN0b3BoDQo+IEhlbGx3aWcgPGhjaEBsc3QuZGU+
OyBNYXJlayBTenlwcm93c2tpIDxtLnN6eXByb3dza2lAc2Ftc3VuZy5jb20+Ow0KPiBSb2JpbiBN
dXJwaHkgPHJvYmluLm11cnBoeUBhcm0uY29tPjsgaW9tbXVAbGlzdHMubGludXgtZm91bmRhdGlv
bi5vcmcNCj4gQ2M6IGtlcm5lbC1qYW5pdG9yc0B2Z2VyLmtlcm5lbC5vcmc7IGxpbnV4LWtlcm5l
bEB2Z2VyLmtlcm5lbC5vcmcNCj4gU3ViamVjdDogW1BBVENIXSBkbWEtbWFwcGluZzogRml4IHNp
emVvZigpIG1pc21hdGNoIG9uIHRzayBhbGxvY2F0aW9uDQo+IA0KPiBGcm9tOiBDb2xpbiBJYW4g
S2luZyA8Y29saW4ua2luZ0BjYW5vbmljYWwuY29tPg0KPiANCj4gQW4gaW5jb3JyZWN0IHNpemVv
ZigpIGlzIGJlaW5nIHVzZWQsIHNpemVvZih0c2spIGlzIG5vdCBjb3JyZWN0LCBpdCBzaG91bGQN
Cj4gYmUgc2l6ZW9mKCp0c2spLiBGaXggaXQuDQo+IA0KPiBBZGRyZXNzZXMtQ292ZXJpdHk6ICgi
U2l6ZW9mIG5vdCBwb3J0YWJsZSAoU0laRU9GX01JU01BVENIKSIpDQo+IEZpeGVzOiBiZmQyZGVm
ZWQ5NGQgKCJkbWEtbWFwcGluZzogYWRkIGJlbmNobWFyayBzdXBwb3J0IGZvciBzdHJlYW1pbmcN
Cj4gRE1BIEFQSXMiKQ0KPiBTaWduZWQtb2ZmLWJ5OiBDb2xpbiBJYW4gS2luZyA8Y29saW4ua2lu
Z0BjYW5vbmljYWwuY29tPg0KPiAtLS0NCj4gIGtlcm5lbC9kbWEvbWFwX2JlbmNobWFyay5jIHwg
MiArLQ0KPiAgMSBmaWxlIGNoYW5nZWQsIDEgaW5zZXJ0aW9uKCspLCAxIGRlbGV0aW9uKC0pDQo+
IA0KPiBkaWZmIC0tZ2l0IGEva2VybmVsL2RtYS9tYXBfYmVuY2htYXJrLmMgYi9rZXJuZWwvZG1h
L21hcF9iZW5jaG1hcmsuYw0KPiBpbmRleCBlMWUzNzYwM2QwMWIuLmIxNDk2ZTc0NGM2OCAxMDA2
NDQNCj4gLS0tIGEva2VybmVsL2RtYS9tYXBfYmVuY2htYXJrLmMNCj4gKysrIGIva2VybmVsL2Rt
YS9tYXBfYmVuY2htYXJrLmMNCj4gQEAgLTEyMSw3ICsxMjEsNyBAQCBzdGF0aWMgaW50IGRvX21h
cF9iZW5jaG1hcmsoc3RydWN0DQo+IG1hcF9iZW5jaG1hcmtfZGF0YSAqbWFwKQ0KPiAgCWludCBy
ZXQgPSAwOw0KPiAgCWludCBpOw0KPiANCj4gLQl0c2sgPSBrbWFsbG9jX2FycmF5KHRocmVhZHMs
IHNpemVvZih0c2spLCBHRlBfS0VSTkVMKTsNCj4gKwl0c2sgPSBrbWFsbG9jX2FycmF5KHRocmVh
ZHMsIHNpemVvZigqdHNrKSwgR0ZQX0tFUk5FTCk7DQoNClRoZSBzaXplIGlzIHNhbWUuIEJ1dCB0
aGUgY2hhbmdlIGlzIGNvcnJlY3QuDQpBY2tlZC1ieTogQmFycnkgU29uZyA8c29uZy5iYW8uaHVh
QGhpc2lsaWNvbi5jb20+DQoNCj4gIAlpZiAoIXRzaykNCj4gIAkJcmV0dXJuIC1FTk9NRU07DQo+
IA0KVGhhbmtzDQpCYXJyeQ0KDQoNCg=
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dma-mapping: Fix sizeof() mismatch on tsk allocation
2020-11-25 18:26 ` Song Bao Hua (Barry Song)
@ 2020-11-25 18:29 ` Christoph Hellwig
2020-11-25 18:49 ` Colin Ian King
0 siblings, 1 reply; 5+ messages in thread
From: Christoph Hellwig @ 2020-11-25 18:29 UTC (permalink / raw)
To: Song Bao Hua (Barry Song)
Cc: kernel-janitors, linux-kernel, iommu, Colin King, Robin Murphy,
Christoph Hellwig
I'll fold this one in as well.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dma-mapping: Fix sizeof() mismatch on tsk allocation
2020-11-25 18:29 ` Christoph Hellwig
@ 2020-11-25 18:49 ` Colin Ian King
2020-11-26 17:06 ` Christoph Hellwig
0 siblings, 1 reply; 5+ messages in thread
From: Colin Ian King @ 2020-11-25 18:49 UTC (permalink / raw)
To: Christoph Hellwig, Song Bao Hua (Barry Song)
Cc: kernel-janitors, iommu, Robin Murphy, linux-kernel
On 25/11/2020 18:29, Christoph Hellwig wrote:
> I'll fold this one in as well.
>
OK, so two SoB's disappear?
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dma-mapping: Fix sizeof() mismatch on tsk allocation
2020-11-25 18:49 ` Colin Ian King
@ 2020-11-26 17:06 ` Christoph Hellwig
0 siblings, 0 replies; 5+ messages in thread
From: Christoph Hellwig @ 2020-11-26 17:06 UTC (permalink / raw)
To: Colin Ian King
Cc: kernel-janitors, linux-kernel, iommu, Robin Murphy, Christoph Hellwig
On Wed, Nov 25, 2020 at 06:49:19PM +0000, Colin Ian King wrote:
> On 25/11/2020 18:29, Christoph Hellwig wrote:
> > I'll fold this one in as well.
> >
> OK, so two SoB's disappear?
Yes, and are replaced by an annotation about folded fixes, just like
for all kinds of other commits.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-11-26 17:06 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-25 14:05 [PATCH] dma-mapping: Fix sizeof() mismatch on tsk allocation Colin King
2020-11-25 18:26 ` Song Bao Hua (Barry Song)
2020-11-25 18:29 ` Christoph Hellwig
2020-11-25 18:49 ` Colin Ian King
2020-11-26 17:06 ` Christoph Hellwig
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).