xf86-video-intel: 2 commits - src/i830_driver.c src/i830_uxa.c
Simon Farnsworth
simon.farnsworth at onelan.com
Wed Jul 8 09:42:18 PDT 2009
Michel Dänzer wrote:
> On Wed, 2009-07-08 at 09:18 -0700, Eric Anholt wrote:
>> New commits:
>> commit 40e7c9505265823786cf730214db84812a5e494e
<snip>
>> + /* Fail very large allocations on 32-bit systems. Large BOs will
>> + * tend to hit SW fallbacks frequently, and also will tend to fail
>> + * to successfully map when doing SW fallbacks because we overcommit
>> + * address space for BO access.
>> + *
>> + * Note that size should fit in 32 bits. We throw out >32767x32767x4,
>> + * and pitch alignment could get us up to 32768x32767x4.
>> + */
>> + if (sizeof(unsigned int) == 4 &&
>> [...]
>
> FWIW, this test is true on 64-bit systems as well.
>
>
Deeper in the code, we assume that size always fits in an unsigned long
(drm_intel_bo_alloc_for_render). I suspect that changing unsigned int to
unsigned long is Good Enough?
--
Simon Farnsworth
More information about the xorg-devel
mailing list