On (03/10/08 18:37), Andy Whitcroft didst pronounce: > Rename and export the pool counter update interface. >
Something is being "exported" that is not in the lds file. Export via the other header and reshuffle so that all the cleanups are together in the patchset? > Signed-off-by: Andy Whitcroft <[EMAIL PROTECTED]> > --- > hugetlbfs.h | 2 ++ > hugeutils.c | 8 ++++---- > 2 files changed, 6 insertions(+), 4 deletions(-) > > diff --git a/hugetlbfs.h b/hugetlbfs.h > index 54fa13b..985e138 100644 > --- a/hugetlbfs.h > +++ b/hugetlbfs.h > @@ -71,6 +71,8 @@ enum { /* The number of pages of a > given size that ... */ > HUGEPAGES_MAX_COUNTERS, > }; > long get_huge_page_counter(long pagesize, unsigned int counter); > +int set_huge_page_counter(long pagesize, unsigned int counter, > + unsigned long val); > int set_nr_hugepages(long pagesize, unsigned long val); > int set_nr_overcommit_hugepages(long pagesize, unsigned long val); > long read_meminfo(const char *tag); > diff --git a/hugeutils.c b/hugeutils.c > index 3eb9af6..da3afd5 100644 > --- a/hugeutils.c > +++ b/hugeutils.c > @@ -189,7 +189,7 @@ static long file_read_ulong(char *file, const char *tag) > p = buf; > > val = strtol(p, &q, 0); > - if (! isspace(*q)) { > + if (*q && ! isspace(*q)) { > ERROR("Couldn't parse %s value\n", file); > return -1; > } > @@ -255,7 +255,7 @@ static int select_pool_counter(unsigned int counter, > unsigned long pagesize, > return 0; > } > > -static int set_pool_counter(long pagesize, unsigned int counter, > +int set_huge_page_counter(long pagesize, unsigned int counter, > unsigned long val) > { > char file[PATH_MAX+1]; > @@ -776,12 +776,12 @@ long get_huge_page_counter(long pagesize, unsigned int > counter) > > int set_nr_hugepages(long pagesize, unsigned long val) > { > - return set_pool_counter(pagesize, HUGEPAGES_TOTAL, val); > + return set_huge_page_counter(pagesize, HUGEPAGES_TOTAL, val); > } > > int set_nr_overcommit_hugepages(long pagesize, unsigned long val) > { > - return set_pool_counter(pagesize, HUGEPAGES_OC, val); > + return set_huge_page_counter(pagesize, HUGEPAGES_OC, val); > } > > /********************************************************************/ > -- > 1.6.0.1.451.gc8d31 > -- Mel Gorman Part-time Phd Student Linux Technology Center University of Limerick IBM Dublin Software Lab ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Libhugetlbfs-devel mailing list Libhugetlbfs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/libhugetlbfs-devel