Stream: t-compiler/wg-self-profile

Topic: format version 3 is not supported


Simon Sapin (Jan 08 2020 at 11:29, on Zulip):

Hi. I tried the instructions in https://github.com/rust-lang/measureme/blob/master/summarize/Readme.md#profiling-the-nightly-compiler in Servo and got Error: "Event stream file format version \'3\' is not supported\n by this version of `measureme`.". Is this a known issue?

Simon Sapin (Jan 08 2020 at 11:30, on Zulip):

Note that we’re on nightly-2019-12-23. Is a more recent Nightly required?

lqd (Jan 08 2020 at 11:35, on Zulip):

likely

lqd (Jan 08 2020 at 11:37, on Zulip):

you're getting this when running summarize right ?

lqd (Jan 08 2020 at 11:42, on Zulip):

(I wonder if summarize can read old formats; just in case it's the tool which is too recent and not the nightly too old; rustc still uses measureme 0.5 and it seems the repo has a newer version; I think the version format in the repo is 5)

Simon Sapin (Jan 08 2020 at 11:50, on Zulip):

Yes, when running summarize, which I compiled per instructions from $ git clone https://github.com/rust-lang/measureme.git

simulacrum (Jan 08 2020 at 11:54, on Zulip):

@Simon Sapin can you try rebuilding measureme at the 0.5 or 0.5.1 tag?

lqd (Jan 08 2020 at 11:55, on Zulip):

yeah I suggested that elsewhere, and am also trying it rn

lqd (Jan 08 2020 at 11:58, on Zulip):

git checkout fb50acb will fix the format error yeah; I'm still seeing other "unexpectedly encountered events" but I'm on a newer nightly, I'll check on 12/23 next

lqd (Jan 08 2020 at 12:00, on Zulip):

yup it'll be enough, it's good on 12/23

Simon Sapin (Jan 08 2020 at 12:11, on Zulip):

0.5.1 works, thanks

Alex Crichton (Jan 10 2020 at 23:33, on Zulip):

@Wesley Wiser @mw was wondering, do y'all know what's going on here? It looks like measureme is updated to 0.7.1 in the rust-lang/rust repository, but is it still emitting an older format?

lqd (Jan 10 2020 at 23:36, on Zulip):

it's https://github.com/rust-lang/rust/pull/67397#issuecomment-573153365 IIUC — perf.rlo was apparently updated to the new summarize version already but it seems not to work yet

lqd (Jan 10 2020 at 23:38, on Zulip):

(0.7.1 emits version 5, if that's what you're talking about)

lqd (Jan 10 2020 at 23:38, on Zulip):

are you getting an error about version 3 ?

Wesley Wiser (Jan 10 2020 at 23:53, on Zulip):

@mw 's PR that updates rustc to the latest version just merged today so this should be resolved in the next nightly.

Wesley Wiser (Jan 10 2020 at 23:53, on Zulip):

@Alex Crichton ^^

lqd (Jan 10 2020 at 23:53, on Zulip):

(my understanding is this: if you have version 3, on nightly, use summarize 0.5.1;if you have version 5, on master, use summarize 0.7.1)

Alex Crichton (Jan 10 2020 at 23:53, on Zulip):

aha, nevermind, carry on then!

Alex Crichton (Jan 10 2020 at 23:54, on Zulip):

sorry for the bother :)

Last update: Jul 02 2020 at 18:50UTC