TraceMonkey Shark builds broken since May 01

RESOLVED FIXED

Status

()

Core
JavaScript Engine
--
major
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: nthomas, Assigned: nthomas)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(2 attachments)

(Assignee)

Description

8 years ago
Eg:
http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1272796320.1272797032.24308.gz

Something in this range of checkins:
http://hg.mozilla.org/tracemonkey/pushloghtml?fromchange=990192b0e052&tochange=b414c98f9030

Bug 562144 ?
On it -- must be some relation between --enable-shark and the dtrace hooks I didn't notice.
Assignee: general → cdleary
Status: NEW → ASSIGNED
Created attachment 443018 [details] [diff] [review]
Fix build break.
http://hg.mozilla.org/tracemonkey/rev/c63d6b20c590

I'll make sure it goes through before marking as fixed-in-tracemonkey, but it works on my 10.5 and Linux test machines.
It's no longer listed as burning on tinderbox. I don't know where it went (she shark build isn't even listed there anymore), but I'll assume it worked.
Whiteboard: fixed-in-tracemonkey
(Assignee)

Comment 5

8 years ago
It only runs once a day, with the other nightly builds, so falls off the waterfall after 12 hours. I've forced a new build to verify the fix. Thanks for the quick work.
(In reply to comment #5)

Thanks for finding it!
(Assignee)

Comment 7

8 years ago
Broken in a different way now:

g++-4.2  -fno-rtti ... -o libmozjs.dylib  jsapi.o ...
Undefined symbols:
  "_jsdtrace_execute_start", referenced from:
      _js_Execute in jsinvoke.o
  "_jsdtrace_function_return", referenced from:
      _js_Interpret in jsinterp.o
      _js_Interpret in jsinterp.o
      _js_Invoke in jsinvoke.o
  "_jsdtrace_execute_done", referenced from:
      _js_Execute in jsinvoke.o
  "_jsdtrace_function_rval", referenced from:
      _js_Interpret in jsinterp.o
      _js_Interpret in jsinterp.o
      _js_Invoke in jsinvoke.o
  "_jsdtrace_object_create", referenced from:
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsapi.o
      js::NewObject(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsarray.o
      _js_NewDateObjectMsec in jsdate.o
      _JS_NewSystemObject in jsdbgapi.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsexn.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsfun.o
      js::NewObject(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsinterp.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsinvoke.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsiter.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsobj.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in json.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsregexp.o
      js::NewObject(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jstypedarray.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsxml.o
  "_jsdtrace_object_finalize", referenced from:
      void FinalizeArenaList<JSObject, &(FinalizeObject(JSContext*, JSObject*, unsigned int))>(JSContext*, unsigned int)in jsgc.o
      void FinalizeArenaList<JSObject, &(FinalizeHookedObject(JSContext*, JSObject*, unsigned int))>(JSContext*, unsigned int)in jsgc.o
      _js_GC in jsgc.o
      _js_GC in jsgc.o
  "_jsdtrace_object_create_start", referenced from:
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsapi.o
      js::NewObject(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsarray.o
      _js_NewDateObjectMsec in jsdate.o
      _JS_NewSystemObject in jsdbgapi.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsexn.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsfun.o
      js::NewObject(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsinterp.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsinvoke.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsiter.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsobj.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in json.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsregexp.o
      js::NewObject(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jstypedarray.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsxml.o
  "_jsdtrace_object_create_done", referenced from:
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsapi.o
      js::NewObject(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsarray.o
      _js_NewDateObjectMsec in jsdate.o
      _JS_NewSystemObject in jsdbgapi.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsexn.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsfun.o
      js::NewObject(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsinterp.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsinvoke.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsiter.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsobj.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in json.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsregexp.o
      js::NewObject(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jstypedarray.o
      js::NewObjectWithGivenProto(JSContext*, JSClass*, JSObject*, JSObject*, unsigned long)in jsxml.o
  "_jsdtrace_function_info", referenced from:
      _js_Interpret in jsinterp.o
      _js_Interpret in jsinterp.o
      _js_Invoke in jsinvoke.o
ld: symbol(s) not found
collect2: ld returned 1 exit status

This has also spread to mozilla-central since tracemonkey merged there.
Severity: normal → major
Whiteboard: fixed-in-tracemonkey
(In reply to comment #7)
> Broken in a different way now:

Hmmm -- did that subsequent build you kicked off work in comment 5? Fix is most likely already committed via bug 563870.
(Assignee)

Comment 9

8 years ago
The May 2nd build on rev c63d6b20c590 looks like it failed in the same way as comment #7.

Comment 10

8 years ago
Created attachment 443820 [details] [diff] [review]
fix bustage on Solaris
Attachment #443820 - Flags: review?(cdleary)
Comment on attachment 443820 [details] [diff] [review]
fix bustage on Solaris

Ginn, thanks for the patch! We happened to put a fix into tracemonkey that covers these changes already in bug 563870 which has now been merged into mozilla-central  ala http://hg.mozilla.org/mozilla-central/rev/c052b348a626#l1.85 -- LMK if you have any questions!
Attachment #443820 - Flags: review?(cdleary)
nthomas, can you confirm unbustage so we can close as resolved?
Assignee: cdleary → nrthomas
(Assignee)

Comment 13

8 years ago
0f5867192284 for the May 06 nightly was green. Please merge to mozilla-central.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: fixed-in-tracemonkey
(In reply to comment #13)
> 0f5867192284 for the May 06 nightly was green. Please merge to mozilla-central.

nthomas, do you mean there's some additional merging that must be done on top of the one mentioned in comment 11?
(Assignee)

Comment 15

8 years ago
Sorry, failed to read that comment properly - no action needed.
You need to log in before you can comment on or make changes to this bug.