Only check MC_SYNC_MODE for SCOM multicast workaround on Nimbus

This commit fixes a problem with the SCOM multicast workaround, which
is that it was checking the MC_SYNC_MODE on all models rather than
just Nimbus for which the attribute was intended. This was causing
issues in the multicast SCOM request to collect a bit from
MC_CHIPLET_UCS_FIR from each MC and put it in GLOBAL_UCS_FIR when a
unit checkstop occurs. Rather than collect the bits as intended, the
SCOM read was returning 0.

Change-Id: Ie20148d1be285d664d11c2ea081b55d80d85b9bd
CQ: SW492080
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/98192
Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com>
Reviewed-by: Corey V Swenson <cswenson@us.ibm.com>
Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>

Author: ibmzach

src/usr/scom/scom.C

@@ -5,7 +5,7 @@
 /*                                                                        */
 /* OpenPOWER HostBoot Project                                             */
 /*                                                                        */
-/* Contributors Listed Below - COPYRIGHT 2011,2019                        */
+/* Contributors Listed Below - COPYRIGHT 2011,2020                        */
 /* [+] International Business Machines Corp.                              */
 /*                                                                        */
 /*                                                                        */
@@ -1431,13 +1431,29 @@ errlHndl_t doMulticastWorkaround( DeviceFW::OperationType i_opType,
             if( !g_useMemChiplets
                 && ((l_unit >= P9_FIRST_MC) && (l_unit <= P9_LAST_MC)) )
             {
-                // Only access the mem chiplets if we're not in async mode
-                //  because we don't start clocks until later on in that case
-                auto l_syncMode =
-                i_target->getAttr<TARGETING::ATTR_MC_SYNC_MODE>();
-                if( l_syncMode == TARGETING::MC_SYNC_MODE_NOT_IN_SYNC )
+                using namespace TARGETING;
+
+                // Only access the mem chiplets if we're not in async mode on
+                // Nimbus systems, because we don't start clocks until later on
+                // in that case
+                TargetHandleList list;
+                PredicateCTM pred(CLASS_CHIP, TYPE_PROC);
+
+                targetService().getAssociated(list, i_target, TargetService::PARENT_BY_AFFINITY,
+                                              TargetService::ALL, &pred);
+
+                assert(list.size() <= 1,
+                       "Expected no more than one PROC parent on HUID 0x%08x",
+                       get_huid(i_target));
+
+                if (!list.empty() && list[0]->getAttr<ATTR_MODEL>() == MODEL_NIMBUS)
                 {
-                    continue;
+                    auto l_syncMode =
+                        i_target->getAttr<ATTR_MC_SYNC_MODE>();
+                    if( l_syncMode == MC_SYNC_MODE_NOT_IN_SYNC )
+                    {
+                        continue;
+                    }
                 }
             }
 #endif