ilo: careful with empty fb state in ilo_gpe_set_fb()

We cannot pass 0 as the width or height to ilo_gpe_init_view_surface_null().

diff --git a/src/gallium/drivers/ilo/ilo_gpe_gen6.c b/src/gallium/drivers/ilo/ilo_gpe_gen6.c
index c3ba9e371479ea8c7d4dbad7692d5bebe9a4e70b..4dd5f13982a014dd49fbdfb28cbb9918eeca5a7b 100644 (file)
--- a/src/gallium/drivers/ilo/ilo_gpe_gen6.c
+++ b/src/gallium/drivers/ilo/ilo_gpe_gen6.c
@@ -1684,6 +1684,8 @@ ilo_gpe_init_view_surface_null_gen6(const struct ilo_dev_info *dev,
 
    ILO_GPE_VALID_GEN(dev, 6, 6);
 
+   assert(width >= 1 && height >= 1 && depth >= 1);
+
    /*
     * From the Sandy Bridge PRM, volume 4 part 1, page 71:
     *
@@ -2359,7 +2361,8 @@ ilo_gpe_set_fb(const struct ilo_dev_info *dev,
    util_copy_framebuffer_state(&fb->state, state);
 
    ilo_gpe_init_view_surface_null(dev,
-         state->width, state->height,
+         (state->width) ? state->width : 1,
+         (state->height) ? state->height : 1,
          1, 0, &fb->null_rt);
 
    first = NULL;

diff --git a/src/gallium/drivers/ilo/ilo_gpe_gen7.c b/src/gallium/drivers/ilo/ilo_gpe_gen7.c
index 3d3953713dac68a9c9a11d6225796b19be84c2b2..6f7399b7f425bb47033d9ee8eea4ea26d77d6802 100644 (file)
--- a/src/gallium/drivers/ilo/ilo_gpe_gen7.c
+++ b/src/gallium/drivers/ilo/ilo_gpe_gen7.c
@@ -243,6 +243,8 @@ ilo_gpe_init_view_surface_null_gen7(const struct ilo_dev_info *dev,
 
    ILO_GPE_VALID_GEN(dev, 7, 7.5);
 
+   assert(width >= 1 && height >= 1 && depth >= 1);
+
    /*
     * From the Ivy Bridge PRM, volume 4 part 1, page 62:
     *