Skip to content
Snippets Groups Projects
  1. Dec 13, 2011
  2. Dec 06, 2011
  3. Dec 05, 2011
  4. Dec 04, 2011
  5. Nov 27, 2011
  6. Nov 10, 2011
  7. Nov 01, 2011
  8. Oct 29, 2011
  9. Oct 28, 2011
    • Emma Anholt's avatar
      intel: Add an interface for removing relocs after they're added. · 515cea6a
      Emma Anholt authored
      
      This lets us replace the current inner drawing loop of mesa:
      
      for each prim {
      	compute bo list
      	if (check_aperture_space(bo list)) {
      		batch_flush()
      		compute bo list
      		if (check_aperture_space(bo list)) {
      			whine_about_batch_size()
      			fall back;
      		}
      	}
      	upload state to BOs
      }
      
      with this inner loop:
      
      for each prim {
       retry:
      	upload state to BOs
      	if (check_aperture_space(batch)) {
      		if (!retried) {
      			reset_to_last_prim()
      			batch_flush()
      		} else {
      			if (batch_flush())
      				whine_about_batch_size()
      			goto retry;
      		}
      	}
      }
      
      This avoids having to implement code to walk over certain sets of GL
      state twice (the "compute bo list" step).  While it's not a
      performance improvement, it's a significant win in code complexity:
      about -200 lines, and one place to make mistakes related to aperture
      space instead of N places to forget some BO we should have included.
      
      Note how if we do a reset in the new loop , we immediately flush.  We
      don't need to check aperture space -- the kernel will tell us if we
      actually ran out of aperture or not.  And if we did run out of
      aperture, it's because either the single prim was too big, or because
      check_aperture was wrong at the point of setting up the last
      primitive.
      
      Reviewed-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
      515cea6a
    • Emma Anholt's avatar
      intel: Use stdbool.h for dealing with boolean values. · 2c2bdb36
      Emma Anholt authored
      
      A few of the bitfield-based booleans are left in place.  Changing them
      to "bool" results in the same code size, so I'm erring on the side of
      not changing things.
      
      Reviewed-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
      2c2bdb36
  10. Oct 19, 2011
  11. Oct 11, 2011
  12. Sep 26, 2011
  13. Sep 23, 2011
  14. Sep 22, 2011
  15. Sep 21, 2011
  16. Sep 18, 2011
    • Marcin Ślusarz's avatar
      nouveau: assert argument cannot have side effects · 7d838161
      Marcin Ślusarz authored
      ... because argument is evaluated only if NDEBUG is not defined
      7d838161
    • Marcin Ślusarz's avatar
      drm mode: fix drmIoctl wrapper · 763b618d
      Marcin Ślusarz authored
      Both drmIoctl and ioctl define second argument as unigned long.
      
      Debugging/tracing tools (like strace or valgrind) on 64-bit machines see
      different request value for ioctls with 32nd bit set, because casting
      signed int to unsigned long extends 32nd bit to upper word, so 0x80000000
      becomes 0xFFFFFFFF80000000)
      
      Nobody noticed because higher 32 bits are chopped off on their way to kernel.
      763b618d
  17. Aug 01, 2011
  18. Jul 20, 2011
  19. Jul 12, 2011
  20. Jun 04, 2011
  21. May 14, 2011
Loading