On Fri, Nov 14, 2008 at 09:01:09PM +0000, Adam Litke wrote:
> Signed-off-by: Adam Litke <[EMAIL PROTECTED]>
> ---
>
> hugeutils.c | 12 ++++++------
> kernel-features.c | 16 ++++++++--------
> 2 files changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/hugeutils.c b/hugeutils.c
> index 1121966..999da74 100644
> --- a/hugeutils.c
> +++ b/hugeutils.c
> @@ -293,7 +293,7 @@ static void probe_default_hpage_size(void)
> int index;
>
> if (nr_hpage_sizes == 0) {
> - DEBUG("No configured huge page sizes\n");
> + INFO("No configured huge page sizes\n");
> hpage_sizes_default_idx = -1;
> return;
Is this sort of message not a WARNING elsewhere?
> }
> @@ -314,12 +314,12 @@ static void probe_default_hpage_size(void)
> if (index >= 0)
> hpage_sizes_default_idx = index;
> else {
> - DEBUG("No mount point found for default huge page "
> + INFO("No mount point found for default huge page "
> "size. Using first available mount point.\n");
Same.
> hpage_sizes_default_idx = 0;
> }
> } else {
> - DEBUG("Unable to determine default huge page size\n");
> + ERROR("Unable to determine default huge page size\n");
> hpage_sizes_default_idx = -1;
Ok.
> }
> }
> @@ -339,7 +339,7 @@ static void add_hugetlbfs_mount(char *path, int
> user_mount)
>
> size = hugetlbfs_test_pagesize(path);
> if (size < 0) {
> - DEBUG("Unable to detect page size for path %s\n", path);
> + WARNING("Unable to detect page size for path %s\n", path);
> return;
> }
>
> @@ -369,9 +369,9 @@ static void debug_show_page_sizes(void)
> {
> int i;
>
> - DEBUG("Detected page sizes:\n");
> + INFO("Detected page sizes:\n");
> for (i = 0; i < nr_hpage_sizes; i++)
> - DEBUG(" Size: %li kB %s Mount: %s\n",
> + INFO(" Size: %li kB %s Mount: %s\n",
> hpage_sizes[i].pagesize / 1024,
> i == hpage_sizes_default_idx ? "(default)" : "",
> hpage_sizes[i].mount);
> diff --git a/kernel-features.c b/kernel-features.c
> index 5f20249..79c499c 100644
> --- a/kernel-features.c
> +++ b/kernel-features.c
> @@ -47,14 +47,14 @@ static void debug_kernel_version(void)
> {
> struct kernel_version *ver = &running_kernel_version;
>
> - DEBUG("Parsed kernel version: [%u] . [%u] . [%u] ",
> + INFO("Parsed kernel version: [%u] . [%u] . [%u] ",
> ver->major, ver->minor, ver->release);
> if (ver->post)
> - DEBUG_CONT(" [post-release: %u]\n", ver->post);
> + INFO_CONT(" [post-release: %u]\n", ver->post);
> else if (ver->pre)
> - DEBUG_CONT(" [pre-release: %u]\n", ver->pre);
> + INFO_CONT(" [pre-release: %u]\n", ver->pre);
> else
> - DEBUG_CONT("\n");
> + INFO_CONT("\n");
> }
>
> static int str_to_ver(const char *str, struct kernel_version *ver)
> @@ -241,12 +241,12 @@ void setup_features()
>
> /* Has the user overridden feature detection? */
> if (env && (pos = strstr(env, name))) {
> - DEBUG("Overriding feature %s: ", name);
> + INFO("Overriding feature %s: ", name);
> /* If feature is preceeded by 'no_' then turn it off */
> if (((pos - 3) >= env) && !strncmp(pos - 3, "no_", 3))
> - DEBUG_CONT("no\n");
> + INFO_CONT("no\n");
> else {
> - DEBUG_CONT("yes\n");
> + INFO_CONT("yes\n");
> feature_mask |= (1UL << i);
> }
> continue;
> @@ -254,7 +254,7 @@ void setup_features()
>
> /* Is the running kernel version newer? */
> if (ver_cmp(&running_kernel_version, &ver) >= 0) {
> - DEBUG("Feature %s is present in this kernel\n",
> + INFO("Feature %s is present in this kernel\n",
> kernel_features[i].name);
> feature_mask |= (1UL << i);
> }
>
>
--
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
