<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" >Maximum size of FSP PELs are 16K. Anything about that gets truncated at a section boundary such that the total size is within 16K</div>
<div dir="ltr" > </div>
<div dir="ltr" >With regards,<br>Venkatesh<br><br>Power Firmware Development<br>IBM India Systems & Technology Lab, Bangalore, India<br>Phone: +91 80 4025 6920
<div> </div>
<div> </div>
<blockquote data-history-content-modified="1" style="border-left:solid #aaaaaa 2px; margin-left:5px; padding-left:5px; direction:ltr" >----- Original message -----<br>From: Patrick Williams III/Rochester/IBM<br>To: "Vasant Hegde" <hegdevasant@linux.vnet.ibm.com>, William G Hoffa/Rochester/IBM@ibmus, Venkatesh Sainath/India/IBM@ibmin<br>Cc: "skiboot@lists.ozlabs.org" <skiboot@lists.ozlabs.org>, "Alistair Popple" <alistair@popple.id.au><br>Subject: Re: [Skiboot] [PATCH] IPMI: Fix eSEL size<br>Date: Sun, Aug 2, 2015 4:14 PM<br> <br><font size="2" >I believe FSP PELs can be up to 64k. </font><br><br><font size="2" >I also believe the BMC code has a limit for the eSEL content to only be 2k. Hostboot truncates our PEL content to the eSEL limit when we send it to the BMC. </font><br><br><font size="2" >Patrick Williams</font><br><br><font size="2" >> On Aug 2, 2015, at 5:36 AM, Vasant Hegde <hegdevasant@linux.vnet.ibm.com> wrote:</font><br><font size="2" >> </font><br><font size="2" >> Currently we are assuming Max PEL size is 64K. But in reality log size</font><br><font size="2" >> much lesser than this. Hence reduce size to 16K...which is the max size</font><br><font size="2" >> we use in FSP based machine as well.</font><br><font size="2" >> </font><br><font size="2" >> Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com></font><br><font size="2" >> Cc: Alistair Popple <alistair@popple.id.au></font><br><font size="2" >> Cc: Jeremy Kerr <jk@ozlabs.org></font><br><font size="2" >> ---</font><br><font size="2" >> hw/ipmi/ipmi-sel.c | 2 +-</font><br><font size="2" >> 1 file changed, 1 insertion(+), 1 deletion(-)</font><br><font size="2" >> </font><br><font size="2" >> diff --git a/hw/ipmi/ipmi-sel.c b/hw/ipmi/ipmi-sel.c</font><br><font size="2" >> index cce4042..087ed97 100644</font><br><font size="2" >> --- a/hw/ipmi/ipmi-sel.c</font><br><font size="2" >> +++ b/hw/ipmi/ipmi-sel.c</font><br><font size="2" >> @@ -84,7 +84,7 @@ static struct oem_esel oem_esel = {</font><br><font size="2" >>  * the possible presence of the user defined section). We chose this</font><br><font size="2" >>  * size because it's what FSP uses, but we could probably reduce</font><br><font size="2" >>  * it. */</font><br><font size="2" >> -#define MAX_PEL_SIZE 0x10000</font><br><font size="2" >> +#define MAX_PEL_SIZE 0x4000</font><br><font size="2" >> </font><br><font size="2" >> #define ESEL_HDR_SIZE 7</font><br><font size="2" >> </font><br><font size="2" >> -- </font><br><font size="2" >> 2.1.0</font><br><font size="2" >> </font><br><font size="2" >> _______________________________________________</font><br><font size="2" >> Skiboot mailing list</font><br><font size="2" >> Skiboot@lists.ozlabs.org</font><br><font size="2" >> <a href="https://lists.ozlabs.org/listinfo/skiboot" target="_blank" >https://lists.ozlabs.org/listinfo/skiboot</a></font></blockquote></div></div><BR>