Fixed compilation issues

* Fixed some compilation issues
* Update GenerateAndroidMk.py fix compile issue and
update tpl for cmrt and media_driver.
* Add 'media_add_curr_to_include_path()' to media_srcs.cmake

Tracked-On: OAM-117146
Signed-off-by: zhangyichix <[email protected]>
Signed-off-by: zhepeng.xu <[email protected]>

Author: zhangyichix

Tools/MediaDriverTools/Android/GenerateAndroidMk.py

@@ -43,42 +43,44 @@ def remove(f):
     cmd = "rm " + f + "> /dev/null 2>&1"
     os.system(cmd)
 
+
 def getDriverName(root):
     driver = "mod"
     if not path.exists(path.join(root, driver)):
         driver = "media-driver"
     if not path.exists(path.join(root, driver)):
-        raise Exception("driver path " + driver +" not existed")
+        raise Exception("driver path " + driver + " not existed")
     return driver
 
+
 class Generator:
     def __init__(self, src, root, makefile=None):
-        #where to put the Android makefile
+        # where to put the Android makefile
         self.makefile = makefile if makefile else src
 
-        #where is the major source file
+        # where is the major source file
         self.src = src
 
-        #driver name
+        # driver name
         driver = getDriverName(root)
-        #where to put build file and template
+        # where to put build file and template
         self.tool = path.join(root, driver, TOOL_DIR)
 
         """where to put the template"""
         self.templ = path.join(root, driver, TEMPLATE_DIR)
 
-    #major function
+    # major function
     def generate(self):
 
-        if path.exists(self.src) == False:
+        if not path.exists(self.src):
             raise Exception(self.src + "not existed")
-        self.generateMakefile()
+        self.generateMakefile(debug=True)
 
         mk = path.join(self.makefile, "Android.mk")
-        #remove old Android.mk
+        # remove old Android.mk
         remove(mk)
 
-        #create new Android.mk
+        # create new Android.mk
         with open(self.getTemplatePath()) as f:
             tpl = f.read()
         tpl = tpl.replace("@LOCAL_SRC_FILES", self.getSources())
@@ -88,7 +90,7 @@ def generate(self):
             f.write(tpl)
         print("generated " + self.getName() + " to " + self.makefile)
 
-    #virtuall functions
+    # virtuall functions
     def getTemplate(self):
         raise Exception("pure virtul function")
 
@@ -108,19 +110,19 @@ def getBuildDir(self):
         return path.join(self.tool, 'build', self.getName())
 
     def adjustSources(self, lines):
-        #print(lines)
+        # print(lines)
         return lines
 
     def adjustIncludes(self, lines):
-        #print(lines)
+        # print(lines)
         return lines
 
     def getCmakeCmd(self):
         return "cmake " + self.src
 
     def generateMakefile(self, debug=False):
-        #Win env can help us debug the script
-        #but we do not want generate makefile on Win-system
+        # Win env can help us debug the script
+        # but we do not want generate makefile on Win-system
         if os.name == "nt":
             return
         verbose = ";" if debug else "> /dev/null 2>&1;"
@@ -136,11 +138,13 @@ def getIncludes(self):
         includes = []
         lines = text.split("\n")
         for l in lines:
-            #normpath will make sure we did not refer outside.
+            # normpath will make sure we did not refer outside.
+            l = l.strip()[2:]
+            if l == '/linux':
+                continue
+
             p = path.normpath(l)
-            j = p.find(self.src)
-            if j != -1:
-                includes.append(p[j:].replace(self.src, "$(LOCAL_PATH)"))
+            includes.append(path.join("$(LOCAL_PATH)", path.relpath(p, self.src)))
         return INDENT + ("\n" + INDENT).join(includes) if includes else ""
 
     def getDefines(self, name):
@@ -161,7 +165,7 @@ def getSources(self):
         lines = re.findall(".*?\\.o:\\n", text)
         lines = [l.replace(".o:\n", " \\") for l in lines]
         self.adjustSources(lines)
-        #make source pretty
+        # make source pretty
         return INDENT + ("\n" + INDENT).join(lines)
 
 
@@ -175,6 +179,7 @@ def getTemplate(self):
 
     def getMakefile(self):
         return "Source/GmmLib/Makefile"
+
     def getFlagsfile(self):
         return "Source/GmmLib/CMakeFiles/igfx_gmmumd_dll.dir/flags.make"
 
@@ -184,7 +189,7 @@ def adjustSources(self, lines):
             if j == -1:
                 lines[i] = path.join("Source/GmmLib", l)
             else:
-                lines[i] = path.join("Source", l[j+3:])
+                lines[i] = path.join("Source", l[j + 3:])
 
 
 class CmrtGeneator(Generator):
@@ -207,7 +212,7 @@ def adjustSources(self, lines):
             if j == -1:
                 lines[i] = path.join("linux", l)
             else:
-                lines[i] = l[j+3:]
+                lines[i] = l[j + 3:]
 
 
 class DriverGeneator(Generator):
@@ -217,11 +222,19 @@ def __init__(self, root):
 
     def getCmakeCmd(self):
         wd = path.join(self.src, "..")
-        cmd = 'cmake ' + wd +' -DCMAKE_INSTALL_PREFIX=/usr'
+        cmd = 'cmake ' + wd + ' -DCMAKE_INSTALL_PREFIX=/usr'
         cmd += ' -DBUILD_ALONG_WITH_CMRTLIB=1 -DBS_DIR_GMMLIB=' + path.join(wd, '../gmmlib/Source/GmmLib/')
         cmd += ' -DBS_DIR_COMMON=' + path.join(wd, '../gmmlib/Source/Common/')
         cmd += ' -DBS_DIR_INC=' + path.join(wd, '../gmmlib/Source/inc/')
         cmd += ' -DBS_DIR_MEDIA=' + wd
+        cmd += (' -DSKIP_GMM_CHECK=ON'
+                ' -DGEN12=ON'
+                ' -DXe_M=ON'
+                ' -DXEHP_SDV=ON'
+                ' -DENABLE_PRODUCTION_KMD=ON'  # For /media_softlet/agnostic/Xe_R/Xe_HPC/renderhal
+                # ' -DGEN10=ON'  # For /media_interface/media_interfaces_m10_cnl 
+                ' -DGEN12_RKL=ON'  # For /media_interface/media_interfaces_m12_rkl
+                )
         return cmd
 
     def getTemplate(self):
@@ -234,14 +247,18 @@ def getFlagsfile(self):
         return "media_driver/CMakeFiles/iHD_drv_video.dir/flags.make"
 
     def adjustSources(self, lines):
-        lines[:] = [l for l in lines if "media_libva_putsurface_linux.cpp" not in l]
+        lines[:] = [l.replace('__/', '../') for l in lines
+                    if "media_libva_putsurface_linux.cpp" not in l
+                    and '/private/' not in l
+                    ]
+
 
 class Main:
 
     def run(self):
         tool = path.dirname(__file__)
         root = path.abspath(path.join(tool, "../../../../"))
-        print("root = "+root)
+        print("root = " + root)
         gens = [GmmGeneator(root), CmrtGeneator(root), DriverGeneator(root)]
         for g in gens:
             g.generate()

Tools/MediaDriverTools/Android/mk/cmrt.tpl

@@ -33,6 +33,10 @@ LOCAL_C_INCLUDES += \
 @LOCAL_C_INCLUDES
 
 LOCAL_CFLAGS += \
+    -Wno-error \
+    -Wno-unused-variable \
+    -Wno-unused-parameter \
+    -Wno-unused-private-field \
     -Wno-non-virtual-dtor \
     -DANDROID=1 \
 @LOCAL_CFLAGS

Tools/MediaDriverTools/Android/mk/media_driver.tpl

@@ -26,37 +26,56 @@ include $(CLEAR_VARS)
 LOCAL_SRC_FILES := \
 @LOCAL_SRC_FILES
 
+LOCAL_HEADER_LIBRARIES := libva_headers
+
 LOCAL_SHARED_LIBRARIES := \
     libsync \
     libcutils \
     libutils \
     libdrm \
     libva \
     liblog \
-
-
-LOCAL_STATIC_LIBRARIES = \
     libgmm_umd \
 
 LOCAL_CPPFLAGS = \
     -DDRV_I915 \
     -DOTC_GRALLOC \
-    -DANDROID_VERSION=800 \
     -fexceptions \
     -frtti \
     -std=c++14 \
+    -Wno-error \
+    -Wno-ignored-qualifiers \
+    -Wno-unused-parameter \
+    -Wno-missing-braces \
+    -Wno-overloaded-virtual \
+    -Wno-unused-variable \
+    -Wno-missing-field-initializers \
+    -Wno-unused-function \
+    -Wno-logical-op-parentheses \
+    -Wno-implicit-fallthrough \
+    -Wno-comment \
+    -Wno-unused-private-field \
+    -Wno-unused-value \
+    -Wno-parentheses-equality \
+    -Wno-unused-label \
+    -Wno-parentheses \
+    -Wno-c++11-narrowing \
+    -Wno-unused-lambda-capture \
+    -Wno-unreachable-code-loop-increment \
+    -Wno-delete-incomplete \
+    -DGMM_LIB_DLL \
 @LOCAL_CFLAGS
 
 LOCAL_CONLYFLAGS = -x c++
 LOCAL_CFLAGS = $(LOCAL_CPPFLAGS)
 
 LOCAL_C_INCLUDES  = \
 @LOCAL_C_INCLUDES
-
+    $(LOCAL_PATH)/../cmrtlib/linux/hardware \
 
 #LOCAL_CPP_FEATURES := rtti exceptions
 
 LOCAL_MODULE := i965_drv_video
 LOCAL_PROPRIETARY_MODULE := true
 
-include $(BUILD_SHARED_LIBRARY)
+include $(BUILD_SHARED_LIBRARY)

cmrtlib/Android.mk

@@ -0,0 +1,84 @@
+# Copyright(c) 2018 Intel Corporation
+
+# Permission is hereby granted, free of charge, to any person obtaining a
+# copy of this software and associated documentation files(the "Software"),
+# to deal in the Software without restriction, including without limitation
+# the rights to use, copy, modify, merge, publish, distribute, sublicense,
+# and / or sell copies of the Software, and to permit persons to whom the
+# Software is furnished to do so, subject to the following conditions:
+
+# The above copyright notice and this permission notice shall be included
+# in all copies or substantial portions of the Software.
+
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
+# OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
+# OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+# ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+# OTHER DEALINGS IN THE SOFTWARE.
+
+LOCAL_PATH:= $(call my-dir)
+
+
+$(info value of CMRT_BUILD_TYPE is: $(CMRT_BUILD_TYPE))
+$(info value of CMRT_ARCH is: $(CMRT_ARCH))
+
+include $(CLEAR_VARS)
+
+LOCAL_SRC_FILES := \
+    agnostic/hardware/cm_device.cpp \
+    agnostic/hardware/cm_device_export.cpp \
+    agnostic/hardware/cm_kernel_debugger.cpp \
+    agnostic/hardware/cm_perf_statistics.cpp \
+    agnostic/hardware/cm_queue.cpp \
+    agnostic/hardware/cm_surface_manager.cpp \
+    agnostic/hardware/cm_timer.cpp \
+    agnostic/share/cm_avs_state_msg_ex.cpp \
+    agnostic/share/cm_printf_host.cpp \
+    agnostic/share/cm_rt_helpers.cpp \
+    linux/hardware/cm_device_export_os.cpp \
+    linux/hardware/cm_device_os.cpp \
+    linux/hardware/cm_surface_manager_os.cpp \
+    linux/hardware/cm_timer_os.cpp \
+    linux/share/cm_performance.cpp \
+
+LOCAL_C_INCLUDES += \
+    $(LOCAL_PATH)/proprietary/share \
+    $(LOCAL_PATH)/agnostic/share \
+    $(LOCAL_PATH)/agnostic/hardware \
+    $(LOCAL_PATH)/linux/share \
+    $(LOCAL_PATH)/linux/hardware
+
+LOCAL_CFLAGS += \
+    -Wno-error \
+    -Wno-unused-variable \
+    -Wno-unused-parameter \
+    -Wno-unused-private-field \
+    -Wno-non-virtual-dtor \
+    -DANDROID=1 \
+    -DCM_RT_EXPORTS \
+    -DISTDLIB_UMD \
+    -DVPHAL \
+    -D__CT__ \
+    -Digfxcmrt_EXPORTS
+
+ifeq ($(CMRT_BUILD_TYPE), debug)
+    LOCAL_CFLAGS += \
+        -D_DEBUG \
+        -D__DEBUG \
+        -O0
+else
+    LOCAL_CFLAGS += \
+        -fno-strict-aliasing \
+        -D_FORTIFY_SOURCE=2
+endif
+
+
+LOCAL_MODULE := libigfxcmrt
+LOCAL_PROPRIETARY_MODULE := true
+
+LOCAL_LDLIBS := -lc -lva -lva-android
+LOCAL_SHARED_LIBRARIES := libc libdl libcutils liblog libutils libm libva libva-android
+
+include $(BUILD_SHARED_LIBRARY)

cmrtlib/linux/hardware/cm_device_os.cpp

@@ -493,8 +493,10 @@ CmDevice_RT::CmDevice_RT(
     m_gtpinBufferUP1(nullptr),
     m_gtpinBufferUP2(nullptr),
     m_createOption(createOption),
-    m_driverStoreEnabled(0),
-    m_driFileDescriptor(0)
+#if !defined(ANDROID)
+    m_driFileDescriptor(0),
+#endif
+    m_driverStoreEnabled(0)
 {
 
     // New Surface Manager

media_common/agnostic/common/cp/media_srcs.cmake

@@ -38,4 +38,5 @@ set(COMMON_CP_DIRECTORIES_
 set(CP_INTERFACE_DIRECTORIES_
     ${CP_INTERFACE_DIRECTORIES_}
     ${CMAKE_CURRENT_LIST_DIR}
-)
+)
+media_add_curr_to_include_path()

media_common/agnostic/common/heap_manager/media_srcs.cmake

@@ -36,4 +36,5 @@ source_group( "Common Files" FILES ${TMP_HEADERS_} )
 set(SOFTLET_COMMON_PRIVATE_INCLUDE_DIRS_
     ${SOFTLET_COMMON_PRIVATE_INCLUDE_DIRS_}
     ${CMAKE_CURRENT_LIST_DIR}
-)
+)
+media_add_curr_to_include_path()

media_common/agnostic/common/hw/media_srcs.cmake

@@ -118,4 +118,5 @@ set(SOFTLET_MHW_SFC_PRIVATE_INCLUDE_DIRS_
 set(SOFTLET_MHW_VEBOX_PRIVATE_INCLUDE_DIRS_
     ${SOFTLET_MHW_VEBOX_PRIVATE_INCLUDE_DIRS_}
     ${CMAKE_CURRENT_LIST_DIR}
-)
+)
+media_add_curr_to_include_path()

media_common/agnostic/common/hw/vdbox/media_srcs.cmake

@@ -53,4 +53,5 @@ set(TMP_SOFTLET_MHW_VDBOX_HUC_HEADERS_ "")
 set(SOFTLET_MHW_VDBOX_PRIVATE_INCLUDE_DIRS_
     ${SOFTLET_MHW_VDBOX_PRIVATE_INCLUDE_DIRS_}
     ${CMAKE_CURRENT_LIST_DIR}
-)
+)
+media_add_curr_to_include_path()