Conversation
cjreynol
force-pushed
the
calculated_gpu_mode_metrics
branch
from
October 1, 2026 03:31
47244a2 to
4611a34
Compare
cjreynol
changed the title
calculated_gpu_mode metricsgraphics_flags.h/cc and their usage in flags.cc
The passed in (or default) flag value was assigned instead of the calculated one. This vector of values was only used for `CheckSnapshotCompatibility`. Bug: 481846356
The goal is to separate the `GpuMode` calculating logic from the `CuttlefishConfig::MutableInstanceSpecific` mutation/setting logic. We need to be able to re-use the calculation logic for metrics without making any modifications. The plan is to pick apart the mixed logic to separate everything, then put all of the determination logic back together. Bug: 481846356
Piecemeal stripping out logic from the monolithic function with mutation. Bug: 481846356
Bug: 481846356
Bug: 481846356
Bug: 481846356
Bug: 481846356
Bug: 481846356
cjreynol
force-pushed
the
calculated_gpu_mode_metrics
branch
from
October 1, 2026 21:39
4611a34 to
8907aea
Compare
So both `#ifdef __APPLE__` and `#ifndef` have the same interface. This will make it easier to use for metrics. Bug: 481846356
To avoid the extra calls and conditionals in the logic for setting config values. Bug: 481846356
Bug: 481846356
The entire anonymous namespace belonged in the same block. The [[maybe_unused]] notation is not necessary with this condensing. The only remaining `#ifdef __APPLE__` code should be the alternative versions of `GetGraphicsAvailability` and `SelectGpuMode`. Bug: 481846356
cjreynol
force-pushed
the
calculated_gpu_mode_metrics
branch
from
October 1, 2026 21:52
8907aea to
b09eb25
Compare
cjreynol
marked this pull request as ready for review
October 1, 2026 21:52
cjreynol
added
kokoro:force-run
GoogleCuttlefishTesterBot
removed
kokoro:run
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is pre-work for gathering the
calculated_gpu_modevalue for metrics. I needed to make the actualgpu_modeselection logic available at the top-level, and while I was at it I split all of the determination logic from the instance updating logic in the formerConfigureGpuSettings.Bug: 481846356