Re: [edk2] Doubt about IPMI

2013-08-23 Thread Neeraj Ladkani
Hi Rafael, if you want to just send one IPMI message and process the response, you don't want to implement whole IPMI protocol, frame the IPMI request send it over transport ( SSIF, KCS, IPMB ) using base protocols ( IO mapped , SMBUS) , and process the response. Thanks Neeraj Thanks Neeraj

Re: [edk2] Doubt about IPMI

2013-08-23 Thread Jeffrey Chou
Hi, I also have the experience about IPMI recently. You could look at the EFI Toolkit at http://sourceforge.net/apps/mediawiki/tianocore/index.php?title=EFI_Toolkit, there are some kcs protocol source codes have been implemented in the toolkit. Just see the folder of EFI_Toolkit_2.0\protocols\ipmi

[edk2] [PATCH 0/1] fix data corruption in SVN r14570

2013-08-23 Thread Laszlo Ersek
Dear MdeModulePkg maintainer, please review the 1/1 patch below. If you deem it worthy and decide to apply it, please consider my request here -- in fact this request is my only reason for writing the 0/1 cover letter (because a cover letter is normally not necessary for single-patch submissions)

[edk2] [PATCH 1/1] MdeModulePkg/DiskIoDxe: fix source/destination pointer of overrun transfer

2013-08-23 Thread Laszlo Ersek
DiskIoCreateSubtaskList() may split the transfer into three segments: - a leading segment, called underrun, which is the fractional, trailing subset of the first underlying block, - a middle segment, which is an integral multiple of underlying blocks, - a trailing segment, called overrun, which i

Re: [edk2] Doubt about IPMI

2013-08-23 Thread Andrew Fish
On Aug 23, 2013, at 11:24 AM, Andrew Fish wrote: > > On Aug 23, 2013, at 11:19 AM, Rafael Machado > wrote: > >> Exactly, and this can be done as suggested by Galla Rao right ? >> Using EFI_SMBUS_HC_PROTOCOL so that IPMI can use this protocol to >> communicate with BMC >> > Opps I see IPM

Re: [edk2] [PATCH] OvmfPkg: Virtio: load used ring element strictly after loading used index

2013-08-23 Thread Jordan Justen
Applied in r14601. Thanks for the contribution. On Mon, Aug 19, 2013 at 12:33 AM, Laszlo Ersek wrote: > Enforce in-order execution of these steps even on not sequentially > consistent architectures, as discussed in [1]. These changes should be > unnecessary on x86 (the only architecture OVMF curr

Re: [edk2] [PATCH] OvmfPkg NvVarsFileLib: Set NvVars variable after writing vars file

2013-08-23 Thread Laszlo Ersek
On 08/23/13 11:27, Michael Chang wrote: > On Thu, Aug 22, 2013 at 06:05:58PM -0700, Jordan Justen wrote: >> On Mon, Aug 19, 2013 at 3:00 AM, Michael Chang wrote: >>> On Sat, Aug 17, 2013 at 11:16:16PM -0700, Jordan Justen wrote: The volatile 'NvVars' variable indicates that the variables do >

Re: [edk2] Doubt about IPMI

2013-08-23 Thread Andrew Fish
On Aug 23, 2013, at 11:19 AM, Rafael Machado wrote: > Exactly, and this can be done as suggested by Galla Rao right ? > Using EFI_SMBUS_HC_PROTOCOL so that IPMI can use this protocol to communicate > with BMC > No that is talking on SMBUS. You just need to read/write an IO port IoRead32 (0x

Re: [edk2] Doubt about IPMI

2013-08-23 Thread Rafael Machado
Exactly, and this can be done as suggested by Galla Rao right ? Using EFI_SMBUS_HC_PROTOCOL so that IPMI can use this protocol to communicate with BMC I'm currectly working on a proposal to read some IPMI information at UEFI environment. PS.: I've never studied IPMI and maybe some of my questions

[edk2] [PATCH] ShellPkg: Fixed build errors related to variable declarations

2013-08-23 Thread Olivier Martin
Dear ShellPkg maintainer, Please find the attached patch that fixes the following build errors: - variable "TempLocation" was set but never used - FileInfo may be used before being set - variable "TempLocation" was set but never used Contributed-under: TianoCore Contribution Agreement 1.

Re: [edk2] [PATCH] comment copy/paste fix

2013-08-23 Thread Olivier Martin
Thanks Roy for the contribution. It looks good to me, but I can only accept patches if submitted under the 'TianoCore Contribution Agreement 1.0'. Have a look at the code contribution process: https://sourceforge.net/apps/mediawiki/tianocore/index.php?title=Code_Contri butions If you agree with t

[edk2] [PATCH] comment copy/paste fix

2013-08-23 Thread Roy Franz
Comments in ArmSetHighVectors and ArmSetLowVectors the same even though one is setting bit and one is clearing it. Signed-off-by: Roy Franz diff --git a/ArmPkg/Library/ArmLib/ArmV7/ArmV7Support.S b/ArmPkg/Library/ArmLib/ArmV7/ArmV7Support.S index 910e50e..71933bd 100644 --- a/ArmPkg/Library/ArmL

Re: [edk2] Doubt about IPMI

2013-08-23 Thread Andrew Fish
I thought you talked to the IMPI controller via a KCS interface that is mapped to I/O ports? I think the default base address is 0xca2. The platforms base address would be in ACPI tables. There is an open source Linux driver for IPMI, not sure if there is any BSD versions. Thanks, Andrew Fish

Re: [edk2] SVN r14570 breaks CD-ROM boot under OvmfPkg

2013-08-23 Thread Laszlo Ersek
On 08/23/13 16:38, Laszlo Ersek wrote: > I bisected a CD-ROM boot problem under OvmfPkg to this commit: > > Author: Ruiyu Ni > Date: Tue Aug 20 03:14:30 2013 + > > Add DiskIo2 protocol definition to MdePkg. > Change DiskIoDxe to produce DiskIo2 protocol when the BlockIo2 > pro

Re: [edk2] Doubt about IPMI

2013-08-23 Thread Rafael Machado
Hi Everyone What I need to do here is by IPMI command, access IMM PORT status Check. Actualy this is done with the ipmtool, but I'm trying to do this at uefi. As far as I could check it can be done using EFI_SMBUS_HC_PROTOCOL, as suggested by galla rao, with the correct command calling the fu

Re: [edk2] [edk2-buildtools] [PATCH] Fixed calculation of BaseOfCode in GenFw when the first code section is aligned

2013-08-23 Thread Laszlo Ersek
(please read to the end) On 08/23/13 15:04, Sergey Isakov wrote: > No Martin, > I deleted folders BaseTools and Conf and started ab ovo. > The same error during compilation > - > GenFw: ERROR 3000: Invalid > --- > > On 23.08.2013, at 16:37, Olivier Martin wrote: > >> I have just tr

Re: [edk2] Efi/Tiano Compress

2013-08-23 Thread galla rao
Thanks Andrew for your detailed response, This info is quite useful On Fri, Aug 23, 2013 at 7:56 PM, Andrew Fish wrote: > > On Aug 23, 2013, at 2:44 AM, galla rao wrote: > > > Hello Everyone > > > > Can any one please elaborate how compression in uEFI environment can help > > reduce a third-p

Re: [edk2] Efi/Tiano Compress

2013-08-23 Thread Andrew Fish
On Aug 23, 2013, at 2:44 AM, galla rao wrote: > Hello Everyone > > Can any one please elaborate how compression in uEFI environment can help > reduce a third-party drive size. > > I have a scenario where the compiled driver size reduced to 1/3rd of it's > size by using EfiCompress.exe with

Re: [edk2] [edk2-buildtools] [PATCH] Fixed calculation of BaseOfCode in GenFw when the first code section is aligned

2013-08-23 Thread Sergey Isakov
What I notice - trunk/edk2/BaseTools/Source/Python/UPT/Parser/InfLibrarySectionParser.py +++ b/trunk/edk2/BaseTools/Source/Python/UPT/Parser/InfLibrarySectionParser.py @@ -170,28 +170,29 @@ File=FileName, Line=LineNo,

Re: [edk2] [edk2-buildtools] [PATCH] Fixed calculation of BaseOfCode in GenFw when the first code section is aligned

2013-08-23 Thread Sergey Isakov
Making svn up -r14590 resolves my problem. So? 23.08.2013, в 13:54, Olivier Martin написал(а): > The BaseTools has been synced up in EDKII r14591. EDK2 were using BaseTools > from June 2012. > > The changes I can see in GenFw since the last sync are: > > -- > commit 37b1901ef42e0442dc

Re: [edk2] Efi/Tiano Compress

2013-08-23 Thread galla rao
I am aware of these algo & pointers of code my concern is when i say load .efi from shell do i need to write decompress code in load.c in Shell. Do i need to tweak Shell to add decompress support before my driver gets called with Supported/Start/Stop On Fri, Aug 23, 2013 at 3:26 PM, Sergey

Re: [edk2] Efi/Tiano Compress

2013-08-23 Thread Sergey Isakov
Hi, There is a good example how to load lzma compressed module and decompress it edk2/DuetPkg/EfiLdr/EfiLoader.c LZMA compression is a subset of 7z compression and available with free license. 23.08.2013, в 13:44, galla rao написал(а): > Hello Everyone > > Can any one please elaborate how compr

Re: [edk2] [edk2-buildtools] [PATCH] Fixed calculation of BaseOfCode in GenFw when the first code section is aligned

2013-08-23 Thread Olivier Martin
The BaseTools has been synced up in EDKII r14591. EDK2 were using BaseTools from June 2012. The changes I can see in GenFw since the last sync are: -- commit 37b1901ef42e0442dc80237b5cae46d8dcc99b54 Author: oliviermartin Date: Wed Jul 31 10:43:59 2013 + BaseTools: Added suppor

Re: [edk2] [edk2-buildtools] [PATCH] Fixed calculation of BaseOfCode in GenFw when the first code section is aligned

2013-08-23 Thread Sergey Isakov
I don't know what is happen but since today I can't compile anything GenFw: ERROR 3000: Invalid /Users/Sergey/src/edk2/Build/OvmfX64/RELEASE_GCC47/X64/OvmfPkg/Sec/SecMain/DEBUG/SecMain.dll bad symbol definition. GenFw: ERROR 3000: Invalid /Users/Sergey/src/edk2/Build/OvmfX64/REL

[edk2] Efi/Tiano Compress

2013-08-23 Thread galla rao
Hello Everyone Can any one please elaborate how compression in uEFI environment can help reduce a third-party drive size. I have a scenario where the compiled driver size reduced to 1/3rd of it's size by using EfiCompress.exe with options Efi/Tiano The Algo it uses it LZ77/Huffman I could not lo

Re: [edk2] [PATCH] OvmfPkg NvVarsFileLib: Set NvVars variable after writing vars file

2013-08-23 Thread Michael Chang
On Thu, Aug 22, 2013 at 06:05:58PM -0700, Jordan Justen wrote: > On Mon, Aug 19, 2013 at 3:00 AM, Michael Chang wrote: > > On Sat, Aug 17, 2013 at 11:16:16PM -0700, Jordan Justen wrote: > >> The volatile 'NvVars' variable indicates that the variables do > >> not need to be loaded from the file aga

Re: [edk2] [edk2-buildtools] [PATCH] Fixed calculation of BaseOfCode in GenFw when the first code section is aligned

2013-08-23 Thread Olivier Martin
Any feedback on this one? > -Original Message- > From: Olivier Martin > Sent: 16 July 2013 05:51 > To: Jordan Justen > Cc: edk2-buildtools-de...@lists.sourceforge.net; edk2- > de...@lists.sourceforge.net > Subject: RE: [edk2-buildtools] [PATCH] Fixed calculation of BaseOfCode > in GenFw wh

Re: [edk2] [PATCH] MdePkg//BaseLib: Add Memory fence for AArch64

2013-08-23 Thread Olivier Martin
Any feedback on this one? From: Olivier Martin [mailto:olivier.mar...@arm.com] Sent: 08 August 2013 10:02 To: 'Gao, Liming'; 'Kinney, Michael D' Cc: 'edk2-devel@lists.sourceforge.net' Subject: [PATCH] MdePkg//BaseLib: Add Memory fence for AArch64 Dear MdePkg maintainers, Please find th

Re: [edk2] [PATCH] MdePkg/AArch64: Fixed some definitions in ProcessorBind.h

2013-08-23 Thread Olivier Martin
Any feedback on this one? From: Olivier Martin [mailto:olivier.mar...@arm.com] Sent: 08 August 2013 10:33 To: 'Gao, Liming'; 'Kinney, Michael D' Cc: edk2-devel@lists.sourceforge.net Subject: [PATCH] MdePkg/AArch64: Fixed some definitions in ProcessorBind.h Dear MdePkg maintainers, Plea

Re: [edk2] [edk2-buildtools] [PATCH] BaseTools Conf: Remove ELFGCC toolchain

2013-08-23 Thread Jordan Justen
I'm not seeing much concern with this change. Probably because no one uses ELFGCC. :) What do you think Dennis? On Sun, Aug 18, 2013 at 12:20 AM, Jordan Justen wrote: > This toolchain was only used with UnixPkg. UnixPkg has been > replaced by EmulatorPkg in EDK II, and EmulatorPkg does not > nee