From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from fllv0015.ext.ti.com (fllv0015.ext.ti.com [198.47.19.141]) by arago-project.org (Postfix) with ESMTPS id 74F72529E4 for ; Mon, 9 Sep 2019 15:42:44 +0000 (UTC) Received: from fllv0034.itg.ti.com ([10.64.40.246]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id x89Ffdba023082 for ; Mon, 9 Sep 2019 10:41:39 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1568043699; bh=FPpcpVDm4+wzwuoQhHi5cCxc4aTuYfVpsPRRzaXKHlM=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=Lz26lAG6csG2uy94tjrF922538xZ6zoF9s5GrmQC2g/3X/EOxsdo2LiogbGoEycwD WrfC415oKxFxmgKqIGGFAf3wJuaJz8CFF5ABHIDcGNx4vVJ+l+yBfoJuejuZ7oG2lI KbcTJ2LoCZy2rHMPztES4L8yWwY30Eyp1TBVl+Cg= Received: from DLEE102.ent.ti.com (dlee102.ent.ti.com [157.170.170.32]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id x89Ffdog032233 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Mon, 9 Sep 2019 10:41:39 -0500 Received: from DLEE104.ent.ti.com (157.170.170.34) by DLEE102.ent.ti.com (157.170.170.32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1713.5; Mon, 9 Sep 2019 10:41:38 -0500 Received: from fllv0039.itg.ti.com (10.64.41.19) by DLEE104.ent.ti.com (157.170.170.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1713.5 via Frontend Transport; Mon, 9 Sep 2019 10:41:38 -0500 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0039.itg.ti.com (8.15.2/8.15.2) with ESMTP id x89FfcYm028878; Mon, 9 Sep 2019 10:41:38 -0500 Date: Mon, 9 Sep 2019 11:41:38 -0400 From: Denys Dmytriyenko To: Yuan Zhao Message-ID: <20190909154138.GJ2400@beryl> References: <20190909152101.84543-1-yuanzhao@ti.com> MIME-Version: 1.0 In-Reply-To: <20190909152101.84543-1-yuanzhao@ti.com> User-Agent: Mutt/1.5.24 (2015-08-30) X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Cc: meta-arago@arago-project.org Subject: Re: [PATCH v2] ocl: update version to 01.01.19.00 X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 09 Sep 2019 15:42:44 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Mon, Sep 09, 2019 at 10:21:01AM -0500, Yuan Zhao wrote: > - Selected OpenCL 1.2 features > - Runtime support update for TIDL API 01.03.01 > - Update OpenCL TIDL firmware to 01.01.19.00 > > Signed-off-by: Yuan Zhao > --- What changed in v2? > .../ocl-rtos/opencl-examples-rtos_git.bb | 2 +- > .../recipes-ti/ocl-rtos/opencl-rtos_git.bb | 2 + > .../ocl/files/0001-OpenCL_K2x_compile.patch | 12 ---- > .../ocl/files/0002-OpenCL_l2_cache_size.patch | 49 ---------------- > ...g-8.3.0-OpenCL-example-undefined-behavior.patch | 68 ---------------------- > meta-arago-extras/recipes-ti/ocl/ocl.inc | 5 +- > .../recipes-ti/ocl/opencl-examples_git.bb | 4 +- > .../recipes-ti/ocl/opencl-monitor_git.bb | 4 +- > .../recipes-ti/ocl/opencl-tidl-fw_git.bb | 4 +- > meta-arago-extras/recipes-ti/ocl/opencl_git.bb | 6 +- > 10 files changed, 13 insertions(+), 143 deletions(-) > delete mode 100644 meta-arago-extras/recipes-ti/ocl/files/0001-OpenCL_K2x_compile.patch > delete mode 100644 meta-arago-extras/recipes-ti/ocl/files/0002-OpenCL_l2_cache_size.patch > delete mode 100644 meta-arago-extras/recipes-ti/ocl/files/0003-Fix-g-8.3.0-OpenCL-example-undefined-behavior.patch > > diff --git a/meta-arago-extras/recipes-ti/ocl-rtos/opencl-examples-rtos_git.bb b/meta-arago-extras/recipes-ti/ocl-rtos/opencl-examples-rtos_git.bb > index 42bc0a9b..15ac0af7 100644 > --- a/meta-arago-extras/recipes-ti/ocl-rtos/opencl-examples-rtos_git.bb > +++ b/meta-arago-extras/recipes-ti/ocl-rtos/opencl-examples-rtos_git.bb > @@ -5,7 +5,7 @@ LICENSE = "BSD" > require recipes-ti/ocl/ocl.inc > require recipes-ti/includes/arago-paths.inc > > -PR = "${INC_PR}.1" > +PR = "${INC_PR}.0" > > COMPATIBLE_MACHINE = "omap-a15" > PACKAGE_ARCH = "${MACHINE_ARCH}" > diff --git a/meta-arago-extras/recipes-ti/ocl-rtos/opencl-rtos_git.bb b/meta-arago-extras/recipes-ti/ocl-rtos/opencl-rtos_git.bb > index 0c9aca7e..fd7e53c5 100644 > --- a/meta-arago-extras/recipes-ti/ocl-rtos/opencl-rtos_git.bb > +++ b/meta-arago-extras/recipes-ti/ocl-rtos/opencl-rtos_git.bb > @@ -38,6 +38,7 @@ export WANT_LLVM_RELEASE = "3.6-ti" > OCL_BUILD_TARGET_omap-a15 = "ARM_AM57" > > ENABLE_ULM = "0" > +SHMEM_MANAGER = "CMEM" > > RELEASE_TARGET = "" > RELEASE_TARGET_omap-a15 = "am57xx" > @@ -63,6 +64,7 @@ do_configure() { > -DBUILD_TARGET=${OCL_BUILD_TARGET} \ > -DBUILD_OUTPUT=lib \ > -DENABLE_ULM=${ENABLE_ULM} \ > + -DSHMEM_MANAGER=${SHMEM_MANAGER} \ > -DBUILD_OS=SYS_BIOS \ > -DIPC_INSTALL_PATH=${IPC_INSTALL_DIR}/packages \ > -DXDC_INSTALL_PATH=${XDC_INSTALL_DIR}/packages \ > diff --git a/meta-arago-extras/recipes-ti/ocl/files/0001-OpenCL_K2x_compile.patch b/meta-arago-extras/recipes-ti/ocl/files/0001-OpenCL_K2x_compile.patch > deleted file mode 100644 > index 0098d321..00000000 > --- a/meta-arago-extras/recipes-ti/ocl/files/0001-OpenCL_K2x_compile.patch > +++ /dev/null > @@ -1,12 +0,0 @@ > -diff --git a/host/src/core/dsp/tal/devmem_map_policy_mpm.cpp b/host/src/core/dsp/tal/devmem_map_policy_mpm.cpp > -index e5da9cc..182e5e3 100644 > ---- a/host/src/core/dsp/tal/devmem_map_policy_mpm.cpp > -+++ b/host/src/core/dsp/tal/devmem_map_policy_mpm.cpp > -@@ -91,6 +91,7 @@ void* DevMemMapPolicyMPM::Map (DSPDevicePtr64 dsp_addr, uint32_t size) const > - else > - ReportError(ErrorType::Fatal, > - ErrorKind::TranslateAddressOutsideMappedAddressRange); > -+ return 0; > - } > - > - void DevMemMapPolicyMPM::Unmap(void* host_addr, uint32_t size) const > diff --git a/meta-arago-extras/recipes-ti/ocl/files/0002-OpenCL_l2_cache_size.patch b/meta-arago-extras/recipes-ti/ocl/files/0002-OpenCL_l2_cache_size.patch > deleted file mode 100644 > index 7a81a137..00000000 > --- a/meta-arago-extras/recipes-ti/ocl/files/0002-OpenCL_l2_cache_size.patch > +++ /dev/null > @@ -1,49 +0,0 @@ > -diff --git a/monitor/src/builtins.c b/monitor/src/builtins.c > -index fbf3bf3..4edf61c 100644 > ---- a/monitor/src/builtins.c > -+++ b/monitor/src/builtins.c > -@@ -159,7 +159,7 @@ EXPORT int __cache_l2_none() > - EXPORT int __cache_l2_32k() > - { > - int32_t scratch_delta = __cache_l2_size() - (32 << 10); > -- uint32_t scratch_size = kernel_config_l2.L2_scratch_size; > -+ int32_t scratch_size = kernel_config_l2.L2_scratch_size; > - if (-scratch_delta > scratch_size) return 0; > - kernel_config_l2.L2_scratch_size += scratch_delta; > - > -@@ -173,7 +173,7 @@ EXPORT int __cache_l2_32k() > - EXPORT int __cache_l2_64k() > - { > - int32_t scratch_delta = __cache_l2_size() - (64 << 10); > -- uint32_t scratch_size = kernel_config_l2.L2_scratch_size; > -+ int32_t scratch_size = kernel_config_l2.L2_scratch_size; > - if (-scratch_delta > scratch_size) return 0; > - kernel_config_l2.L2_scratch_size += scratch_delta; > - > -@@ -187,7 +187,7 @@ EXPORT int __cache_l2_64k() > - EXPORT int __cache_l2_128k() > - { > - int32_t scratch_delta = __cache_l2_size() - (128 << 10); > -- uint32_t scratch_size = kernel_config_l2.L2_scratch_size; > -+ int32_t scratch_size = kernel_config_l2.L2_scratch_size; > - if (-scratch_delta > scratch_size) return 0; > - kernel_config_l2.L2_scratch_size += scratch_delta; > - > -@@ -205,7 +205,7 @@ EXPORT int __cache_l2_512k() { return 0; } > - EXPORT int __cache_l2_256k() > - { > - int32_t scratch_delta = __cache_l2_size() - (256 << 10); > -- uint32_t scratch_size = kernel_config_l2.L2_scratch_size; > -+ int32_t scratch_size = kernel_config_l2.L2_scratch_size; > - if (-scratch_delta > scratch_size) return 0; > - kernel_config_l2.L2_scratch_size += scratch_delta; > - > -@@ -219,7 +219,7 @@ EXPORT int __cache_l2_256k() > - EXPORT int __cache_l2_512k() > - { > - int32_t scratch_delta = __cache_l2_size() - (512 << 10); > -- uint32_t scratch_size = kernel_config_l2.L2_scratch_size; > -+ int32_t scratch_size = kernel_config_l2.L2_scratch_size; > - if (-scratch_delta > scratch_size) return 0; > - kernel_config_l2.L2_scratch_size += scratch_delta; > - > diff --git a/meta-arago-extras/recipes-ti/ocl/files/0003-Fix-g-8.3.0-OpenCL-example-undefined-behavior.patch b/meta-arago-extras/recipes-ti/ocl/files/0003-Fix-g-8.3.0-OpenCL-example-undefined-behavior.patch > deleted file mode 100644 > index b2c40c84..00000000 > --- a/meta-arago-extras/recipes-ti/ocl/files/0003-Fix-g-8.3.0-OpenCL-example-undefined-behavior.patch > +++ /dev/null > @@ -1,68 +0,0 @@ > -From 7cf77b704544b6012105f7ed19807b798b6cb242 Mon Sep 17 00:00:00 2001 > -From: Yuan Zhao > -Date: Mon, 24 Jun 2019 13:24:11 -0500 > -Subject: [PATCH] Fix g++ 8.3.0 OpenCL example undefined behavior > - > -- Fix undefined behavior of non-void functions with no return statements, > - when using g++ 8.3.0. Change them to void functions. > - > -Upstream-Status: Submitted [Remove this patch when next release is out] > - > -Signed-off-by: Yuan Zhao > ---- > - examples/persistent_common/host_assist.cpp | 2 +- > - examples/persistent_common/host_assist.h | 2 +- > - examples/sgemm/main.cpp | 4 ++-- > - 3 files changed, 4 insertions(+), 4 deletions(-) > - > -diff --git a/examples/persistent_common/host_assist.cpp b/examples/persistent_common/host_assist.cpp > -index f1640bc..fd8bb18 100644 > ---- a/examples/persistent_common/host_assist.cpp > -+++ b/examples/persistent_common/host_assist.cpp > -@@ -34,7 +34,7 @@ void print_completion_code(uint32_t completion_code) > - /****************************************************************************** > - * Given an OpenCL Device, Abort if the device is not part of the AM57x platform > - ******************************************************************************/ > --int assert_am57x(cl::Device &device) > -+void assert_am57x(cl::Device &device) > - { > - cl::Platform platform; > - std::string platform_name; > -diff --git a/examples/persistent_common/host_assist.h b/examples/persistent_common/host_assist.h > -index dec381e..4abf837 100644 > ---- a/examples/persistent_common/host_assist.h > -+++ b/examples/persistent_common/host_assist.h > -@@ -7,7 +7,7 @@ > - /*----------------------------------------------------------------------------- > - * Prototypes > - *----------------------------------------------------------------------------*/ > --int assert_am57x (cl::Device &device); > -+void assert_am57x (cl::Device &device); > - void print_completion_code(uint32_t completion_code); > - > - #endif // _HOST_ASSIST_H_ > -diff --git a/examples/sgemm/main.cpp b/examples/sgemm/main.cpp > -index 85f19ac..dfac651 100644 > ---- a/examples/sgemm/main.cpp > -+++ b/examples/sgemm/main.cpp > -@@ -96,7 +96,7 @@ int NUMCOMPUNITS = 0; > - /* ======================================================================== */ > - void PrintUsageAndExit(); > - void HandleOptions(int argc, char* argv[]); > --bool SetSgemmParams(Device& device); > -+void SetSgemmParams(Device& device); > - > - /* ======================================================================== */ > - /* MAIN */ > -@@ -428,7 +428,7 @@ static cl_ulong roundDownPower2(cl_ulong value) > - /*----------------------------------------------------------------------------- > - * Check platform name, set sgemm blocking/tiling parameters accordingly > - *----------------------------------------------------------------------------*/ > --bool SetSgemmParams(Device& device) > -+void SetSgemmParams(Device& device) > - { > - int APanelSz = 8 << 10; > - int BPanelSz = 16 << 10; > --- > -2.17.1 > - > diff --git a/meta-arago-extras/recipes-ti/ocl/ocl.inc b/meta-arago-extras/recipes-ti/ocl/ocl.inc > index 21247177..f6775299 100644 > --- a/meta-arago-extras/recipes-ti/ocl/ocl.inc > +++ b/meta-arago-extras/recipes-ti/ocl/ocl.inc > @@ -1,5 +1,5 @@ > # patch version at the end needs to be in double digits > -PV = "1.1.18.00" > +PV = "1.1.19.00" > INC_PR = "r0" > > LIC_FILES_CHKSUM = "file://../debian/copyright;md5=2e3965a73a8a49c23836467266120dff" > @@ -10,7 +10,8 @@ OCL_GIT_URI = "git://git.ti.com/opencl/ti-opencl.git" > OCL_GIT_PROTOCOL = "git" > OCL_GIT_BRANCH = "master" > > -OCL_SRCREV = "a50d8b41cc30bdf4faf1998ec3fc7d0a3093b795" > + > +OCL_SRCREV = "0f4be82079ba6ec9c7c8ee00f879c9f6f0ad4803" > > BRANCH = "${OCL_GIT_BRANCH}" > SRC_URI = "${OCL_GIT_URI};protocol=${OCL_GIT_PROTOCOL};branch=${BRANCH}" > diff --git a/meta-arago-extras/recipes-ti/ocl/opencl-examples_git.bb b/meta-arago-extras/recipes-ti/ocl/opencl-examples_git.bb > index c562bbf7..4732c73d 100644 > --- a/meta-arago-extras/recipes-ti/ocl/opencl-examples_git.bb > +++ b/meta-arago-extras/recipes-ti/ocl/opencl-examples_git.bb > @@ -5,9 +5,7 @@ LICENSE = "BSD" > include ocl.inc > require recipes-ti/includes/ti-paths.inc > > -SRC_URI += "file://0003-Fix-g-8.3.0-OpenCL-example-undefined-behavior.patch;patchdir=.." > - > -PR = "${INC_PR}.1" > +PR = "${INC_PR}.0" > > COMPATIBLE_MACHINE = "dra7xx|keystone" > PACKAGE_ARCH = "${MACHINE_ARCH}" > diff --git a/meta-arago-extras/recipes-ti/ocl/opencl-monitor_git.bb b/meta-arago-extras/recipes-ti/ocl/opencl-monitor_git.bb > index 00aa773c..0ab21a1a 100644 > --- a/meta-arago-extras/recipes-ti/ocl/opencl-monitor_git.bb > +++ b/meta-arago-extras/recipes-ti/ocl/opencl-monitor_git.bb > @@ -5,9 +5,7 @@ LICENSE = "BSD" > include ocl.inc > require recipes-ti/includes/arago-paths.inc > > -SRC_URI += "file://0002-OpenCL_l2_cache_size.patch;patchdir=${WORKDIR}/git" > - > -PR = "${INC_PR}.1" > +PR = "${INC_PR}.0" > > inherit update-alternatives > > diff --git a/meta-arago-extras/recipes-ti/ocl/opencl-tidl-fw_git.bb b/meta-arago-extras/recipes-ti/ocl/opencl-tidl-fw_git.bb > index a9ec413a..64dfd0d7 100644 > --- a/meta-arago-extras/recipes-ti/ocl/opencl-tidl-fw_git.bb > +++ b/meta-arago-extras/recipes-ti/ocl/opencl-tidl-fw_git.bb > @@ -3,7 +3,7 @@ SUMMARY = "OpenCL TIDL firmware for AM57xx" > LICENSE = "TI-TFL" > LIC_FILES_CHKSUM = "file://LICENSE.ti;md5=082a028431c455252c1e1d3d1021d382" > > -PV = "01.01.18.00" > +PV = "01.01.19.00" > PR = "r0" > > require recipes-ti/includes/arago-paths.inc > @@ -15,7 +15,7 @@ PACKAGE_ARCH = "${MACHINE_ARCH}" > GIT_URI = "git://git.ti.com/opencl/opencl-firmware.git" > GIT_PROTOCOL = "git" > BRANCH = "master" > -SRCREV = "3b98f405d98416a66678dec01d8f7ef244f379b6" > +SRCREV = "32772d41ae6978b0ec66173d2bfc8c041017bda1" > > SRC_URI = "${GIT_URI};protocol=${GIT_PROTOCOL};branch=${BRANCH}" > > diff --git a/meta-arago-extras/recipes-ti/ocl/opencl_git.bb b/meta-arago-extras/recipes-ti/ocl/opencl_git.bb > index 8b363573..69f93309 100644 > --- a/meta-arago-extras/recipes-ti/ocl/opencl_git.bb > +++ b/meta-arago-extras/recipes-ti/ocl/opencl_git.bb > @@ -3,9 +3,8 @@ HOMEPAGE = "https://downloads.ti.com/mctools/esd/docs/opencl/index.html" > LICENSE = "BSD" > > include ocl.inc > -SRC_URI += "file://0001-OpenCL_K2x_compile.patch;patchdir=${WORKDIR}/git" > > -PR = "${INC_PR}.1" > +PR = "${INC_PR}.0" > > inherit cmake systemd > > @@ -59,8 +58,9 @@ OCL_BUILD_TARGET_k2g = "ARM_K2G" > > ENABLE_ULM = "1" > ENABLE_ULM_k2g = "0" > +SHMEM_MANAGER = "CMEM" > > -EXTRA_OECMAKE += " -DBUILD_TARGET=${OCL_BUILD_TARGET} -DBUILD_OUTPUT=lib -DENABLE_ULM=${ENABLE_ULM} -DOCL_VERSION=${PV}" > +EXTRA_OECMAKE += " -DBUILD_TARGET=${OCL_BUILD_TARGET} -DBUILD_OUTPUT=lib -DENABLE_ULM=${ENABLE_ULM} -DOCL_VERSION=${PV} -DSHMEM_MANAGER=${SHMEM_MANAGER}" > > EXTRA_OEMAKE += "KERNEL_INSTALL_DIR=${STAGING_KERNEL_DIR} LINUX_DEVKIT_ROOT=${STAGING_DIR_HOST}" > export KERNEL_INSTALL_DIR = "${STAGING_KERNEL_DIR}" > -- > 2.14.1 > > _______________________________________________ > meta-arago mailing list > meta-arago@arago-project.org > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago