All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ruei, Eric" <e-ruei1@ti.com>
To: "meta-arago@arago-project.org" <meta-arago@arago-project.org>
Subject: Re: [EXTERNAL] [PATCH] matrix-gui: sleep 1 second prior to weston shutdown
Date: Thu, 8 Nov 2018 14:40:09 +0000	[thread overview]
Message-ID: <64d6cb22d8a14a2f97f52d1ff8a2fbaf@ti.com> (raw)
In-Reply-To: <1541624710-59127-1-git-send-email-e-ruei1@ti.com>

NAK

Please ignore this one. There is a better solution!

Best regards,

Eric


-----Original Message-----
From: meta-arago-bounces@arago-project.org [mailto:meta-arago-bounces@arago-project.org] On Behalf Of Ruei, Eric
Sent: Wednesday, November 07, 2018 4:05 PM
To: meta-arago@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-ruei1@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-ruei1@ti.com>
-Date: Wed, 28 Feb 2018 12:20:32 -0500
+From 156ce8d8d9ee1b614b0079e1e9506af1835bdab7 Mon Sep 17 00:00:00 2001
+From: Jacob Stiffler <j-stiffler@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-stiffler@ti.com>
 Signed-off-by: Eric Ruei <e-ruei1@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

_______________________________________________
meta-arago mailing list
meta-arago@arago-project.org
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


  reply	other threads:[~2018-11-08 14:40 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-07 21:05 [PATCH] matrix-gui: sleep 1 second prior to weston shutdown Eric Ruei
2018-11-08 14:40 ` Ruei, Eric [this message]
2018-11-08 15:57   ` [EXTERNAL] " Denys Dmytriyenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=64d6cb22d8a14a2f97f52d1ff8a2fbaf@ti.com \
    --to=e-ruei1@ti.com \
    --cc=meta-arago@arago-project.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.