Discussion:
[meta-arago] [PATCH] matrix-gui: sleep 1 second prior to weston shutdown
Eric Ruei
2018-11-07 21:05:10 UTC
Permalink
Update execute_command.sh:
Wait 1 second to make sure the application freeing all SGX
resources prior to weston shutdown


Signed-off-by: Eric Ruei <e-***@ti.com>
---
...e_command-workaround-for-GUI-apps-with-weston.patch | 18 ++++++++++--------
.../recipes-core/matrix/matrix-gui_2.0.bb | 2 +-
2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch b/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
index 5a85935..90a604e 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
@@ -1,6 +1,6 @@
-From 9647ddc10dc9424b1433db2f8184268523d829da Mon Sep 17 00:00:00 2001
-From: Eric Ruei <e-***@ti.com>
-Date: Wed, 28 Feb 2018 12:20:32 -0500
+From 156ce8d8d9ee1b614b0079e1e9506af1835bdab7 Mon Sep 17 00:00:00 2001
+From: Jacob Stiffler <j-***@ti.com>
+Date: Wed, 7 Nov 2018 15:48:47 -0500
Subject: [PATCH] execute_command: workaround for GUI apps with weston

* For all GUI apps, shutdown matrix before launching
@@ -8,17 +8,18 @@ Subject: [PATCH] execute_command: workaround for GUI apps with weston
GUI app may get hidden behind matrix with no way to recover.
* For MM and 3D and null window apps, shutdown weston before launching
- These apps use KMS sink or DRM, which collides with weston
-
+ - Wait 1 second to make sure the application freeing all SGX resources
+ prior to weston shutdown
Signed-off-by: Jacob Stiffler <j-***@ti.com>
Signed-off-by: Eric Ruei <e-***@ti.com>
---
- execute_command.sh | 39 ++++++++++++++++++++++++++++++++++++++-
+ execute_command.sh | 40 +++++++++++++++++++++++++++++++++++++++-
generate.php | 4 ++++
run_script.php | 10 +++++++++-
- 3 files changed, 51 insertions(+), 2 deletions(-)
+ 3 files changed, 52 insertions(+), 2 deletions(-)

diff --git a/execute_command.sh b/execute_command.sh
-index 325dcb9..180e71f 100755
+index 325dcb9..b8bb6da 100755
--- a/execute_command.sh
+++ b/execute_command.sh
@@ -63,7 +63,22 @@ do
@@ -45,7 +46,7 @@ index 325dcb9..180e71f 100755
then
touch "lock/"$item

-@@ -72,10 +87,32 @@ do
+@@ -72,10 +87,33 @@ do

done

@@ -55,6 +56,7 @@ index 325dcb9..180e71f 100755
+
+ if [ $category == 3d -o $category == multimedia -o $category == touch -o $win_type == null ]
+ then
++ sleep 1
+ /etc/init.d/weston stop
+ sleep 1
+ fi
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
index d8aa00b..4e335e7 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
@@ -6,7 +6,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=a886c9ef769b2d8271115d2502512e5d"

SECTION = "multimedia"

-PR = "r35"
+PR = "r36"

INITSCRIPT_NAME = "matrix-gui-2.0"
INITSCRIPT_PARAMS = "defaults 97"
--
1.9.1
Eric Ruei
2018-11-07 23:01:51 UTC
Permalink
Update execute_command.sh:
Wait 1 second to make sure the application freeing all SGX
resources prior to weston shutdown

Signed-off-by: Eric Ruei <e-***@ti.com>
---
...e_command-workaround-for-GUI-apps-with-weston.patch | 18 +++++++++++-------
recipes-core/matrix/matrix-gui_2.0.bbappend | 2 +-
2 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch b/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
index f5399a0..2ddfed2 100644
--- a/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
+++ b/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
@@ -1,6 +1,6 @@
-From 5104914897db7ebad07d5c82302ad176bd84f4fe Mon Sep 17 00:00:00 2001
-From: Jacob Stiffler <j-***@ti.com>
-Date: Mon, 16 Jul 2018 15:52:45 -0400
+From c8f2849adf7e02aa55ee70103c188b9dbf9b6f80 Mon Sep 17 00:00:00 2001
+From: Eric Ruei <e-***@ti.com>
+Date: Wed, 7 Nov 2018 14:55:35 -0500
Subject: [PATCH] execute_command: workaround for GUI apps with weston

* For all GUI apps, shutdown matrix before launching
@@ -8,18 +8,20 @@ Subject: [PATCH] execute_command: workaround for GUI apps with weston
GUI app may get hidden behind matrix with no way to recover.
* For MM and 3D and null window apps, shutdown weston before launching
- These apps use KMS sink or DRM, which collides with weston
+ - Wait 1 second to make sure the application freeing all SGX resources
+ prior to weston shutdown
* For qwindow window apps, use qwindow-compositor instead of weston

Signed-off-by: Jacob Stiffler <j-***@ti.com>
Signed-off-by: Eric Ruei <e-***@ti.com>
---
- execute_command.sh | 51 ++++++++++++++++++++++++++++++++++++++++++++++++++-
+ execute_command.sh | 53 ++++++++++++++++++++++++++++++++++++++++++++++++++++-
generate.php | 4 ++++
run_script.php | 10 +++++++++-
- 3 files changed, 63 insertions(+), 2 deletions(-)
+ 3 files changed, 65 insertions(+), 2 deletions(-)

diff --git a/execute_command.sh b/execute_command.sh
-index 325dcb9..9769f4f 100755
+index 325dcb9..ffc347d 100755
--- a/execute_command.sh
+++ b/execute_command.sh
@@ -63,7 +63,22 @@ do
@@ -46,7 +48,7 @@ index 325dcb9..9769f4f 100755
then
touch "lock/"$item

-@@ -72,10 +87,44 @@ do
+@@ -72,10 +87,46 @@ do

done

@@ -56,10 +58,12 @@ index 325dcb9..9769f4f 100755
+
+ if [ $category == 3d -o $category == multimedia -o $category == touch -o $win_type == null ]
+ then
++ sleep 1
+ /etc/init.d/weston stop
+ sleep 1
+ elif [ $win_type == qwindow ]
+ then
++ sleep 1
+ /etc/init.d/weston stop
+ sleep 1
+ /etc/init.d/qwindow start
diff --git a/recipes-core/matrix/matrix-gui_2.0.bbappend b/recipes-core/matrix/matrix-gui_2.0.bbappend
index 8626d1f..112a4a6 100644
--- a/recipes-core/matrix/matrix-gui_2.0.bbappend
+++ b/recipes-core/matrix/matrix-gui_2.0.bbappend
@@ -1,4 +1,4 @@
-PR_append = ".tisdk0"
+PR_append = ".tisdk1"

FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
--
1.9.1
Ruei, Eric
2018-11-08 14:39:10 UTC
Permalink
NAK

Please ignore this one! I got a better solution.

Eric


-----Original Message-----
From: meta-arago-***@arago-project.org [mailto:meta-arago-***@arago-project.org] On Behalf Of Ruei, Eric
Sent: Wednesday, November 07, 2018 6:02 PM
To: meta-***@arago-project.org
Subject: [EXTERNAL] [meta-arago] [meta-processor-sdk][PATCH] matrix-gui: sleep 1 second prior to weston shutdown

Update execute_command.sh:
Wait 1 second to make sure the application freeing all SGX
resources prior to weston shutdown

Signed-off-by: Eric Ruei <e-***@ti.com>
---
...e_command-workaround-for-GUI-apps-with-weston.patch | 18 +++++++++++-------
recipes-core/matrix/matrix-gui_2.0.bbappend | 2 +-
2 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch b/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
index f5399a0..2ddfed2 100644
--- a/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
+++ b/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
@@ -1,6 +1,6 @@
-From 5104914897db7ebad07d5c82302ad176bd84f4fe Mon Sep 17 00:00:00 2001
-From: Jacob Stiffler <j-***@ti.com>
-Date: Mon, 16 Jul 2018 15:52:45 -0400
+From c8f2849adf7e02aa55ee70103c188b9dbf9b6f80 Mon Sep 17 00:00:00 2001
+From: Eric Ruei <e-***@ti.com>
+Date: Wed, 7 Nov 2018 14:55:35 -0500
Subject: [PATCH] execute_command: workaround for GUI apps with weston

* For all GUI apps, shutdown matrix before launching
@@ -8,18 +8,20 @@ Subject: [PATCH] execute_command: workaround for GUI apps with weston
GUI app may get hidden behind matrix with no way to recover.
* For MM and 3D and null window apps, shutdown weston before launching
- These apps use KMS sink or DRM, which collides with weston
+ - Wait 1 second to make sure the application freeing all SGX resources
+ prior to weston shutdown
* For qwindow window apps, use qwindow-compositor instead of weston

Signed-off-by: Jacob Stiffler <j-***@ti.com>
Signed-off-by: Eric Ruei <e-***@ti.com>
---
- execute_command.sh | 51 ++++++++++++++++++++++++++++++++++++++++++++++++++-
+ execute_command.sh | 53 ++++++++++++++++++++++++++++++++++++++++++++++++++++-
generate.php | 4 ++++
run_script.php | 10 +++++++++-
- 3 files changed, 63 insertions(+), 2 deletions(-)
+ 3 files changed, 65 insertions(+), 2 deletions(-)

diff --git a/execute_command.sh b/execute_command.sh
-index 325dcb9..9769f4f 100755
+index 325dcb9..ffc347d 100755
--- a/execute_command.sh
+++ b/execute_command.sh
@@ -63,7 +63,22 @@ do
@@ -46,7 +48,7 @@ index 325dcb9..9769f4f 100755
then
touch "lock/"$item

-@@ -72,10 +87,44 @@ do
+@@ -72,10 +87,46 @@ do

done

@@ -56,10 +58,12 @@ index 325dcb9..9769f4f 100755
+
+ if [ $category == 3d -o $category == multimedia -o $category == touch -o $win_type == null ]
+ then
++ sleep 1
+ /etc/init.d/weston stop
+ sleep 1
+ elif [ $win_type == qwindow ]
+ then
++ sleep 1
+ /etc/init.d/weston stop
+ sleep 1
+ /etc/init.d/qwindow start
diff --git a/recipes-core/matrix/matrix-gui_2.0.bbappend b/recipes-core/matrix/matrix-gui_2.0.bbappend
index 8626d1f..112a4a6 100644
--- a/recipes-core/matrix/matrix-gui_2.0.bbappend
+++ b/recipes-core/matrix/matrix-gui_2.0.bbappend
@@ -1,4 +1,4 @@
-PR_append = ".tisdk0"
+PR_append = ".tisdk1"

FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
--
1.9.1
Ruei, Eric
2018-11-08 14:40:09 UTC
Permalink
NAK

Please ignore this one. There is a better solution!

Best regards,

Eric


-----Original Message-----
From: meta-arago-***@arago-project.org [mailto:meta-arago-***@arago-project.org] On Behalf Of Ruei, Eric
Sent: Wednesday, November 07, 2018 4:05 PM
To: meta-***@arago-project.org
Subject: [EXTERNAL] [meta-arago] [PATCH] matrix-gui: sleep 1 second prior to weston shutdown

Update execute_command.sh:
Wait 1 second to make sure the application freeing all SGX
resources prior to weston shutdown


Signed-off-by: Eric Ruei <e-***@ti.com>
---
...e_command-workaround-for-GUI-apps-with-weston.patch | 18 ++++++++++--------
.../recipes-core/matrix/matrix-gui_2.0.bb | 2 +-
2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch b/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
index 5a85935..90a604e 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
@@ -1,6 +1,6 @@
-From 9647ddc10dc9424b1433db2f8184268523d829da Mon Sep 17 00:00:00 2001
-From: Eric Ruei <e-***@ti.com>
-Date: Wed, 28 Feb 2018 12:20:32 -0500
+From 156ce8d8d9ee1b614b0079e1e9506af1835bdab7 Mon Sep 17 00:00:00 2001
+From: Jacob Stiffler <j-***@ti.com>
+Date: Wed, 7 Nov 2018 15:48:47 -0500
Subject: [PATCH] execute_command: workaround for GUI apps with weston

* For all GUI apps, shutdown matrix before launching
@@ -8,17 +8,18 @@ Subject: [PATCH] execute_command: workaround for GUI apps with weston
GUI app may get hidden behind matrix with no way to recover.
* For MM and 3D and null window apps, shutdown weston before launching
- These apps use KMS sink or DRM, which collides with weston
-
+ - Wait 1 second to make sure the application freeing all SGX resources
+ prior to weston shutdown
Signed-off-by: Jacob Stiffler <j-***@ti.com>
Signed-off-by: Eric Ruei <e-***@ti.com>
---
- execute_command.sh | 39 ++++++++++++++++++++++++++++++++++++++-
+ execute_command.sh | 40 +++++++++++++++++++++++++++++++++++++++-
generate.php | 4 ++++
run_script.php | 10 +++++++++-
- 3 files changed, 51 insertions(+), 2 deletions(-)
+ 3 files changed, 52 insertions(+), 2 deletions(-)

diff --git a/execute_command.sh b/execute_command.sh
-index 325dcb9..180e71f 100755
+index 325dcb9..b8bb6da 100755
--- a/execute_command.sh
+++ b/execute_command.sh
@@ -63,7 +63,22 @@ do
@@ -45,7 +46,7 @@ index 325dcb9..180e71f 100755
then
touch "lock/"$item

-@@ -72,10 +87,32 @@ do
+@@ -72,10 +87,33 @@ do

done

@@ -55,6 +56,7 @@ index 325dcb9..180e71f 100755
+
+ if [ $category == 3d -o $category == multimedia -o $category == touch -o $win_type == null ]
+ then
++ sleep 1
+ /etc/init.d/weston stop
+ sleep 1
+ fi
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
index d8aa00b..4e335e7 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
@@ -6,7 +6,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=a886c9ef769b2d8271115d2502512e5d"

SECTION = "multimedia"

-PR = "r35"
+PR = "r36"

INITSCRIPT_NAME = "matrix-gui-2.0"
INITSCRIPT_PARAMS = "defaults 97"
--
1.9.1
Denys Dmytriyenko
2018-11-08 15:57:47 UTC
Permalink
http://arago-project.org/git/?p=meta-arago.git;a=commitdiff;h=f026e412d763889250be2b99b08f5f5c86338f19
Post by Ruei, Eric
NAK
Please ignore this one. There is a better solution!
Best regards,
Eric
-----Original Message-----
Sent: Wednesday, November 07, 2018 4:05 PM
Subject: [EXTERNAL] [meta-arago] [PATCH] matrix-gui: sleep 1 second prior to weston shutdown
Wait 1 second to make sure the application freeing all SGX
resources prior to weston shutdown
---
...e_command-workaround-for-GUI-apps-with-weston.patch | 18 ++++++++++--------
.../recipes-core/matrix/matrix-gui_2.0.bb | 2 +-
2 files changed, 11 insertions(+), 9 deletions(-)
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch b/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
index 5a85935..90a604e 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui/0001-execute_command-workaround-for-GUI-apps-with-weston.patch
@@ -1,6 +1,6 @@
-From 9647ddc10dc9424b1433db2f8184268523d829da Mon Sep 17 00:00:00 2001
-Date: Wed, 28 Feb 2018 12:20:32 -0500
+From 156ce8d8d9ee1b614b0079e1e9506af1835bdab7 Mon Sep 17 00:00:00 2001
+Date: Wed, 7 Nov 2018 15:48:47 -0500
Subject: [PATCH] execute_command: workaround for GUI apps with weston
* For all GUI apps, shutdown matrix before launching
@@ -8,17 +8,18 @@ Subject: [PATCH] execute_command: workaround for GUI apps with weston
GUI app may get hidden behind matrix with no way to recover.
* For MM and 3D and null window apps, shutdown weston before launching
- These apps use KMS sink or DRM, which collides with weston
-
+ - Wait 1 second to make sure the application freeing all SGX resources
+ prior to weston shutdown
---
- execute_command.sh | 39 ++++++++++++++++++++++++++++++++++++++-
+ execute_command.sh | 40 +++++++++++++++++++++++++++++++++++++++-
generate.php | 4 ++++
run_script.php | 10 +++++++++-
- 3 files changed, 51 insertions(+), 2 deletions(-)
+ 3 files changed, 52 insertions(+), 2 deletions(-)
diff --git a/execute_command.sh b/execute_command.sh
-index 325dcb9..180e71f 100755
+index 325dcb9..b8bb6da 100755
--- a/execute_command.sh
+++ b/execute_command.sh
@@ -63,7 +63,22 @@ do
@@ -45,7 +46,7 @@ index 325dcb9..180e71f 100755
then
touch "lock/"$item
done
@@ -55,6 +56,7 @@ index 325dcb9..180e71f 100755
+
+ if [ $category == 3d -o $category == multimedia -o $category == touch -o $win_type == null ]
+ then
++ sleep 1
+ /etc/init.d/weston stop
+ sleep 1
+ fi
diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
index d8aa00b..4e335e7 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui_2.0.bb
@@ -6,7 +6,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=a886c9ef769b2d8271115d2502512e5d"
SECTION = "multimedia"
-PR = "r35"
+PR = "r36"
INITSCRIPT_NAME = "matrix-gui-2.0"
INITSCRIPT_PARAMS = "defaults 97"
--
1.9.1
_______________________________________________
meta-arago mailing list
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
_______________________________________________
meta-arago mailing list
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
Loading...