<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Sounds great.</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Reviewed-by: Jose Fonseca <jfonseca@vmware.com></div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
BTW, it's not really difficult to do gather with unsigned offsets: add 0x80000000 to the base, subtract 0x8000000 to the offsets, and use the signed gather. If the cost of doing so is significant, we could do this just for large textures, by adding a bit
per texture to the shader key.</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Jose</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div id="appendonsend"></div>
<div style="font-family:Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> sroland@vmware.com <sroland@vmware.com><br>
<b>Sent:</b> Thursday, October 10, 2019 19:18<br>
<b>To:</b> Jose Fonseca <jfonseca@vmware.com>; mesa-dev@lists.freedesktop.org <mesa-dev@lists.freedesktop.org><br>
<b>Cc:</b> Roland Scheidegger <sroland@vmware.com><br>
<b>Subject:</b> [PATCH] llvmpipe: increase max texture size to 2GB</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt">
<div class="PlainText">From: Roland Scheidegger <sroland@vmware.com><br>
<br>
The 1GB limit was arbitrary, increase this to 2GB (which is the max<br>
possible without code changes).<br>
---<br>
src/gallium/drivers/llvmpipe/lp_limits.h | 6 +++++-<br>
1 file changed, 5 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/gallium/drivers/llvmpipe/lp_limits.h b/src/gallium/drivers/llvmpipe/lp_limits.h<br>
index c2808162c78..569179ecdf4 100644<br>
--- a/src/gallium/drivers/llvmpipe/lp_limits.h<br>
+++ b/src/gallium/drivers/llvmpipe/lp_limits.h<br>
@@ -43,7 +43,11 @@<br>
/**<br>
* Max texture sizes<br>
*/<br>
-#define LP_MAX_TEXTURE_SIZE (1 * 1024 * 1024 * 1024ULL) /* 1GB for now */<br>
+/**<br>
+ * 2GB is the actual max currently (we always use 32bit offsets, and both<br>
+ * llvm GEP as well as avx2 gather use signed offsets).<br>
+ */<br>
+#define LP_MAX_TEXTURE_SIZE (2 * 1024 * 1024 * 1024ULL)<br>
#define LP_MAX_TEXTURE_2D_LEVELS 14 /* 8K x 8K for now */<br>
#define LP_MAX_TEXTURE_3D_LEVELS 12 /* 2K x 2K x 2K for now */<br>
#define LP_MAX_TEXTURE_CUBE_LEVELS 14 /* 8K x 8K for now */<br>
-- <br>
2.17.1<br>
<br>
</div>
</span></font></div>
</body>
</html>