diff options
author | Victor "Nate" Graf <nategraf1@gmail.com> | 2017-08-24 15:13:49 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-08-24 15:13:49 -0700 |
commit | 421f9e7c64b05c71db6ef71791998c06249953f6 (patch) | |
tree | 3d416f3d85b892ff1b7c006de1453b1b5498bd2d /src/vm/eventpipebuffer.h | |
parent | 970c41b10cca6f0f4bc6c6524c6733ac2c5011ba (diff) | |
download | coreclr-421f9e7c64b05c71db6ef71791998c06249953f6.tar.gz coreclr-421f9e7c64b05c71db6ef71791998c06249953f6.tar.bz2 coreclr-421f9e7c64b05c71db6ef71791998c06249953f6.zip |
Remove unnecessary buffer copy from EventPipe::WriteEvent pipeline (#13347)
* [WIP] Eliminate extra buffer copy with new api path
* Copy blobs to a flat buffer is Rundown is on
* Refactor to use payload class and dedupe code
* Add contracts
* Fix many small errors
* Make names unambiguous
* Add EventPipe::WriteEventBlob to ecalllist.h
* Address code review
* Add test and fix a buffer copy bug
* Copy data instead of data pointer
* Add optional output file arg to tests
* Change failure return code
* Renamed variables for clarity
Diffstat (limited to 'src/vm/eventpipebuffer.h')
-rw-r--r-- | src/vm/eventpipebuffer.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/vm/eventpipebuffer.h b/src/vm/eventpipebuffer.h index f279a2865c..c96ad26609 100644 --- a/src/vm/eventpipebuffer.h +++ b/src/vm/eventpipebuffer.h @@ -7,6 +7,7 @@ #ifdef FEATURE_PERFTRACING +#include "eventpipe.h" #include "eventpipeevent.h" #include "eventpipeeventinstance.h" @@ -81,7 +82,7 @@ public: // Returns: // - true: The write succeeded. // - false: The write failed. In this case, the buffer should be considered full. - bool WriteEvent(Thread *pThread, EventPipeEvent &event, BYTE *pData, unsigned int dataLength, LPCGUID pActivityId, LPCGUID pRelatedActivityId, StackContents *pStack = NULL); + bool WriteEvent(Thread *pThread, EventPipeEvent &event, EventPipeEventPayload &payload, LPCGUID pActivityId, LPCGUID pRelatedActivityId, StackContents *pStack = NULL); // Get the timestamp of the most recent event in the buffer. LARGE_INTEGER GetMostRecentTimeStamp() const; |