All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/8] Transform kzalloc calls to kcalloc
@ 2015-03-10 11:52 Navya Sri Nizamkari
  2015-03-10 11:57 ` [PATCH v2 1/8] staging: rtl8188eu: Use kcalloc instead of kzalloc Navya Sri Nizamkari
                   ` (7 more replies)
  0 siblings, 8 replies; 9+ messages in thread
From: Navya Sri Nizamkari @ 2015-03-10 11:52 UTC (permalink / raw)
  To: outreachy-kernel

[changes in v2]
In version v2, I break the original patchset into two.
One for kmalloc -> k{z,c}alloc and other for 
kzalloc -> kcalloc. This patchset includes patches for
kzalloc to kmalloc conversion.

[original cover letter]
This patchset changes some kzalloc calls to kcalloc and
kmalloc to kzalloc.

 The semantic patch used to make this change is:

 @@
 type T, T2;
 expression x;
 identifier f,f1;
 expression E1,E2,E3,E4;
 statement S,S1,S2;
 @@
 - x = (T)kmalloc(E1,E2)
 + x = kzalloc(E1,E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 if ((x!=NULL)&&...) {
 ... when != \(f1(...,x,...)\|<+...x...+>=E4\)
 when != \(while(...) S1\|for(...;...;...) S1\)
 - memset((T2)x,0,E1);
 ...
 } else S2
 |
 - memset((T2)x,0,E1);
 )
 @@
 type T, T2;
 type T1;
 T1 *x;
 T1 *y;
 identifier f,f1;
 expression E2,E3,E4;
 statement S,S1,S2;
 @@
 - x = (T)kmalloc(sizeof(T1),E2)
 + x = kzalloc(sizeof(T1),E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 if ((x!=NULL)&&...) {
 ... when != \(f1(...,x,...)\|<+...x...+>=E4\)
 when != \(while(...) S1\|for(...;...;...) S1\)
 - memset((T2)x,0,sizeof(*y));
 ...
 } else S2
 |
 - memset((T2)x,0,sizeof(*y));
 )
 @@
 type T, T2;
 type T1;
 T1 *x;
 T1 *y;
 identifier f,f1;
 expression E,E2,E3,E4;
 statement S,S1,S2;
 @@
 - x = (T)kmalloc(sizeof(T1)*E,E2)
 + x = kzalloc(sizeof(T1)*E,E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 if ((x!=NULL)&&...) {
 ... when != \(f1(...,x,...)\|<+...x...+>=E4\)
 when != \(while(...) S1\|for(...;...;...) S1\)
 - memset((T2)x,0,sizeof(*y)*E);
 ...
 } else S2
 |
 - memset((T2)x,0,sizeof(*y)*E);
 )
 @@
 type T, T2;
 type T1;
 T1 *x;
 T1 *y;
 identifier f,f1;
 expression E2,E3,E4;
 statement S,S1,S2;
 @@
 - x = (T)kmalloc(sizeof(*y),E2)
 + x = kzalloc(sizeof(*y),E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 if ((x!=NULL)&&...) {
 ... when != \(f1(...,x,...)\|<+...x...+>=E4\)
 when != \(while(...) S1\|for(...;...;...) S1\)
 - memset((T2)x,0,sizeof(T1));
 ...
 } else S2
 |
 - memset((T2)x,0,sizeof(T1));
 )
 @@
 type T, T2;
 type T1;
 T1 *x;
 T1 *y;
 identifier f,f1;
 expression E,E2,E3,E4;
 statement S,S1,S2;
 @@
 - x = (T)kmalloc(sizeof(*y)*E,E2)
 + x = kzalloc(sizeof(*y)*E,E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 if ((x!=NULL)&&...) {
 ... when != \(f1(...,x,...)\|<+...x...+>=E4\)
 when != \(while(...) S1\|for(...;...;...) S1\)
 - memset((T2)x,0,sizeof(T1)*E);
 ...
 } else S2
 |
 - memset((T2)x,0,sizeof(T1)*E);
 )
 @ disable neg_if, mult_comm @
 type T, T2;
 expression x;
 identifier f,f1;
 expression E0,E1,E2,E3,E4;
 statement S,S1,S2;
 @@
 - x = (T)kmalloc(E0 * E1,E2)
 + x = kzalloc(E0 * E1,E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 if ((x!=NULL)&&...) {
 ... when != \(f1(...,x,...)\|<+...x...+>=E4\)
 when != \(while(...) S1\|for(...;...;...) S1\)
 - memset((T2)x,0,E1 * E0);
 ...
 } else S2
 |
 - memset((T2)x,0,E1 * E0);
 )
 // ---------------------------------------------------------------------
 // ---------------------------------------------------------------------
 // have to duplicate everything again to make a version with no braces
 //\(x->fld\|f(...,x,...)\|x=E\)
 @@
 type T, T2;
 expression x;
 identifier f;
 expression E1,E2,E3;
 statement S,S2;
 @@
 - x = (T)kmalloc(E1,E2)
 + x = kzalloc(E1,E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 - if ((x!=NULL)&&...) memset((T2)x,0,E1);
 |
 + if (!x)
 - if ((x!=NULL)&&...) memset((T2)x,0,E1); else
 S2
 )
 @ @
 type T, T2;
 type T1;
 T1 *x;
 T1 *y;
 identifier f;
 expression E2,E3;
 statement S,S2;
 @@
 - x = (T)kmalloc(sizeof(T1),E2)
 + x = kzalloc(sizeof(T1),E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 - if ((x!=NULL)&&...) memset((T2)x,0,sizeof(*y));
 |
 + if (!x)
 - if ((x!=NULL)&&...) memset((T2)x,0,sizeof(*y)); else
 S2
 )
 @@
 type T, T2;
 type T1;
 T1 *x;
 T1 *y;
 identifier f;
 expression E,E2,E3;
 statement S,S2;
 @@
 - x = (T)kmalloc(sizeof(T1)*E,E2)
 + x = kzalloc(sizeof(T1)*E,E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 - if ((x!=NULL)&&...) memset((T2)x,0,sizeof(*y)*E);
 |
 + if (!x)
 - if ((x!=NULL)&&...) memset((T2)x,0,sizeof(*y)*E); else
 S2
 )
 @@
 type T, T2;
 type T1;
 T1 *x;
 T1 *y;
 identifier f;
 expression E2,E3;
 statement S,S2;
 @@
 - x = (T)kmalloc(sizeof(*y),E2)
 + x = kzalloc(sizeof(*y),E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 - if ((x!=NULL)&&...) memset((T2)x,0,sizeof(T1));
 |
 + if (!x)
 - if ((x!=NULL)&&...) memset((T2)x,0,sizeof(T1)); else
 S2
 )
 @@
 type T, T2;
 type T1;
 T1 *x;
 T1 *y;
 identifier f;
 expression E,E2,E3;
 statement S,S2;
 @@
 - x = (T)kmalloc(sizeof(*y)*E,E2)
 + x = kzalloc(sizeof(*y)*E,E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 - if ((x!=NULL)&&...) memset((T2)x,0,sizeof(T1)*E);
 |
 + if (!x)
 - if ((x!=NULL)&&...) memset((T2)x,0,sizeof(T1)*E); else
 S2
 )
 @ disable neg_if, mult_comm @
 type T, T2;
 expression x;
 identifier f;
 expression E0,E1,E2,E3;
 statement S,S2;
 @@
 - x = (T)kmalloc(E0 * E1,E2)
 + x = kzalloc(E0 * E1,E2)
 ... when != \(f(...,x,...)\|<+...x...+>=E3\)
 when != \(while(...) S\|for(...;...;...) S\)
 (
 - if ((x!=NULL)&&...) memset((T2)x,0,E1 * E0);
 |
 + if (!x)
 - if ((x!=NULL)&&...) memset((T2)x,0,E1 * E0); else
 S2
 )
 // ---------------------------------------------------------------------
 @@
 expression E;
 statement S;
 @@
 kzalloc(...)
 ...
 if (E) S
 - else { }
 // ---------------------------------------------------------------------
 // ---------------------------------------------------------------------
 @@
 expression E1,E2,E3;
 @@
 - kzalloc(E1*sizeof(E2),E3)
 + kcalloc(E1,sizeof(E2),E3)
 @@
 expression E1,E3;
 type T;
 @@
 - kzalloc(E1*sizeof(T),E3)
 + kcalloc(E1,sizeof(T),E3)
 @@
 expression E1,E2,E3,E4;
 @@
 - kzalloc(E1*E2*sizeof(E3),E4)
 + kcalloc(E1*E2,sizeof(E3),E4)
 @@
 expression E1,E2,E3;
 type T;
 @@
 - kzalloc(E1*E2*sizeof(T),E3)
 + kcalloc(E1*E2,sizeof(T),E3)
 @@
 expression E1,E2,E3,E4;
 @@
 - kzalloc(sizeof(E3)*E1*E2,E4)
 + kcalloc(E1*E2,sizeof(E3),E4)
 @@
 expression E1,E2,E3;
 type T;
 @@
 - kzalloc(sizeof(T)*E1*E2,E3)
 + kcalloc(E1*E2,sizeof(T),E3)
 @@
 constant E1;
 expression E2,E3;
 @@
 - kzalloc(E1*E2,E3)
 + kcalloc(E1,E2,E3)

Navya Sri Nizamkari (8):
  staging: rtl8188eu: Use kcalloc instead of kzalloc.
  staging: rtl8188eu: Use kcalloc instead of kzalloc
  staging: dgnc: Use kcalloc instead of kzalloc.
  staging: iio: Use kcalloc instead of kzalloc.
  staging: iio: Use kcalloc instead of kzalloc
  staging: media: Use kcalloc instead of kzalloc.
  staging: media: Use kcalloc instead of kzalloc
  staging: unisys: Use kcalloc instead of kzalloc.

 drivers/staging/dgnc/dgnc_driver.c                   | 2 +-
 drivers/staging/iio/accel/lis3l02dq_ring.c           | 2 +-
 drivers/staging/iio/adc/ad7280a.c                    | 5 +++--
 drivers/staging/media/bcm2048/radio-bcm2048.c        | 2 +-
 drivers/staging/media/davinci_vpfe/vpfe_mc_capture.c | 7 ++++---
 drivers/staging/rtl8188eu/core/rtw_xmit.c            | 3 ++-
 drivers/staging/rtl8188eu/hal/rtl8188eu_recv.c       | 2 +-
 drivers/staging/unisys/visorutil/procobjecttree.c    | 6 +++---
 8 files changed, 16 insertions(+), 13 deletions(-)

-- 
1.9.1



^ permalink raw reply	[flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-03-10 12:07 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-10 11:52 [PATCH v2 0/8] Transform kzalloc calls to kcalloc Navya Sri Nizamkari
2015-03-10 11:57 ` [PATCH v2 1/8] staging: rtl8188eu: Use kcalloc instead of kzalloc Navya Sri Nizamkari
2015-03-10 11:59 ` [PATCH v2 2/8] " Navya Sri Nizamkari
2015-03-10 12:01 ` [PATCH v2 3/8] staging: dgnc: " Navya Sri Nizamkari
2015-03-10 12:02 ` [PATCH v2 4/8] staging: iio: " Navya Sri Nizamkari
2015-03-10 12:04 ` [PATCH v2 5/8] " Navya Sri Nizamkari
2015-03-10 12:05 ` [PATCH v2 6/8] staging: media: " Navya Sri Nizamkari
2015-03-10 12:07 ` [PATCH v2 7/8] " Navya Sri Nizamkari
2015-03-10 12:08 ` [PATCH v2 8/8] staging: unisys: " Navya Sri Nizamkari

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.