> -----Original Message-----
> From: Ye, Xiaolong
> Sent: Wednesday, September 25, 2019 12:37 AM
> To: Pei, Andy <andy....@intel.com>
> Cc: dev@dpdk.org; Xu, Rosen <rosen...@intel.com>; Zhang, Tianfei
> <tianfei.zh...@intel.com>; Zhang, Qi Z <qi.z.zh...@intel.com>; Lomartire,
> David <david.lomart...@intel.com>; Yigit, Ferruh <ferruh.yi...@intel.com>
> Subject: Re: [PATCH v6 04/17] raw/ifpga/base: add SEU error support
>
> On 09/19, Andy Pei wrote:
> >From: Tianfei zhang <tianfei.zh...@intel.com>
> >
> >This patch exposes SEU error information to application then
> >application could compare this information (128bit) with its own SMH
> >file to know if this SEU is a fatal error or not.
> >
> >Signed-off-by: Tianfei zhang <tianfei.zh...@intel.com>
> >Signed-off-by: Andy Pei <andy....@intel.com>
> >---
> > drivers/raw/ifpga/base/ifpga_defines.h | 5 +++-
> > drivers/raw/ifpga/base/ifpga_fme_error.c | 43
> ++++++++++++++++++++++++++++++
> > drivers/raw/ifpga/base/opae_ifpga_hw_api.h | 2 ++
> > 3 files changed, 49 insertions(+), 1 deletion(-)
> >
> >diff --git a/drivers/raw/ifpga/base/ifpga_defines.h
> >b/drivers/raw/ifpga/base/ifpga_defines.h
> >index 4216128..b450cb1 100644
> >--- a/drivers/raw/ifpga/base/ifpga_defines.h
> >+++ b/drivers/raw/ifpga/base/ifpga_defines.h
> >@@ -1149,7 +1149,8 @@ struct feature_fme_error_capability {
> > u8 support_intr:1;
> > /* MSI-X vector table entry number */
> > u16 intr_vector_num:12;
> >- u64 rsvd:51; /* Reserved */
> >+ u64 rsvd:50; /* Reserved */
> >+ u64 seu_support:1;
> > };
> > };
> > };
> >@@ -1171,6 +1172,8 @@ struct feature_fme_err {
> > struct feature_fme_ras_catfaterror ras_catfaterr;
> > struct feature_fme_ras_error_inj ras_error_inj;
> > struct feature_fme_error_capability fme_err_capability;
> >+ u64 seu_emr_l;
> >+ u64 seu_emr_h;
> > };
> >
> > /* FME Partial Reconfiguration Control */ diff --git
> >a/drivers/raw/ifpga/base/ifpga_fme_error.c
> >b/drivers/raw/ifpga/base/ifpga_fme_error.c
> >index a6d3dab..c9bac15 100644
> >--- a/drivers/raw/ifpga/base/ifpga_fme_error.c
> >+++ b/drivers/raw/ifpga/base/ifpga_fme_error.c
> >@@ -257,6 +257,45 @@ static void fme_global_error_uinit(struct
> ifpga_feature *feature)
> > UNUSED(feature);
> > }
> >
> >+static int fme_err_check_seu(struct feature_fme_err *fme_err) {
> >+ struct feature_fme_error_capability error_cap;
> >+
> >+ error_cap.csr = readq(&fme_err->fme_err_capability);
> >+
> >+ return error_cap.seu_support ? 1 : 0; }
> >+
> >+static int fme_err_get_seu_emr_low(struct ifpga_fme_hw *fme,
> >+ u64 *val)
> >+{
> >+ struct feature_fme_err *fme_err
> >+ = get_fme_feature_ioaddr_by_index(fme,
> >+ FME_FEATURE_ID_GLOBAL_ERR);
> >+
> >+ if (!fme_err_check_seu(fme_err))
> >+ return -ENODEV;
> >+
> >+ *val = readq(&fme_err->seu_emr_l);
> >+
> >+ return 0;
> >+}
> >+
> >+static int fme_err_get_seu_emr_high(struct ifpga_fme_hw *fme,
> >+ u64 *val)
> >+{
> >+ struct feature_fme_err *fme_err
> >+ = get_fme_feature_ioaddr_by_index(fme,
> >+ FME_FEATURE_ID_GLOBAL_ERR);
> >+
> >+ if (!fme_err_check_seu(fme_err))
> >+ return -ENODEV;
> >+
> >+ *val = readq(&fme_err->seu_emr_h);
> >+
> >+ return 0;
> >+}
>
> Above 2 functions can be combined to reduce duplication.
>
Thanks your suggestion, I will fix it in next version.
