789693 - Implement CalculateProcessCreationTimestamp() on DragonFly/FreeBSD/NetBSD

Status: RESOLVED FIXED

(Toolkit :: Startup and Profile System, defect)

Description

[Jan Beich]

Attachment: sample program to test ifdef maze

Comment 1

[Jan Beich]

Attachment: Bug 789693 - Unify CalculateProcessCreationTimestamp() on OS X and BSDs.

The approach tries to address comment 2 in bug 633193 by sharing OS X code. It may be better than having 5 slightly different implementations racing for bitrot.

Tested as a sample program by me on FreeBSD and DragonFly, by :gaston on OpenBSD and NetBSD. Only Mac OS X wasn't tested.

Comment 2

[(dormant account)]

Comment on attachment 659486 [details] [diff] [review]
Bug 789693 - Unify CalculateProcessCreationTimestamp() on OS X and BSDs.

I would like to structure this define/ifdef mess differently. 
For example
+#if defined(__NetBSD__)
+  int mib[6] = { CTL_KERN, KERN_PROC2, KERN_PROC_PID, getpid(), sizeof(struct kinfo_proc2), 1 }
+#elif defined(__OpenBSD__)
+  int mib[6] = { CTL_KERN, KERN_PROC, KERN_PROC_PID, getpid(), sizeof(struct kinfo_proc), 1 };
+#else
   int mib[4] = { CTL_KERN, KERN_PROC, KERN_PROC_PID, getpid() };
+#endif


should just setup a dictinary of different field names and do 
#if OPENBSD
#define KINFO_PROC kinfo_proc
#elseif NETBSD
#define KINFO_PROC kinfo_proc2

 int mib[6] = { CTL_KERN, KERN_PROC, KERN_PROC_PID, getpid(), sizeof(KINFO_PROC), 1 };


-  struct kinfo_proc *proc = (kinfo_proc*) malloc(buffer_size);  
+#if defined(__NetBSD__)
+  struct kinfo_proc2 *proc = (struct kinfo_proc2 *) malloc(buffer_size);
+#else
+  struct kinfo_proc *proc = (struct kinfo_proc *) malloc(buffer_size);
+#endif
this mess would also be fixed by above

+
+#if defined(__NetBSD__) || defined(__OpenBSD__)
+  if (sysctl(mib, 6, proc, &buffer_size, NULL, 0)) {
+#else
   if (sysctl(mib, 4, proc, &buffer_size, NULL, 0)) {
+#endif
instead of using the 6 or 4 constant use
sizeof(mib)/sizeof(mib[0])

+#elif defined(__DragonFly__)
+  PRTime starttime = static_cast<PRTime>(proc->kp_start.tv_sec) * PR_USEC_PER_SEC;
+  starttime += proc->kp_start.tv_usec;
+#elif defined(__FreeBSD__)
+  PRTime starttime = static_cast<PRTime>(proc->ki_start.tv_sec) * PR_USEC_PER_SEC;
+  starttime += proc->ki_start.tv_usec;
+#else
   PRTime starttime = static_cast<PRTime>(proc->p_ustart_sec) * PR_USEC_PER_SEC;
   starttime += proc->p_ustart_usec;
+#endif

Similarly, above just define away the different structures like proc->kp_start.tv_sec instead of copying logic.

Comment 3

[Jan Beich]

Attachment: Bug 789693 - Unify CalculateProcessCreationTimestamp() on OS X and BSDs.

(In reply to Taras Glek (:taras) from comment #2)
> I would like to structure this define/ifdef mess differently.

v2. no malloc/free, less ifdefs + block of macros (bigger than the code itself).

 #undef KERN_PROC still looks ugly but the alternative is

     int mib[] = {
     ...
   #if defined(__NetBSD__)
       KERN_PROC2,
   #else
       KERN_PROC,
   #endif
     ...
     };

Comment 4

[Jan Beich]

Attachment: sample program to test ifdef maze, v2

Comment 5

[(dormant account)]

Comment on attachment 659569 [details] [diff] [review]
Bug 789693 - Unify CalculateProcessCreationTimestamp() on OS X and BSDs.

Thanks for the quick revision.

Sheriffs, please push this to try before checkin

Comment 6

[(dormant account)]

Comment on attachment 659569 [details] [diff] [review]
Bug 789693 - Unify CalculateProcessCreationTimestamp() on OS X and BSDs.

sorry, didn't notice that this was doing #define magic above #include calls. Please move #include calls up above defines.

Comment 7

[Jan Beich]

Attachment: Bug 789693 - Unify CalculateProcessCreationTimestamp() on OS X and BSDs.

Like this:

   #if defined(__DragonFly__) || defined(__FreeBSD__)
   #include <sys/user.h>
   #endif

  +#include "mozilla/Telemetry.h"
  +#include "mozilla/StartupTimeline.h"
  +
   #if defined(__NetBSD__)
   #undef KERN_PROC
  [...]
   #define KP_START_USEC p_ustart_usec
   #endif

  -#include "mozilla/Telemetry.h"
  -#include "mozilla/StartupTimeline.h"
  -
   static NS_DEFINE_CID(kAppShellCID, NS_APPSHELL_CID);

   #define kPrefLastSuccess "toolkit.startup.last_success"
   #define kPrefMaxResumedCrashes "toolkit.startup.max_resumed_crashes"
   #define kPrefRecentCrashes "toolkit.startup.recent_crashes"

   #if defined(XP_WIN)
   #include "mozilla/perfprobe.h"
   
or further below, e.g. after XP_WIN ifdef ?

Comment 8

[Jan Beich]

Needs a Try push first per comment 5.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/?tree=Try&rev=91e09969f338

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Looks green.

https://hg.mozilla.org/integration/mozilla-inbound/rev/61486f816524

Comment 11

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/61486f816524