1194935 - kill_and_get_minidump doesn't work on win64

Status: RESOLVED FIXED

(Testing :: Mozbase, defect)

Description

[Chris Manchester (limited bugmail, email directly)]

We're writing a 0 length dump. GetLastError leads me to http://stackoverflow.com/a/8835305 -- writing a dump of a 64 bit process (Firefox) from a 32 bit process (python) doesn't seem like something that's going to work.

Comment 1

[(not currently active) Ted Mielczarek]

:-/ We could write a little C++ helper app to do the dump writing, and compile that as a 64-bit binary.

Comment 2

[Chris Manchester (limited bugmail, email directly)]

(In reply to Ted Mielczarek [:ted.mielczarek] from comment #1)
> :-/ We could write a little C++ helper app to do the dump writing, and
> compile that as a 64-bit binary.

I'll give it a shot.

Comment 3

[Chris Manchester (limited bugmail, email directly)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=83ac13fd8d78

Comment 4

[Chris Manchester (limited bugmail, email directly)]

(In reply to Chris Manchester [:chmanchester] from comment #3)
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=83ac13fd8d78

This is working pretty well. I didn't make it unicode safe, but the other code path for kill_and_get_minidump does, so I'll do that now.

Comment 5

[Chris Manchester (limited bugmail, email directly)]

Attachment: MozReview Request: Bug 1194935 - Add a C++ implementation of kill_and_get_minidump as part of the build to accommodate dumping 64 bit Firefox from 32 bit python.

Bug 1194935 - Add a C++ implementation of kill_and_get_minidump as part of the build to accodomate dumping 64 bit Firefox from 32 bit python.

Comment 6

[Chris Manchester (limited bugmail, email directly)]

Comment on attachment 8649598 [details]
MozReview Request: Bug 1194935 - Add a C++ implementation of kill_and_get_minidump as part of the build to accommodate dumping 64 bit Firefox from 32 bit python.

Bug 1194935 - Add a C++ implementation of kill_and_get_minidump as part of the build to accommodate dumping 64 bit Firefox from 32 bit python.

Comment 7

[(not currently active) Ted Mielczarek]

Comment on attachment 8649598 [details]
MozReview Request: Bug 1194935 - Add a C++ implementation of kill_and_get_minidump as part of the build to accommodate dumping 64 bit Firefox from 32 bit python.

https://reviewboard.mozilla.org/r/16437/#review28981

Sorry I let this sit so long. :-(

::: testing/mozbase/mozcrash/mozcrash/mozcrash.py:377
(Diff revision 2)
> +            # we're running a python compiled for a different architecture.

I think I would move the explanation (dumping cross-architecure) up to the top of the block here to make it a little clearer.

::: testing/mozbase/mozcrash/mozcrash/mozcrash.py:391
(Diff revision 2)
> +            log.info("minidumpwriter status: %d" % status)

Do you care about failing on failure here? You could use subprocess.check_call. I feel like we should print some sort of failure line if this fails, certainly, but if it turns out to be ugly to plumb that up through the harness you could punt on it.

::: testing/tools/minidumpwriter/minidumpwriter.cpp:59
(Diff revision 2)
> +  return rv;

You never actually set rv=0 anywhere, do you?

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d18ca24050b7

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/d18ca24050b7