diff options
author | Shakeeb <shakeebbk@in.ibm.com> | 2016-07-27 05:08:51 -0500 |
---|---|---|
committer | Sachin Gupta <sgupta2m@in.ibm.com> | 2016-07-28 01:13:36 -0400 |
commit | c8dab863b9e28312fae8531c51e133de56522b09 (patch) | |
tree | 9afd3c6466c5f06e5d9c850ed7234e944129188a /sbe | |
parent | 65c5f96611fd8cd6b6ff75959f82a80f6c51cbe4 (diff) | |
download | talos-sbe-c8dab863b9e28312fae8531c51e133de56522b09.tar.gz talos-sbe-c8dab863b9e28312fae8531c51e133de56522b09.zip |
Removal of FFDC maximum length check
Change-Id: Idda8a40aa25de6f4f822fa2a36f57f5ce04dc1c3
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/27497
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Diffstat (limited to 'sbe')
-rw-r--r-- | sbe/sbefw/sbeFFDC.H | 24 |
1 files changed, 1 insertions, 23 deletions
diff --git a/sbe/sbefw/sbeFFDC.H b/sbe/sbefw/sbeFFDC.H index 8cf8408e..c891c64b 100644 --- a/sbe/sbefw/sbeFFDC.H +++ b/sbe/sbefw/sbeFFDC.H @@ -31,10 +31,6 @@ #include "sbeFFDCType.H" #include "sbeSpMsg.H" - -//Since maximum error log supported in HB is 4KB -#define SBE_FFDC_MAX_LENGTH 0x1000 - //PIBMEM attribute dump extern G_sbe_attrs_t G_sbe_attrs; @@ -43,7 +39,7 @@ extern G_sbe_attrs_t G_sbe_attrs; //Definition - Identifier // length of the blob // pointer to the data -constexpr sbeFFDCUserData_t sbeFFDCUserDataArray[] = +const sbeFFDCUserData_t sbeFFDCUserDataArray[] = {{{SBE_FFDC_TRACE_DUMP, sizeof(PkTraceBuffer)}, (const void *)&g_pk_trace_buf, @@ -53,17 +49,6 @@ constexpr sbeFFDCUserData_t sbeFFDCUserDataArray[] = (const void *)&G_sbe_attrs, }, }; -#define SBE_FFDC_NUM_USER_DATA (sizeof(sbeFFDCUserDataArray)/\ - sizeof(sbeFFDCUserData_t)) - -//Compile time usage - to derive the sum of all user data -//as defined in sbeFFDCUserDataArray -constexpr uint16_t getMaxFFDCUserDataLength(int8_t index) -{ - return index<0 ? 0 : sbeFFDCUserDataArray[index].userDataId.fieldLen +\ - getMaxFFDCUserDataLength(index-1); -} - //SBE internal FFDC package singleton class class SbeFFDCPackage @@ -96,13 +81,6 @@ public: iv_sbeFFDCDataHeader.primaryStatus = SBE_PRI_OPERATION_SUCCESSFUL; iv_sbeFFDCDataHeader.secondaryStatus = SBE_SEC_OPERATION_SUCCESSFUL; - //Making sure that the maximum length of FFDC package - //can be accomodated - static_assert((sizeof(sbeResponseFfdc_t) + sizeof(sbeFFDCDataHeader_t) + - getMaxFFDCUserDataLength(SBE_FFDC_NUM_USER_DATA-1)) \ - <= SBE_FFDC_MAX_LENGTH, - "length of FFDC package must never exceed" - "SBE_FFDC_MAX_LENGTH "); //length and dumpFields will be filled up depending on the fields //to be sent in send APIs iv_sbeFFDCDataHeader.dumpFields = {0}; |