mirrors/sway
1
0
Fork 0
mirror of https://github.com/swaywm/sway.git synced 2024-12-27 07:26:55 +01:00
Code Issues Projects Releases Packages Wiki Activity

config/output: Print output state during tests

Browse source 
Instead of having each search function print its various test decisions,
print the full state at the end of every search. This makes it much
clearer what state a particular test includes.
This commit is contained in:
Kenny Levinsen 2024-04-21 17:41:19 +02:00
parent 4c28916d68
commit 2686afb95c
1 changed files with 32 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

46
sway/config/output.c
View file

@ -641,6 +641,30 @@ struct search_context {
bool degrade_to_off; bool degrade_to_off;
}; };
static void dump_output_state(struct wlr_output *wlr_output, struct wlr_output_state *state) {
sway_log(SWAY_DEBUG, "Output state for %s", wlr_output->name);
if (state->committed & WLR_OUTPUT_STATE_ENABLED) {
sway_log(SWAY_DEBUG, " enabled: %s", state->enabled ? "yes" : "no");
}
if (state->committed & WLR_OUTPUT_STATE_RENDER_FORMAT) {
sway_log(SWAY_DEBUG, " render_format: %d", state->render_format);
}
if (state->committed & WLR_OUTPUT_STATE_MODE) {
if (state->mode_type == WLR_OUTPUT_STATE_MODE_CUSTOM) {
sway_log(SWAY_DEBUG, " custom mode: %dx%d@%dmHz",
state->custom_mode.width, state->custom_mode.height, state->custom_mode.refresh);
} else {
sway_log(SWAY_DEBUG, " mode: %dx%d@%dmHz%s",
state->mode->width, state->mode->height, state->mode->refresh,
state->mode->preferred ? " (preferred)" : "");
}
}
if (state->committed & WLR_OUTPUT_STATE_ADAPTIVE_SYNC_ENABLED) {
sway_log(SWAY_DEBUG, " adaptive_sync: %s",
state->adaptive_sync_enabled ? "enabled": "disabled");
}
}
static bool search_valid_config(struct search_context *ctx, size_t output_idx); static bool search_valid_config(struct search_context *ctx, size_t output_idx);
static void reset_output_state(struct wlr_output_state *state) { static void reset_output_state(struct wlr_output_state *state) {
@ -664,7 +688,12 @@ static void clear_later_output_states(struct wlr_backend_output_state *states,
} }
static bool search_finish(struct search_context *ctx, size_t output_idx) { static bool search_finish(struct search_context *ctx, size_t output_idx) {
struct wlr_backend_output_state *backend_state = &ctx->states[output_idx];
struct wlr_output_state *state = &backend_state->base;
struct wlr_output *wlr_output = backend_state->output;
clear_later_output_states(ctx->states, ctx->configs_len, output_idx); clear_later_output_states(ctx->states, ctx->configs_len, output_idx);
dump_output_state(wlr_output, state);
return wlr_output_swapchain_manager_prepare(ctx->swapchain_mgr, ctx->states, ctx->configs_len) && return wlr_output_swapchain_manager_prepare(ctx->swapchain_mgr, ctx->states, ctx->configs_len) &&
search_valid_config(ctx, output_idx+1); search_valid_config(ctx, output_idx+1);
} }
@ -681,8 +710,6 @@ static bool search_adaptive_sync(struct search_context *ctx, size_t output_idx)
} }
} }
if (!cfg->config || cfg->config->adaptive_sync != -1) { if (!cfg->config || cfg->config->adaptive_sync != -1) {
sway_log(SWAY_DEBUG, "Trying with adaptive sync disabled for: %s",
backend_state->output->name);
wlr_output_state_set_adaptive_sync_enabled(state, false); wlr_output_state_set_adaptive_sync_enabled(state, false);
if (search_finish(ctx, output_idx)) { if (search_finish(ctx, output_idx)) {
return true; return true;
@ -690,8 +717,6 @@ static bool search_adaptive_sync(struct search_context *ctx, size_t output_idx)
} }
// If adaptive sync has not been set, or fallback in case we are on a // If adaptive sync has not been set, or fallback in case we are on a
// backend that cannot disable adaptive sync such as the wayland backend. // backend that cannot disable adaptive sync such as the wayland backend.
sway_log(SWAY_DEBUG, "Trying with adaptive sync unset for: %s",
backend_state->output->name);
state->committed &= ~WLR_OUTPUT_STATE_ADAPTIVE_SYNC_ENABLED; state->committed &= ~WLR_OUTPUT_STATE_ADAPTIVE_SYNC_ENABLED;
return search_finish(ctx, output_idx); return search_finish(ctx, output_idx);
} }
@ -708,7 +733,6 @@ static bool search_mode(struct search_context *ctx, size_t output_idx) {
struct wlr_output_mode *preferred_mode = wlr_output_preferred_mode(wlr_output); struct wlr_output_mode *preferred_mode = wlr_output_preferred_mode(wlr_output);
if (preferred_mode) { if (preferred_mode) {
sway_log(SWAY_DEBUG, "Trying with preferred mode for: %s", backend_state->output->name);
wlr_output_state_set_mode(state, preferred_mode); wlr_output_state_set_mode(state, preferred_mode);
if (search_adaptive_sync(ctx, output_idx)) { if (search_adaptive_sync(ctx, output_idx)) {
return true; return true;
@ -725,8 +749,6 @@ static bool search_mode(struct search_context *ctx, size_t output_idx) {
if (mode == preferred_mode) { if (mode == preferred_mode) {
continue; continue;
} }
sway_log(SWAY_DEBUG, "Trying with mode %dx%d@%dmHz for: %s",
mode->width, mode->height, mode->refresh, backend_state->output->name);
wlr_output_state_set_mode(state, mode); wlr_output_state_set_mode(state, mode);
if (search_adaptive_sync(ctx, output_idx)) { if (search_adaptive_sync(ctx, output_idx)) {
return true; return true;
@ -765,8 +787,6 @@ static bool search_render_format(struct search_context *ctx, size_t output_idx)
// This is not a supported format for this output // This is not a supported format for this output
continue; continue;
} }
sway_log(SWAY_DEBUG, "Trying with render format %d for: %s", fmts[idx],
wlr_output->name);
wlr_output_state_set_render_format(state, fmts[idx]); wlr_output_state_set_render_format(state, fmts[idx]);
if (search_mode(ctx, output_idx)) { if (search_mode(ctx, output_idx)) {
return true; return true;
@ -784,9 +804,7 @@ static bool search_valid_config(struct search_context *ctx, size_t output_idx) {
struct matched_output_config *cfg = &ctx->configs[output_idx]; struct matched_output_config *cfg = &ctx->configs[output_idx];
struct wlr_backend_output_state *backend_state = &ctx->states[output_idx]; struct wlr_backend_output_state *backend_state = &ctx->states[output_idx];
struct wlr_output_state *state = &backend_state->base; struct wlr_output_state *state = &backend_state->base;
struct wlr_output *wlr_output = backend_state->output;
sway_log(SWAY_DEBUG, "Finding valid config for: %s",
backend_state->output->name);
if (!output_config_is_disabling(cfg->config)) { if (!output_config_is_disabling(cfg->config)) {
// Search through our possible configurations, doing a depth-first // Search through our possible configurations, doing a depth-first
@ -800,8 +818,8 @@ static bool search_valid_config(struct search_context *ctx, size_t output_idx) {
} }
// We could not get anything to work, try to disable this output to see // We could not get anything to work, try to disable this output to see
// if we can at least make the outputs before us work. // if we can at least make the outputs before us work.
sway_log(SWAY_DEBUG, "Trying with disabled output for: %s", sway_log(SWAY_DEBUG, "Unable to find valid config with output %s, disabling",
backend_state->output->name); wlr_output->name);
reset_output_state(state); reset_output_state(state);
} }