[PATCH] fullscreen-shell: Fix modeset on transformed outputs

Jason Ekstrand jason at jlekstrand.net
Thu Jan 8 08:57:44 PST 2015


Previously, we blindly created a mode for the output based on surface size
and completely ignoring the output transform.  This caused modesets to fail
on outputs that were transformed by 90 or 270 degrees.  We should be
swapping the width and the height in this case.
---
 fullscreen-shell/fullscreen-shell.c | 23 +++++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/fullscreen-shell/fullscreen-shell.c b/fullscreen-shell/fullscreen-shell.c
index 35e6d8f..5fd7cd6 100644
--- a/fullscreen-shell/fullscreen-shell.c
+++ b/fullscreen-shell/fullscreen-shell.c
@@ -463,9 +463,28 @@ fs_output_configure_for_mode(struct fs_output *fsout,
 					&surf_x, &surf_y,
 					&surf_width, &surf_height);
 
+	/* The actual output mode is in physical units.  We need to
+	 * transform the surface size to physical unit size by flipping ans
+	 * possibly scaling it.
+	 */
+	switch (fsout->output->transform) {
+	case WL_OUTPUT_TRANSFORM_90:
+	case WL_OUTPUT_TRANSFORM_FLIPPED_90:
+	case WL_OUTPUT_TRANSFORM_270:
+	case WL_OUTPUT_TRANSFORM_FLIPPED_270:
+		mode.width = surf_height * fsout->output->native_scale;
+		mode.height = surf_width * fsout->output->native_scale;
+		break;
+
+	case WL_OUTPUT_TRANSFORM_NORMAL:
+	case WL_OUTPUT_TRANSFORM_FLIPPED:
+	case WL_OUTPUT_TRANSFORM_180:
+	case WL_OUTPUT_TRANSFORM_FLIPPED_180:
+	default:
+		mode.width = surf_width * fsout->output->native_scale;
+		mode.height = surf_height * fsout->output->native_scale;
+	}
 	mode.flags = 0;
-	mode.width = surf_width * fsout->output->native_scale;
-	mode.height = surf_height * fsout->output->native_scale;
 	mode.refresh = fsout->pending.framerate;
 
 	ret = weston_output_mode_switch_to_temporary(fsout->output, &mode,
-- 
2.2.0



More information about the wayland-devel mailing list