Netdev List
 help / color / mirror / Atom feed
From: Joe Perches <joe@perches.com>
To: Jeff Kirsher <jeffrey.t.kirsher@intel.com>, davem@davemloft.net
Cc: Alice Michael <alice.michael@intel.com>,
	netdev@vger.kernel.org, nhorman@redhat.com, sassmann@redhat.com,
	Alan Brady <alan.brady@intel.com>,
	Phani Burra <phani.r.burra@intel.com>,
	Joshua Hay <joshua.a.hay@intel.com>,
	Madhu Chittim <madhu.chittim@intel.com>,
	Pavan Kumar Linga <pavan.kumar.linga@intel.com>,
	Donald Skidmore <donald.c.skidmore@intel.com>,
	Jesse Brandeburg <jesse.brandeburg@intel.com>,
	Sridhar Samudrala <sridhar.samudrala@intel.com>
Subject: Re: [net-next 02/15] iecm: Add framework set of header files
Date: Wed, 17 Jun 2020 22:32:33 -0700	[thread overview]
Message-ID: <57110139759cc77f21a150a3faed0d2584dfbe21.camel@perches.com> (raw)
In-Reply-To: <20200618051344.516587-3-jeffrey.t.kirsher@intel.com>

On Wed, 2020-06-17 at 22:13 -0700, Jeff Kirsher wrote:
> From: Alice Michael <alice.michael@intel.com>
[]
> 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?

> +	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	other threads:[~2020-06-18  5:32 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-18  5:13 [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2020-06-17 Jeff Kirsher
2020-06-18  5:13 ` [net-next 01/15] virtchnl: Extend AVF ops Jeff Kirsher
2020-06-18  5:13 ` [net-next 02/15] iecm: Add framework set of header files Jeff Kirsher
2020-06-18  5:32   ` Joe Perches [this message]
2020-06-18 23:07     ` Michael, Alice
2020-06-18  5:13 ` [net-next 03/15] iecm: Add TX/RX " Jeff Kirsher
2020-06-18  5:13 ` [net-next 04/15] iecm: Common module introduction and function stubs Jeff Kirsher
2020-06-18  5:13 ` [net-next 05/15] iecm: Add basic netdevice functionality Jeff Kirsher
2020-06-18  5:13 ` [net-next 06/15] iecm: Implement mailbox functionality Jeff Kirsher
2020-06-18  5:13 ` [net-next 07/15] iecm: Implement virtchnl commands Jeff Kirsher
2020-06-18  5:13 ` [net-next 08/15] iecm: Implement vector allocation Jeff Kirsher
2020-06-18  5:13 ` [net-next 09/15] iecm: Init and allocate vport Jeff Kirsher
2020-06-18  5:13 ` [net-next 10/15] iecm: Deinit vport Jeff Kirsher
2020-06-18  5:13 ` [net-next 11/15] iecm: Add splitq TX/RX Jeff Kirsher
2020-06-18  5:13 ` [net-next 12/15] iecm: Add singleq TX/RX Jeff Kirsher
2020-06-18  5:13 ` [net-next 13/15] iecm: Add ethtool Jeff Kirsher
2020-06-18 21:17   ` Michal Kubecek
2020-06-19  0:36     ` Brady, Alan
2020-06-18 23:50   ` Jakub Kicinski
2020-06-19  0:46     ` Michael, Alice
2020-06-18  5:13 ` [net-next 14/15] iecm: Add iecm to the kernel build system Jeff Kirsher
2020-06-18 15:43   ` Jakub Kicinski
2020-06-18  5:13 ` [net-next 15/15] idpf: Introduce idpf driver Jeff Kirsher

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=57110139759cc77f21a150a3faed0d2584dfbe21.camel@perches.com \
    --to=joe@perches.com \
    --cc=alan.brady@intel.com \
    --cc=alice.michael@intel.com \
    --cc=davem@davemloft.net \
    --cc=donald.c.skidmore@intel.com \
    --cc=jeffrey.t.kirsher@intel.com \
    --cc=jesse.brandeburg@intel.com \
    --cc=joshua.a.hay@intel.com \
    --cc=madhu.chittim@intel.com \
    --cc=netdev@vger.kernel.org \
    --cc=nhorman@redhat.com \
    --cc=pavan.kumar.linga@intel.com \
    --cc=phani.r.burra@intel.com \
    --cc=sassmann@redhat.com \
    --cc=sridhar.samudrala@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox