Re: make OOM more "user friendly"

2005-04-02 Thread Indrek Kruusa
Matthias-Christian Ott wrote:
Diego Calleja schrieb:
When people gets OOM messages, many of them don't know what is 
happening or what
OOM means. This brief message explains it.

--- stable/mm/oom_kill.c.orig2005-04-02 17:44:14.0 +0200
+++ stable/mm/oom_kill.c2005-04-02 18:01:02.0 +0200
@@ -189,7 +189,8 @@
return;
}
task_unlock(p);
-printk(KERN_ERR "Out of Memory: Killed process %d (%s).\n", 
p->pid, p->comm);
+printk(KERN_ERR "The system has run Out Of Memory (RAM + swap), 
a process will be killed to free some memory\n");
+printk(KERN_ERR "OOM: Killed process %d (%s).\n", p->pid, p->comm);

/*
 * We give our sacrificial lamb high priority and access to
-
To unsubscribe from this list: send the line "unsubscribe 
linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 

I disagree this is _not_ usefull. If the user don't knows what OOM 
means he can use google to get this information.

:)  Somewhat like "Use your mobile phone to call helpdesk if your mobile 
phone is broken". Maybe such messages should have some kind of link to 
information  in Documentation/  ?

regards,
Indrek
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: make OOM more "user friendly"

2005-04-02 Thread Diego Calleja
El Sat, 02 Apr 2005 18:58:53 +0200,
Matthias-Christian Ott <[EMAIL PROTECTED]> escribió:

> I disagree this is _not_ usefull. If the user don't knows what OOM means 
> he can use google to get this information.

And google will take them to what random source of information? There's no 
"official"
meaning of what OOM is outside the kernel And anyway, why shouldn't the 
kernel tell
what's happening? That printk is not exactly a fifty-page explanation, it just 
says "your
system has run out of memory" instead of "OOM", which is what it's really 
happening and
it's not verbose at all, and it doesn't scare users.

OOM doesn't prints just those messages, if prints a lot of "debugging info" 
about the state
of the memory subsystem, I've found people in usenet who reboots their systems 
when
they see that because they think it's a critical failure or something - and 
looking at how it's
printed, I don't blame them. This is the reason why I submitted this patch.

(and I'd have added a "look at Documentation/oom.txt", but there's zero 
documentation of
what OOM is, what are the causes of it, tips of how to find apps triggering it 
and tips to fix
it, and I'm not the right person to write it, so...)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: make OOM more "user friendly"

2005-04-02 Thread Matthias-Christian Ott
Diego Calleja schrieb:
When people gets OOM messages, many of them don't know what is happening or what
OOM means. This brief message explains it.
--- stable/mm/oom_kill.c.orig   2005-04-02 17:44:14.0 +0200
+++ stable/mm/oom_kill.c2005-04-02 18:01:02.0 +0200
@@ -189,7 +189,8 @@
return;
}
task_unlock(p);
-   printk(KERN_ERR "Out of Memory: Killed process %d (%s).\n", p->pid, 
p->comm);
+   printk(KERN_ERR "The system has run Out Of Memory (RAM + swap), a process 
will be killed to free some memory\n");
+   printk(KERN_ERR "OOM: Killed process %d (%s).\n", p->pid, p->comm);
/*
 * We give our sacrificial lamb high priority and access to
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
 

I disagree this is _not_ usefull. If the user don't knows what OOM means 
he can use google to get this information.

Matthias-Christian Ott
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: make OOM more user friendly

2005-04-02 Thread Matthias-Christian Ott
Diego Calleja schrieb:
When people gets OOM messages, many of them don't know what is happening or what
OOM means. This brief message explains it.
--- stable/mm/oom_kill.c.orig   2005-04-02 17:44:14.0 +0200
+++ stable/mm/oom_kill.c2005-04-02 18:01:02.0 +0200
@@ -189,7 +189,8 @@
return;
}
task_unlock(p);
-   printk(KERN_ERR Out of Memory: Killed process %d (%s).\n, p-pid, 
p-comm);
+   printk(KERN_ERR The system has run Out Of Memory (RAM + swap), a process 
will be killed to free some memory\n);
+   printk(KERN_ERR OOM: Killed process %d (%s).\n, p-pid, p-comm);
/*
 * We give our sacrificial lamb high priority and access to
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
 

I disagree this is _not_ usefull. If the user don't knows what OOM means 
he can use google to get this information.

Matthias-Christian Ott
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: make OOM more user friendly

2005-04-02 Thread Diego Calleja
El Sat, 02 Apr 2005 18:58:53 +0200,
Matthias-Christian Ott [EMAIL PROTECTED] escribió:

 I disagree this is _not_ usefull. If the user don't knows what OOM means 
 he can use google to get this information.

And google will take them to what random source of information? There's no 
official
meaning of what OOM is outside the kernel And anyway, why shouldn't the 
kernel tell
what's happening? That printk is not exactly a fifty-page explanation, it just 
says your
system has run out of memory instead of OOM, which is what it's really 
happening and
it's not verbose at all, and it doesn't scare users.

OOM doesn't prints just those messages, if prints a lot of debugging info 
about the state
of the memory subsystem, I've found people in usenet who reboots their systems 
when
they see that because they think it's a critical failure or something - and 
looking at how it's
printed, I don't blame them. This is the reason why I submitted this patch.

(and I'd have added a look at Documentation/oom.txt, but there's zero 
documentation of
what OOM is, what are the causes of it, tips of how to find apps triggering it 
and tips to fix
it, and I'm not the right person to write it, so...)
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/