Re: [PATCH] Add better debugging capabilities to loader

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/30/2010 01:26 PM, Martin Sivak wrote:
- when development ('devel' argument) mode is requested, init enables
   core dumping and starts shell in the case of unexpected exit
---
  loader/init.c   |   54 +++++++++++++++++++++++++++++++++++++++++++++++++-----
  loader/loader.c |    7 -------
  2 files changed, 49 insertions(+), 12 deletions(-)


Hi,

I applaud to this initiative. No ack yet though:

This:
+    if (isDebugMode) {
+        /* if anaconda dies suddenly we are doomed, so at least make a coredump */
+        struct rlimit corelimit = { RLIM_INFINITY,  RLIM_INFINITY};
+        int ret = setrlimit(RLIMIT_CORE,&corelimit);
+        if (ret) {
+            perror("setrlimit failed - no coredumps will be available");

Together with this:
-    /* if anaconda dies suddenly we are doomed, so at least make a coredump */
-    struct rlimit corelimit = { RLIM_INFINITY,  RLIM_INFINITY};
-    ret = setrlimit(RLIMIT_CORE,&corelimit);
-    if (ret) {
-        perror("setrlimit failed");
-    }
-

Means core won't be dumped if the anaconda process is killed suddenly for an obscure reason, which was exactly why I introduced this. In other words: the next time we see strange irregular crashes we want to get the core file straight away and not ask the QAs to try again with a special cmdline parameter.

Also, since you are setting the core limit in init.c now, please also test that sending ABRT to anaconda still results in the core dump.

Ales

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list


[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux