Handle non-platInit temporary attributes

A fapi attribute that is not platInit must be associated with a
volatile-zeroed persistency, rather than volatile.

Change-Id: Id5d3193e07158bc42c98f1bd2c8e9fdb2ee6b1b5
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/22713
Tested-by: Jenkins Server
Reviewed-by: Martin Gloff <mgloff@us.ibm.com>
Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Tested-by: FSP CI Jenkins
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

1 files changed, 9 insertions, 2 deletions

diff --git a/src/usr/targeting/xmltohb/updatetempsxml.pl b/src/usr/targeting/xmltohb/updatetempsxml.pl
index f8bd350b6..0777e489c 100755
--- a/src/usr/targeting/xmltohb/updatetempsxml.pl
+++ b/src/usr/targeting/xmltohb/updatetempsxml.pl
@@ -164,6 +164,8 @@ foreach my $TempAttr ( @{$fwDfltsXml->{attribute}} )
     $generic_id =~ s/ATTR_//;
     my $found = 0;
 
+    my $persistency = "volatile";
+
     # First, check if attribute definition exists in fapiattrs.xml
     foreach my $FapiAttr ( @{$fapiXml->{attribute}} )
     {
@@ -214,6 +216,13 @@ foreach my $TempAttr ( @{$fwDfltsXml->{attribute}} )
                 }
             }
 
+            # Non-platInit attributes need to be volatile-zeroed
+            #  or xmltohb.pl will throw an error
+            if( !(exists $FapiAttr->{platInit}) )
+            {
+                $persistency = "volatile-zeroed";
+            }
+
             last;
         }
     }
@@ -282,8 +291,6 @@ foreach my $TempAttr ( @{$fwDfltsXml->{attribute}} )
         }
     }
 
-    my $persistency = "volatile";
-
     # Create enumeration definition to support generic attribute if FAPI
     # attribute associated with temp FW default uses enum
     my $enum_id = "";