Re: [PATCH 2/3] zswap: Improve a size determination in zswap_frontswap_init()
On Sun, May 21, 2017 at 4:26 AM, SF Markus Elfringwrote: > From: Markus Elfring > Date: Sat, 20 May 2017 22:44:03 +0200 > > Replace the specification of a data structure by a pointer dereference > as the parameter for the operator "sizeof" to make the corresponding size > determination a bit safer according to the Linux coding style convention. > > Signed-off-by: Markus Elfring Acked-by: Dan Streetman > --- > mm/zswap.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/zswap.c b/mm/zswap.c > index 18d8e87119a6..a6e67633be03 100644 > --- a/mm/zswap.c > +++ b/mm/zswap.c > @@ -1156,5 +1156,5 @@ static void zswap_frontswap_init(unsigned type) > { > struct zswap_tree *tree; > > - tree = kzalloc(sizeof(struct zswap_tree), GFP_KERNEL); > + tree = kzalloc(sizeof(*tree), GFP_KERNEL); > if (!tree) { > -- > 2.13.0 >
Re: [PATCH 2/3] zswap: Improve a size determination in zswap_frontswap_init()
On Sun, May 21, 2017 at 4:26 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sat, 20 May 2017 22:44:03 +0200 > > Replace the specification of a data structure by a pointer dereference > as the parameter for the operator "sizeof" to make the corresponding size > determination a bit safer according to the Linux coding style convention. > > Signed-off-by: Markus Elfring Acked-by: Dan Streetman > --- > mm/zswap.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/zswap.c b/mm/zswap.c > index 18d8e87119a6..a6e67633be03 100644 > --- a/mm/zswap.c > +++ b/mm/zswap.c > @@ -1156,5 +1156,5 @@ static void zswap_frontswap_init(unsigned type) > { > struct zswap_tree *tree; > > - tree = kzalloc(sizeof(struct zswap_tree), GFP_KERNEL); > + tree = kzalloc(sizeof(*tree), GFP_KERNEL); > if (!tree) { > -- > 2.13.0 >
[PATCH 2/3] zswap: Improve a size determination in zswap_frontswap_init()
From: Markus ElfringDate: Sat, 20 May 2017 22:44:03 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus Elfring --- mm/zswap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/zswap.c b/mm/zswap.c index 18d8e87119a6..a6e67633be03 100644 --- a/mm/zswap.c +++ b/mm/zswap.c @@ -1156,5 +1156,5 @@ static void zswap_frontswap_init(unsigned type) { struct zswap_tree *tree; - tree = kzalloc(sizeof(struct zswap_tree), GFP_KERNEL); + tree = kzalloc(sizeof(*tree), GFP_KERNEL); if (!tree) { -- 2.13.0
[PATCH 2/3] zswap: Improve a size determination in zswap_frontswap_init()
From: Markus Elfring Date: Sat, 20 May 2017 22:44:03 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus Elfring --- mm/zswap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/zswap.c b/mm/zswap.c index 18d8e87119a6..a6e67633be03 100644 --- a/mm/zswap.c +++ b/mm/zswap.c @@ -1156,5 +1156,5 @@ static void zswap_frontswap_init(unsigned type) { struct zswap_tree *tree; - tree = kzalloc(sizeof(struct zswap_tree), GFP_KERNEL); + tree = kzalloc(sizeof(*tree), GFP_KERNEL); if (!tree) { -- 2.13.0