> -----Original Message-----
> From: [email protected] <[email protected]> On
> Behalf Of Joe Perches
> Sent: Wednesday, June 17, 2020 10:33 PM
> To: Kirsher, Jeffrey T <[email protected]>; [email protected]
> Cc: Michael, Alice <[email protected]>; [email protected];
> [email protected]; [email protected]; Brady, Alan
> <[email protected]>; Burra, Phani R <[email protected]>; Hay,
> Joshua A <[email protected]>; Chittim, Madhu
> <[email protected]>; Linga, Pavan Kumar
> <[email protected]>; Skidmore, Donald C
> <[email protected]>; Brandeburg, Jesse
> <[email protected]>; Samudrala, Sridhar
> <[email protected]>
> Subject: Re: [net-next 02/15] iecm: Add framework set of header files
> 
> On Wed, 2020-06-17 at 22:13 -0700, Jeff Kirsher wrote:
> > From: Alice Michael <[email protected]>
> []
> > diff --git a/include/linux/net/intel/iecm_controlq_api.h
> > b/include/linux/net/intel/iecm_controlq_api.h
> []
> > +enum iecm_ctlq_err {
> > +   IECM_CTLQ_RC_OK         = 0,  /* Success */
> 
> Why is it necessary to effectively duplicate the generic error codes with
> different error numbers?
> 
Ah this is a good point.  This was an old API we were looking at early on and 
abandoned and I am working to pull this chunk of memory out of the patches for 
a V2.  Thanks

> > +   IECM_CTLQ_RC_EPERM      = 1,  /* Operation not permitted */
> > +   IECM_CTLQ_RC_ENOENT     = 2,  /* No such element */
> > +   IECM_CTLQ_RC_ESRCH      = 3,  /* Bad opcode */
> > +   IECM_CTLQ_RC_EINTR      = 4,  /* Operation interrupted */
> > +   IECM_CTLQ_RC_EIO        = 5,  /* I/O error */
> > +   IECM_CTLQ_RC_ENXIO      = 6,  /* No such resource */
> > +   IECM_CTLQ_RC_E2BIG      = 7,  /* Arg too long */
> > +   IECM_CTLQ_RC_EAGAIN     = 8,  /* Try again */
> > +   IECM_CTLQ_RC_ENOMEM     = 9,  /* Out of memory */
> > +   IECM_CTLQ_RC_EACCES     = 10, /* Permission denied */
> > +   IECM_CTLQ_RC_EFAULT     = 11, /* Bad address */
> > +   IECM_CTLQ_RC_EBUSY      = 12, /* Device or resource busy */
> > +   IECM_CTLQ_RC_EEXIST     = 13, /* object already exists */
> > +   IECM_CTLQ_RC_EINVAL     = 14, /* Invalid argument */
> > +   IECM_CTLQ_RC_ENOTTY     = 15, /* Not a typewriter */
> > +   IECM_CTLQ_RC_ENOSPC     = 16, /* No space left or allocation
> failure */
> > +   IECM_CTLQ_RC_ENOSYS     = 17, /* Function not implemented */
> > +   IECM_CTLQ_RC_ERANGE     = 18, /* Parameter out of range */
> > +   IECM_CTLQ_RC_EFLUSHED   = 19, /* Cmd flushed due to prev cmd
> error */
> > +   IECM_CTLQ_RC_BAD_ADDR   = 20, /* Descriptor contains a bad
> pointer */
> > +   IECM_CTLQ_RC_EMODE      = 21, /* Op not allowed in current
> dev mode */
> > +   IECM_CTLQ_RC_EFBIG      = 22, /* File too big */
> > +   IECM_CTLQ_RC_ENOSEC     = 24, /* Missing security manifest */
> > +   IECM_CTLQ_RC_EBADSIG    = 25, /* Bad RSA signature */
> > +   IECM_CTLQ_RC_ESVN       = 26, /* SVN number prohibits this package
> */
> > +   IECM_CTLQ_RC_EBADMAN    = 27, /* Manifest hash mismatch */
> > +   IECM_CTLQ_RC_EBADBUF    = 28, /* Buffer hash mismatches
> manifest */
> > +};
> 

Reply via email to