[PATCH] mm: Declare hugetlb_sysfs_add_hstate __meminit

[PATCH] mm: Declare hugetlb_sysfs_add_hstate __meminit

[Thomas Renninger]

Initially found by Mel, I just put this into a patch.

Signed-off-by: Thomas Renninger <trenn@suse.de>
Reviewed-by: Mel Gorman <mgorman@novell.com>
CC: majordomo@kvack.org
---
 mm/hugetlb.c |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/mm/hugetlb.c b/mm/hugetlb.c
index bfcf153..2c59a0a 100644
--- a/mm/hugetlb.c
+++ b/mm/hugetlb.c
@@ -1543,9 +1543,10 @@ static struct attribute_group hstate_attr_group = {
 	.attrs = hstate_attrs,
 };
 
-static int hugetlb_sysfs_add_hstate(struct hstate *h, struct kobject *parent,
-				    struct kobject **hstate_kobjs,
-				    struct attribute_group *hstate_attr_group)
+static int
+__meminit hugetlb_sysfs_add_hstate(struct hstate *h, struct kobject *parent,
+				   struct kobject **hstate_kobjs,
+				   struct attribute_group *hstate_attr_group)
 {
 	int retval;
 	int hi = h - hstates;
-- 
1.7.3.4

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] mm: Declare hugetlb_sysfs_add_hstate __meminit

[David Rientjes]

On Tue, 26 Jul 2011, Thomas Renninger wrote:

> Initially found by Mel, I just put this into a patch.
> 
> Signed-off-by: Thomas Renninger <trenn@suse.de>
> Reviewed-by: Mel Gorman <mgorman@novell.com>
> CC: majordomo@kvack.org

Not sure where majordomo@kvack.org comes into this :)

>  mm/hugetlb.c |    7 ++++---
>  1 files changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/mm/hugetlb.c b/mm/hugetlb.c
> index bfcf153..2c59a0a 100644
> --- a/mm/hugetlb.c
> +++ b/mm/hugetlb.c
> @@ -1543,9 +1543,10 @@ static struct attribute_group hstate_attr_group = {
>  	.attrs = hstate_attrs,
>  };
>  
> -static int hugetlb_sysfs_add_hstate(struct hstate *h, struct kobject *parent,
> -				    struct kobject **hstate_kobjs,
> -				    struct attribute_group *hstate_attr_group)
> +static int
> +__meminit hugetlb_sysfs_add_hstate(struct hstate *h, struct kobject *parent,
> +				   struct kobject **hstate_kobjs,
> +				   struct attribute_group *hstate_attr_group)
>  {
>  	int retval;
>  	int hi = h - hstates;

I'm looking at this based on the latest git; if this was intended only as 
a fix for -mm, please add that tag to the subject line.

That would be right if hugetlb_register_node() didn't use it or it was 
moved to meminit.text as well, and that would require that register_node() 
was in the same section.

It's a bit tricky to see since hugetlb_register_node() in 
drivers/base/node.c is really calling into hugetlb_register_node() from 
mm/hugetlb.c.

 [ And the drivers/base/node.c function has bool type for CONFIG_HUGETLBFS
   and void for !CONFIG_HUGETLBFS.  It can probably be changed to just be
   void everywhere. ]

So, unless this is a fix for -mm, I don't think this is right.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] mm: Declare hugetlb_sysfs_add_hstate __meminit

[Thomas Renninger]

On Tuesday 26 July 2011 02:52:09 David Rientjes wrote:
> On Tue, 26 Jul 2011, Thomas Renninger wrote:
> 
...
> So, unless this is a fix for -mm, I don't think this is right.
Thanks for looking at it.

Please ignore this one.

   Thomas

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>