From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 101189] Latest git fails to compile with radeon Date: Sat, 17 Jun 2017 06:44:45 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0769537264==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 6AF0E6E0BD for ; Sat, 17 Jun 2017 06:44:45 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============0769537264== Content-Type: multipart/alternative; boundary="14976818850.Eb12.9926"; charset="UTF-8" --14976818850.Eb12.9926 Date: Sat, 17 Jun 2017 06:44:45 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated https://bugs.freedesktop.org/show_bug.cgi?id=3D101189 --- Comment #19 from Mauro Rossi --- Hi,=20 with current patch "ac: remove amdgpu.h dependency" applied, on Android (nougat-x86) I get the following building errors: In file included from external/mesa/src/amd/common/ac_gpu_info.c:26: external/mesa/src/amd/common/ac_gpu_info.h:41:2: error: unknown type name 'uint32_t' uint32_t pci_domain; ^ external/mesa/src/amd/common/ac_gpu_info.h:42:2: error: unknown type name 'uint32_t' uint32_t pci_bus; ^ external/mesa/src/amd/common/ac_gpu_info.h:43:2: error: unknown type name 'uint32_t' uint32_t pci_dev; ^ external/mesa/src/amd/common/ac_gpu_info.h:44:2: error: unknown type name 'uint32_t' uint32_t pci_func; ^ external/mesa/src/amd/common/ac_gpu_info.h:47:2: error: unknown type name 'uint32_t' uint32_t pci_id; ^ external/mesa/src/amd/common/ac_gpu_info.h:50:2: error: unknown type name 'uint32_t' uint32_t pte_fragment_size; ^ external/mesa/src/amd/common/ac_gpu_info.h:51:2: error: unknown type name 'uint32_t' uint32_t gart_page_size; ^ external/mesa/src/amd/common/ac_gpu_info.h:52:2: error: unknown type name 'uint64_t' uint64_t gart_size; ^ external/mesa/src/amd/common/ac_gpu_info.h:53:2: error: unknown type name 'uint64_t' uint64_t vram_size; ^ external/mesa/src/amd/common/ac_gpu_info.h:54:2: error: unknown type name 'uint64_t' uint64_t vram_vis_size; ^ external/mesa/src/amd/common/ac_gpu_info.h:55:2: error: unknown type name 'uint64_t' uint64_t max_alloc_size; ^ external/mesa/src/amd/common/ac_gpu_info.h:56:2: error: unknown type name 'uint32_t' uint32_t min_alloc_size; ^ external/mesa/src/amd/common/ac_gpu_info.h:57:2: error: unknown type name 'bool' bool has_dedicated_vram; ^ external/mesa/src/amd/common/ac_gpu_info.h:58:2: error: unknown type name 'bool' bool has_virtual_memory; ^ external/mesa/src/amd/common/ac_gpu_info.h:59:2: error: unknown type name 'bool' bool gfx_ib_pad_with_type2; ^ external/mesa/src/amd/common/ac_gpu_info.h:60:2: error: unknown type name 'bool' bool has_hw_decode; ^ external/mesa/src/amd/common/ac_gpu_info.h:61:2: error: unknown type name 'uint32_t' uint32_t num_sdma_rings; ^ external/mesa/src/amd/common/ac_gpu_info.h:62:2: error: unknown type name 'uint32_t' uint32_t num_compute_rings; ^ external/mesa/src/amd/common/ac_gpu_info.h:63:2: error: unknown type name 'uint32_t' uint32_t uvd_fw_version; ^ fatal error: too many errors emitted, stopping now [-ferror-limit=3D] 20 errors generated. The following changes are necessary in src/amd/common/ac_gpu_info.h utente@utente-System-Product-Name:~/nougat-x86/external/mesa$ git diff diff --git a/src/amd/common/ac_gpu_info.h b/src/amd/common/ac_gpu_info.h index a72ab58f9a..616a1df7fa 100644 --- a/src/amd/common/ac_gpu_info.h +++ b/src/amd/common/ac_gpu_info.h @@ -26,6 +26,9 @@ #ifndef AC_GPU_INFO_H #define AC_GPU_INFO_H +#include +#include + #include "amd_family.h" #ifdef __cplusplus Having applied that I still get the following build error: In file included from external/mesa/src/amd/common/ac_surface.c:38: external/libdrm/amdgpu/amdgpu.h:107:31: error: typedef redefinition with different types ('struct amdgpu_device *' vs 'void *') typedef struct amdgpu_device *amdgpu_device_handle; ^ 1 error generated. Could you please check on the latter and provide corrective patch? I am available to test build on nougat-x86 Thanks Mauro --=20 You are receiving this mail because: You are the assignee for the bug.= --14976818850.Eb12.9926 Date: Sat, 17 Jun 2017 06:44:45 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated

Comme= nt # 19 on bug 10118= 9 from Mauro Rossi
Hi,=20

with current patch "ac: remove amdgpu.h dependency" applied,
on Android (nougat-x86) I get the following building errors:

In file included from external/mesa/src/amd/common/ac_gpu_info.c:26:
external/mesa/src/amd/common/ac_gpu_info.h:41:2: error: unknown type name
'uint32_t'
        uint32_t                    pci_domain;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:42:2: error: unknown type name
'uint32_t'
        uint32_t                    pci_bus;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:43:2: error: unknown type name
'uint32_t'
        uint32_t                    pci_dev;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:44:2: error: unknown type name
'uint32_t'
        uint32_t                    pci_func;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:47:2: error: unknown type name
'uint32_t'
        uint32_t                    pci_id;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:50:2: error: unknown type name
'uint32_t'
        uint32_t                    pte_fragment_size;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:51:2: error: unknown type name
'uint32_t'
        uint32_t                    gart_page_size;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:52:2: error: unknown type name
'uint64_t'
        uint64_t                    gart_size;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:53:2: error: unknown type name
'uint64_t'
        uint64_t                    vram_size;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:54:2: error: unknown type name
'uint64_t'
        uint64_t                    vram_vis_size;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:55:2: error: unknown type name
'uint64_t'
        uint64_t                    max_alloc_size;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:56:2: error: unknown type name
'uint32_t'
        uint32_t                    min_alloc_size;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:57:2: error: unknown type name
'bool'
        bool                        has_dedicated_vram;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:58:2: error: unknown type name
'bool'
        bool                        has_virtual_memory;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:59:2: error: unknown type name
'bool'
        bool                        gfx_ib_pad_with_type2;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:60:2: error: unknown type name
'bool'
        bool                        has_hw_decode;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:61:2: error: unknown type name
'uint32_t'
        uint32_t                    num_sdma_rings;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:62:2: error: unknown type name
'uint32_t'
        uint32_t                    num_compute_rings;
        ^
external/mesa/src/amd/common/ac_gpu_info.h:63:2: error: unknown type name
'uint32_t'
        uint32_t                    uvd_fw_version;
        ^
fatal error: too many errors emitted, stopping now [-ferror-limit=3D]
20 errors generated.


The following changes are necessary in src/amd/common/ac_gpu_info.h

utente@utente-System-Product-Name:~/nougat-x86/external/mesa$ git diff
diff --git a/src/amd/common/ac_gpu_info.h b/src/amd/common/ac_gpu_info.h
index a72ab58f9a..616a1df7fa 100644
--- a/src/amd/common/ac_gpu_info.h
+++ b/src/amd/common/ac_gpu_info.h
@@ -26,6 +26,9 @@
 #ifndef AC_GPU_INFO_H
 #define AC_GPU_INFO_H

+#include <stdint.h>
+#include <stdbool.h>
+
 #include "amd_family.h"

 #ifdef __cplusplus


Having applied that I still get the following build error:

In file included from external/mesa/src/amd/common/ac_surface.c:38:
external/libdrm/amdgpu/amdgpu.h:107:31: error: typedef redefinition with
different types ('struct amdgpu_device *' vs 'void *')
typedef struct amdgpu_device *amdgpu_device_handle;
                              ^
1 error generated.


Could you please check on the latter and provide corrective patch?
I am available to test build on nougat-x86

Thanks

Mauro


You are receiving this mail because:
  • You are the assignee for the bug.
= --14976818850.Eb12.9926-- --===============0769537264== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============0769537264==--