Re: [PATCH] efi/cper, cxl: Remove cxl_err.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 3 Feb 2023 at 22:51, Ira Weiny <ira.weiny@xxxxxxxxx> wrote:
>
> Dan Williams wrote:
> > While going to create include/linux/cxl.h for some cross-subsystem CXL
> > definitions I noticed that include/linux/cxl_err.h was already present.
> > That header has no reason to be global, and it duplicates the RAS
> > Capability Structure definitions in drivers/cxl/cxl.h.
>
> Change looks fine to me but I'm failing to find these duplicate
> definitions?  Without any duplication it seems like stronger change to me.
>
> > A follow-on patch
> > can consider unifying the CXL native error tracing with the CPER error
> > printing.
> >
> > Also fixed up the spec reference as the latest released spec is v3.0.
> >
> > Cc: Smita Koralahalli <Smita.KoralahalliChannabasappa@xxxxxxx>
>
> Reviewed-by: Ira Weiny <ira.weiny@xxxxxxxxx>
>

Thanks. I'll queue this up in efi/next


> > Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx>
> > ---
> >  drivers/firmware/efi/cper_cxl.c |   12 +++++++++++-
> >  include/linux/cxl_err.h         |   22 ----------------------
> >  2 files changed, 11 insertions(+), 23 deletions(-)
> >  delete mode 100644 include/linux/cxl_err.h
> >
> > diff --git a/drivers/firmware/efi/cper_cxl.c b/drivers/firmware/efi/cper_cxl.c
> > index 53e435c4f310..a55771b99a97 100644
> > --- a/drivers/firmware/efi/cper_cxl.c
> > +++ b/drivers/firmware/efi/cper_cxl.c
> > @@ -9,7 +9,6 @@
> >
> >  #include <linux/cper.h>
> >  #include "cper_cxl.h"
> > -#include <linux/cxl_err.h>
> >
> >  #define PROT_ERR_VALID_AGENT_TYPE            BIT_ULL(0)
> >  #define PROT_ERR_VALID_AGENT_ADDRESS         BIT_ULL(1)
> > @@ -19,6 +18,17 @@
> >  #define PROT_ERR_VALID_DVSEC                 BIT_ULL(5)
> >  #define PROT_ERR_VALID_ERROR_LOG             BIT_ULL(6)
> >
> > +/* CXL RAS Capability Structure, CXL v3.0 sec 8.2.4.16 */
> > +struct cxl_ras_capability_regs {
> > +     u32 uncor_status;
> > +     u32 uncor_mask;
> > +     u32 uncor_severity;
> > +     u32 cor_status;
> > +     u32 cor_mask;
> > +     u32 cap_control;
> > +     u32 header_log[16];
> > +};
> > +
> >  static const char * const prot_err_agent_type_strs[] = {
> >       "Restricted CXL Device",
> >       "Restricted CXL Host Downstream Port",
> > diff --git a/include/linux/cxl_err.h b/include/linux/cxl_err.h
> > deleted file mode 100644
> > index 629e1bdeda44..000000000000
> > --- a/include/linux/cxl_err.h
> > +++ /dev/null
> > @@ -1,22 +0,0 @@
> > -/* SPDX-License-Identifier: GPL-2.0-only */
> > -/*
> > - * Copyright (C) 2022 Advanced Micro Devices, Inc.
> > - *
> > - * Author: Smita Koralahalli <Smita.KoralahalliChannabasappa@xxxxxxx>
> > - */
> > -
> > -#ifndef LINUX_CXL_ERR_H
> > -#define LINUX_CXL_ERR_H
> > -
> > -/* CXL RAS Capability Structure, CXL v3.1 sec 8.2.4.16 */
> > -struct cxl_ras_capability_regs {
> > -     u32 uncor_status;
> > -     u32 uncor_mask;
> > -     u32 uncor_severity;
> > -     u32 cor_status;
> > -     u32 cor_mask;
> > -     u32 cap_control;
> > -     u32 header_log[16];
> > -};
> > -
> > -#endif //__CXL_ERR_
> >
>
>



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux