On 07/07/2014 05:12 PM, Ilia Mirkin wrote:
On Mon, Jul 7, 2014 at 11:47 AM, Samuel Pitoiset
<samuel.pitoi...@gmail.com> wrote:
V2:
- make group_id the last field of this struct, this fixes
drivers which use pipe_driver_query_info like radeon
While I have no problem with this patch, how exactly would placing
group_id at some other locatoin break radeon?
I believe Marek's comment was regarding the fact that you were
removing some fields that were used by radeon without updating the
driver accordingly.
When group_id is at the end of pipe_driver_query_info, I don't break
anything in radeon and in other drivers which use that struct, group_id
is defined implicitely to 0.
You can have a quick look at the code here :
http://cgit.freedesktop.org/mesa/mesa/tree/src/gallium/drivers/radeon/r600_pipe_common.c#n543
Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com>
---
src/gallium/include/pipe/p_defines.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/include/pipe/p_defines.h
b/src/gallium/include/pipe/p_defines.h
index 8f20974..065c8d9 100644
--- a/src/gallium/include/pipe/p_defines.h
+++ b/src/gallium/include/pipe/p_defines.h
@@ -733,6 +733,7 @@ struct pipe_driver_query_info
unsigned query_type; /* PIPE_QUERY_DRIVER_SPECIFIC + i */
uint64_t max_value; /* max value that can be returned */
boolean uses_byte_units; /* whether the result is in bytes */
+ unsigned group_id;
};
struct pipe_driver_query_group_info
--
2.0.0
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev