| Message ID | AS8PR02MB72372490C53FB2E35DA1ADD08BFE2@AS8PR02MB7237.eurprd02.prod.outlook.com (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | 0858582e7e1df72eb44edf779008ba4dee7f0d75 |
| Headers | show |
| Series | nvdimm/btt: use sizeof(*pointer) instead of sizeof(type) | expand |
On 6/2/24 1:18 AM, Erick Archer wrote: > It is preferred to use sizeof(*pointer) instead of sizeof(type) > due to the type of the variable can change and one needs not > change the former (unlike the latter). This patch has no effect > on runtime behavior. > > Signed-off-by: Erick Archer <erick.archer@outlook.com> Reviewed-by: Dave Jiang <dave.jiang@intel.com> > --- > drivers/nvdimm/btt.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/nvdimm/btt.c b/drivers/nvdimm/btt.c > index 1e5aedaf8c7b..b25df8fa8e8e 100644 > --- a/drivers/nvdimm/btt.c > +++ b/drivers/nvdimm/btt.c > @@ -751,7 +751,7 @@ static struct arena_info *alloc_arena(struct btt *btt, size_t size, > u64 logsize, mapsize, datasize; > u64 available = size; > > - arena = kzalloc(sizeof(struct arena_info), GFP_KERNEL); > + arena = kzalloc(sizeof(*arena), GFP_KERNEL); > if (!arena) > return NULL; > arena->nd_btt = btt->nd_btt; > @@ -978,7 +978,7 @@ static int btt_arena_write_layout(struct arena_info *arena) > if (ret) > return ret; > > - super = kzalloc(sizeof(struct btt_sb), GFP_NOIO); > + super = kzalloc(sizeof(*super), GFP_NOIO); > if (!super) > return -ENOMEM; >
diff --git a/drivers/nvdimm/btt.c b/drivers/nvdimm/btt.c index 1e5aedaf8c7b..b25df8fa8e8e 100644 --- a/drivers/nvdimm/btt.c +++ b/drivers/nvdimm/btt.c @@ -751,7 +751,7 @@ static struct arena_info *alloc_arena(struct btt *btt, size_t size, u64 logsize, mapsize, datasize; u64 available = size; - arena = kzalloc(sizeof(struct arena_info), GFP_KERNEL); + arena = kzalloc(sizeof(*arena), GFP_KERNEL); if (!arena) return NULL; arena->nd_btt = btt->nd_btt; @@ -978,7 +978,7 @@ static int btt_arena_write_layout(struct arena_info *arena) if (ret) return ret; - super = kzalloc(sizeof(struct btt_sb), GFP_NOIO); + super = kzalloc(sizeof(*super), GFP_NOIO); if (!super) return -ENOMEM;
It is preferred to use sizeof(*pointer) instead of sizeof(type) due to the type of the variable can change and one needs not change the former (unlike the latter). This patch has no effect on runtime behavior. Signed-off-by: Erick Archer <erick.archer@outlook.com> --- drivers/nvdimm/btt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)