Skip to content

Honor malloc_perturb from vt config#4356

Open
bssrikanth wants to merge 1 commit into
avocado-framework:masterfrom
bssrikanth:malloc_perturb
Open

Honor malloc_perturb from vt config#4356
bssrikanth wants to merge 1 commit into
avocado-framework:masterfrom
bssrikanth:malloc_perturb

Conversation

@bssrikanth
Copy link
Copy Markdown

If a VT config file is supplied, leave malloc_perturb to the cfg and log that vt.qemu.malloc_perturb is ignored.

If a VT config file is supplied, leave malloc_perturb to the cfg and log that
vt.qemu.malloc_perturb is ignored.

Signed-off-by: Srikanth Aithal <srikanth.aithal@amd.com>
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request modifies the _process_malloc_perturb method in avocado_vt/options.py to conditionally assign the malloc_perturb setting. It now checks for the presence of a general configuration via vt.config and skips the assignment if one is found, logging the action instead. I have no feedback to provide as no review comments were submitted.

@harvey0100
Copy link
Copy Markdown
Contributor

Closing this PR due to current team constraints. This is part of a broader effort to triage all in-flight work across our upstream repos. If this work is still needed, please feel free to reopen and it will be picked up. Apologies for any inconvenience.

@harvey0100 harvey0100 closed this May 25, 2026
@bssrikanth
Copy link
Copy Markdown
Author

This PR is fixing a issue in the framework. I dont have any permissions to reopen it. Please consider to reopen. I am seeing above comment in many of my pull requests.
If you have any team constraints then I will be happy to assist, please let me know how I could be of any help.

@harvey0100 harvey0100 reopened this May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants