Re: [oe] [meta-oe][PATCH 1/1 v2] microcode-ctl: add new recipe

2017-06-15 Thread Huang, Jie (Jackie)
> 
> 
> From: Burton, Ross [mailto:ross.bur...@intel.com] 
> Sent: Thursday, June 15, 2017 19:41
> To: Huang, Jie (Jackie)
> Cc: OpenEmbedded Devel List
> Subject: Re: [oe] [meta-oe][PATCH 1/1 v2] microcode-ctl: add new recipe
> 
> 
> On 15 June 2017 at 02:36,  wrote:
> The microcode_ctl utility is a companion to the
> IA32 microcode driver.
> 
> Wouldn't it make sense for this to be in meta-intel, alongside the microcode? 
> Also what's the difference between this and iucode-tool, which is already in 
> meta-intel?

Sorry I didn't use meta-intel and not aware of that, and yes, it make senses to 
be in meta-intel, I will investigate the difference to see if this still needed.

Thanks,
Jackie

> 
> Ross
>
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH 1/1 v2] microcode-ctl: add new recipe

2017-06-15 Thread Burton, Ross
On 15 June 2017 at 02:36,  wrote:

> The microcode_ctl utility is a companion to the
> IA32 microcode driver.
>

Wouldn't it make sense for this to be in meta-intel, alongside the
microcode? Also what's the difference between this and iucode-tool, which
is already in meta-intel?

Ross
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][PATCH 1/1 v2] microcode-ctl: add new recipe

2017-06-14 Thread jackie.huang
From: Jackie Huang 

The microcode_ctl utility is a companion to the
IA32 microcode driver.

Signed-off-by: Jackie Huang 
---
 meta-oe/licenses/Intel-ucode   | 30 ++
 .../0001-add-support-for-other-distributions.patch | 27 +
 .../files/0001-fix-the-help-return-code.patch  | 57 +++
 .../files/fix-No-GNU_HASH-in-the-elf-binary.patch  | 30 ++
 .../microcode-ctl/files/microcode_ctl.service  | 11 
 .../recipes-bsp/microcode-ctl/microcode-ctl_git.bb | 65 ++
 6 files changed, 220 insertions(+)
 create mode 100644 meta-oe/licenses/Intel-ucode
 create mode 100644 
meta-oe/recipes-bsp/microcode-ctl/files/0001-add-support-for-other-distributions.patch
 create mode 100644 
meta-oe/recipes-bsp/microcode-ctl/files/0001-fix-the-help-return-code.patch
 create mode 100644 
meta-oe/recipes-bsp/microcode-ctl/files/fix-No-GNU_HASH-in-the-elf-binary.patch
 create mode 100644 
meta-oe/recipes-bsp/microcode-ctl/files/microcode_ctl.service
 create mode 100755 meta-oe/recipes-bsp/microcode-ctl/microcode-ctl_git.bb

diff --git a/meta-oe/licenses/Intel-ucode b/meta-oe/licenses/Intel-ucode
new file mode 100644
index 0..964953bb9
--- /dev/null
+++ b/meta-oe/licenses/Intel-ucode
@@ -0,0 +1,30 @@
+Copyright (c) <1995-2017>, Intel Corporation.
+All rights reserved.
+
+Redistribution. Redistribution and use in binary form, without modification, 
are
+permitted provided that the following conditions are met:
+.Redistributions must reproduce the above copyright notice and the 
following
+disclaimer in the documentation and/or other materials provided with the
+distribution.
+.Neither the name of Intel Corporation nor the names of its suppliers may 
be used
+to endorse or promote products derived from this software without specific 
prior
+written permission.
+.No reverse engineering, decompilation, or disassembly of this software is
+permitted.
+."Binary form" includes any format commonly used for electronic conveyance
+which is a reversible, bit-exact translation of binary representation to ASCII 
or
+ISO text, for example, "uuencode."
+
+DISCLAIMER. THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT
+HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED
+WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER
+OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
+ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
diff --git 
a/meta-oe/recipes-bsp/microcode-ctl/files/0001-add-support-for-other-distributions.patch
 
b/meta-oe/recipes-bsp/microcode-ctl/files/0001-add-support-for-other-distributions.patch
new file mode 100644
index 0..fe5428493
--- /dev/null
+++ 
b/meta-oe/recipes-bsp/microcode-ctl/files/0001-add-support-for-other-distributions.patch
@@ -0,0 +1,27 @@
+Subject: [PATCH] add support for other distributions
+
+Upstream-Status: Inappropriate [OE specific]
+
+Signed-off-by: Jackie Huang 
+---
+ microcode_ctl.start | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/microcode_ctl.start b/microcode_ctl.start
+index 6fa9d72..e9321d0 100755
+--- a/microcode_ctl.start
 b/microcode_ctl.start
+@@ -47,8 +47,8 @@ elif [ -f /etc/slackware-version ]; then
+   END=slack_end
+   fi
+ else
+-  echo "$0: Can't guess distribution, aborting!"
+-  exit 1
++  START=
++  END=
+ fi
+ 
+ # Lets just be sure we have a device file...
+-- 
+2.11.0
+
diff --git 
a/meta-oe/recipes-bsp/microcode-ctl/files/0001-fix-the-help-return-code.patch 
b/meta-oe/recipes-bsp/microcode-ctl/files/0001-fix-the-help-return-code.patch
new file mode 100644
index 0..cbe7755e9
--- /dev/null
+++ 
b/meta-oe/recipes-bsp/microcode-ctl/files/0001-fix-the-help-return-code.patch
@@ -0,0 +1,57 @@
+[PATCH] fix the help return code
+
+Upstream-Status: Pending
+
+not always return 1 when call usage(), differ the return code by user action;
+if a use wants to get help information, it is normal action, should be
+return 0;
+if input parameter is wrong, and microcode_ctl prompts the help information,
+it should be return 1;
+
+Signed-off-by: Roy Li 
+---
+ microcode_ctl.c | 7 +--
+ 1 file changed, 5 insertions(+), 2 deletions(-)
+
+diff --git a/microcode_ctl.c b/microcode_ctl.c
+index 3f9c151..cdcdc10 100644
+--- a/microcode_ctl.c
 b/microcode_ctl.c
+@@ -40,7 +40,6 @@ static void usage(void)
+   "  -u   upload microcode (default filename:\"%s\"\n"
+   "  -f   upload