Attachment 54605 Details for Bug 43893 – libdrm_radeon: account for write domain or read domains, not both

[patch] libdrm_radeon: account for write domain or read domains, not both

fdo43893-libdrm_radeon.diff (text/plain), 1.04 KB, created by Michel Dänzer on 2011-12-20 10:38:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Michel Dänzer

Created: 2011-12-20 10:38:08 UTC

Size: 1.04 KB

patch

obsolete

>diff --git a/radeon/radeon_cs_space.c b/radeon/radeon_cs_space.c
>index 208e1bb..be047a7 100644
>--- a/radeon/radeon_cs_space.c
>+++ b/radeon/radeon_cs_space.c
>@@ -65,13 +65,16 @@ static inline int radeon_cs_setup_bo(struct radeon_cs_space_check *sc, struct ra
>     }
> 
>     if (bo->space_accounted == 0) {
>-        if (write_domain == RADEON_GEM_DOMAIN_VRAM)
>-            sizes->op_vram_write += bo->size;
>-        else if (write_domain == RADEON_GEM_DOMAIN_GTT)
>-            sizes->op_gart_write += bo->size;
>-        else
>+        if (write_domain) {
>+            if (write_domain == RADEON_GEM_DOMAIN_VRAM)
>+                sizes->op_vram_write += bo->size;
>+            else if (write_domain == RADEON_GEM_DOMAIN_GTT)
>+                sizes->op_gart_write += bo->size;
>+            sc->new_accounted = write_domain;
>+        } else {
>             sizes->op_read += bo->size;
>-        sc->new_accounted = (read_domains << 16) | write_domain;
>+            sc->new_accounted = read_domains << 16;
>+        }
>     } else {
>         uint16_t old_read, old_write;
>

Actions: View | Splinter Review

Attachments on bug 43893: 54504 | 54505 | 54605 | 54606

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.