diff options
| author | SeongJae Park <sj@kernel.org> | 2024-04-21 09:25:49 -0700 |
|---|---|---|
| committer | SeongJae Park <sj@kernel.org> | 2024-04-21 09:26:46 -0700 |
| commit | 01def38580612da18b0d789d0eb9acab2090ee53 (patch) | |
| tree | 38a3df86d0bb658284e0bd256e5a61c45d94f7c3 | |
| parent | f6c5cb33375e70450cff2c2f3ef3cb5706825cc9 (diff) | |
| download | damo-01def38580612da18b0d789d0eb9acab2090ee53.tar.gz | |
_damo_records: Handle profile and memory footprint from finish_recording()
When handle.file_format == file_type_perf_data, finish_recording()
doesn't care profile and memory footprint recording. Handle the case.
Signed-off-by: SeongJae Park <sj@kernel.org>
| -rw-r--r-- | _damo_records.py | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/_damo_records.py b/_damo_records.py index d8641279..9da8c93b 100644 --- a/_damo_records.py +++ b/_damo_records.py @@ -748,13 +748,12 @@ def finish_recording(handle): if handle.file_format == file_type_perf_data: os.chmod(handle.file_path, handle.file_permission) - return - - err = update_records_file(handle.file_path, handle.file_format, - handle.file_permission, handle.monitoring_intervals) - if err != None: - print('converting format from perf_data to %s failed (%s)' % - (handle.file_format, err)) + else: + err = update_records_file(handle.file_path, handle.file_format, + handle.file_permission, handle.monitoring_intervals) + if err != None: + print('converting format from perf_data to %s failed (%s)' % + (handle.file_format, err)) if handle.perf_profile_pipe is not None: try: |