perfview icon indicating copy to clipboard operation
perfview copied to clipboard

Add Nettrace compression and multi-process support

Open mjsabby opened this issue 4 years ago • 6 comments

  • Adds C# implementation for LZ-based compression and decompression that is used in BPerf File Format (the file format we're intending to replace)
  • Adds a flag for the compression type
  • Adds next 4 bytes to header, this is the decompressed size.

mjsabby avatar Sep 05 '20 23:09 mjsabby

cc @noahfalk

mjsabby avatar Sep 05 '20 23:09 mjsabby

@mjsabby, are there corresponding runtime changes for this?

brianrob avatar Sep 08 '20 18:09 brianrob

@brianrob The runtime does not yet emit this. @noahfalk wanted the file format to be settled on first. We will have our tool generate it, and then hopefully I can port it to the runtime as an option as well.

mjsabby avatar Sep 08 '20 18:09 mjsabby

@noahfalk If you could do a once over to see if this is the direction you wanted ...

mjsabby avatar Sep 10 '20 07:09 mjsabby

@noahfalk If you could do a once over to see if this is the direction you wanted ...

Direction seemed fine to me, thanks @mjsabby! I spotted a few things in the details I commented on above

noahfalk avatar Sep 12 '20 03:09 noahfalk

cc @josalem @sywhang

noahfalk avatar Sep 12 '20 03:09 noahfalk