Keith Packard kei...@keithp.com writes:
Eric Anholt e...@anholt.net writes:
Keith Packard kei...@keithp.com writes:
I don't see anything indicating that this code path is only used by
glamor.
True. It's a fix for DRI3 for either UXA or none. Mesa allocates a
single page for a 1x1
Eric Anholt e...@anholt.net writes:
OK, but isn't the problem with the code you're #if 0ing (which, really?
#if 0 in a patch being submitted for review?) that it's aligning to
2*height instead of height?
I went and did a bit of archaeology to figure out why it's using
2*height instead of just
Keith Packard kei...@keithp.com writes:
I don't see anything indicating that this code path is only used by
glamor.
---
src/uxa/intel_uxa.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/uxa/intel_uxa.c b/src/uxa/intel_uxa.c
index b396188..717754f 100644
---
Eric Anholt e...@anholt.net writes:
Keith Packard kei...@keithp.com writes:
I don't see anything indicating that this code path is only used by
glamor.
True. It's a fix for DRI3 for either UXA or none. Mesa allocates a
single page for a 1x1 texture, but this code thinks that should take two
---
src/uxa/intel_uxa.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/uxa/intel_uxa.c b/src/uxa/intel_uxa.c
index b396188..717754f 100644
--- a/src/uxa/intel_uxa.c
+++ b/src/uxa/intel_uxa.c
@@ -758,6 +758,7 @@ free_priv:
goto free_priv;
}
+#if