glhd: Grab framebuffer state checks from r300g.
authorCorbin Simpson <MostAwesomeDude@gmail.com>
Wed, 23 Jun 2010 06:00:44 +0000 (23:00 -0700)
committerCorbin Simpson <MostAwesomeDude@gmail.com>
Wed, 23 Jun 2010 06:00:44 +0000 (23:00 -0700)
src/gallium/drivers/galahad/glhd_context.c
src/gallium/drivers/galahad/glhd_context.h
src/gallium/drivers/r300/r300_state.c

index 6dfee2c7a71645fcefa3df241a860cdc39a0eca6..7a856ef5e1b71e1029681c5ee8369b7710dab389 100644 (file)
@@ -497,6 +497,16 @@ galahad_set_framebuffer_state(struct pipe_context *_pipe,
    struct pipe_framebuffer_state *state = NULL;
    unsigned i;
 
+   if (_state->nr_cbufs > PIPE_MAX_COLOR_BUFS) {
+      glhd_error("%d render targets bound, but only %d are permitted by API",
+         _state->nr_cbufs, PIPE_MAX_COLOR_BUFS);
+   } else if (_state->nr_cbufs >
+      pipe->screen->get_param(pipe->screen, PIPE_CAP_MAX_RENDER_TARGETS)) {
+      glhd_warn("%d render targets bound, but only %d are supported",
+         _state->nr_cbufs,
+         pipe->screen->get_param(pipe->screen, PIPE_CAP_MAX_RENDER_TARGETS));
+   }
+
    /* unwrap the input state */
    if (_state) {
       memcpy(&unwrapped_state, _state, sizeof(unwrapped_state));
index a8753d0255a4f72b85490ea7aad7d2526ad94bdd..4e71753ac3708b822a146b1cf0f4ff7ae998b7a5 100644 (file)
@@ -58,4 +58,7 @@ do { \
     fprintf(stderr, "\n"); \
 } while (0)
 
+#define glhd_error(...) \
+    glhd_warn(__VA_ARGS__);
+
 #endif /* GLHD_CONTEXT_H */
index bc2b62ba54177041c3815ca6f29d5690244f5c20..927e9362fb112fa4709b8b30e2ad6b8d138dc047 100644 (file)
@@ -668,12 +668,6 @@ static void
     unsigned max_width, max_height, i;
     uint32_t zbuffer_bpp = 0;
 
-    if (state->nr_cbufs > 4) {
-        fprintf(stderr, "r300: Implementation error: Too many MRTs in %s, "
-            "refusing to bind framebuffer state!\n", __FUNCTION__);
-        return;
-    }
-
     if (r300->screen->caps.is_r500) {
         max_width = max_height = 4096;
     } else if (r300->screen->caps.is_r400) {