DPDK patches and discussions
 help / color / mirror / Atom feed
From: Ibtisam Tariq <ibtisam.tariq@emumba.com>
To: David Marchand <david.marchand@redhat.com>
Cc: Maxime Coquelin <maxime.coquelin@redhat.com>,
	"Xia, Chenbo" <chenbo.xia@intel.com>,
	 Cristian Dumitrescu <cristian.dumitrescu@intel.com>,
	 "Singh, Jasvinder" <jasvinder.singh@intel.com>,
	"Mcnamara, John" <john.mcnamara@intel.com>,
	 "Pattan, Reshma" <reshma.pattan@intel.com>,
	 "Ananyev, Konstantin" <konstantin.ananyev@intel.com>,
	 "Kovacevic, Marko" <marko.kovacevic@intel.com>,
	dev <dev@dpdk.org>, Jiayu Hu <jiayu.hu@intel.com>,
	huawei.xie@intel.com
Subject: Re: [dpdk-dev] [PATCH 6/7] examples/vhost: enhance getopt_long usage
Date: Thu, 31 Dec 2020 12:00:16 +0500	[thread overview]
Message-ID: <CA+8bGBsFF2Fn5vtSXdeQGnKFoV67YV32eR_TNSqt-WZzZ+NK9A@mail.gmail.com> (raw)
In-Reply-To: <CAJFAV8ywJNCWVg_vEFO-2zjd=KwLVj3F6Wc3j6X7iJJOTDhCpw@mail.gmail.com>

Hi,
Sorry, for the late reply.

We have two solutions:
> - we keep this &builtin_net_driver flag use, but we must accept a 0
> return value from getopt_long.
> - we convert this option to the same scheme as the other and add a
> OPT_BUILTIN_NET_DRIVER_NUM like other options in this example,
>
What should I do for built_net_driver. The convention used in eal is the
2nd option.

On Mon, Dec 7, 2020 at 3:51 PM David Marchand <david.marchand@redhat.com>
wrote:

> On Tue, Nov 24, 2020 at 1:35 PM Ibtisam Tariq <ibtisam.tariq@emumba.com>
> wrote:
> > diff --git a/examples/vhost/main.c b/examples/vhost/main.c
> > index 8d8c3038b..ce8b64035 100644
> > --- a/examples/vhost/main.c
> > +++ b/examples/vhost/main.c
> > @@ -466,6 +466,33 @@ us_vhost_usage(const char *prgname)
> >                prgname);
> >  }
> >
> > +enum {
> > +#define OPT_VM2VM               "vm2vm"
> > +       OPT_VM2VM_NUM = 256,
> > +#define OPT_RX_RETRY            "rx-retry"
> > +       OPT_RX_RETRY_NUM,
> > +#define OPT_RX_RETRY_DELAY      "rx-retry-delay"
> > +       OPT_RX_RETRY_DELAY_NUM,
> > +#define OPT_RX_RETRY_NUMB        "rx-retry-num"
> > +       OPT_RX_RETRY_NUMB_NUM,
> > +#define OPT_MERGEABLE           "mergeable"
> > +       OPT_MERGEABLE_NUM,
> > +#define OPT_STATS               "stats"
> > +       OPT_STATS_NUM,
> > +#define OPT_SOCKET_FILE         "socket-file"
> > +       OPT_SOCKET_FILE_NUM,
> > +#define OPT_TX_CSUM             "tx-csum"
> > +       OPT_TX_CSUM_NUM,
> > +#define OPT_TSO                 "tso"
> > +       OPT_TSO_NUM,
> > +#define OPT_CLIENT              "client"
> > +#define OPT_BUILTIN_NET_DRIVER  "builtin-net-driver"
> > +#define OPT_DMA_TYPE                   "dma-type"
> > +       OPT_DMA_TYPE_NUM,
> > +#define OPT_DMAS                               "dmas"
> > +       OPT_DMAS_NUM,
>
> Nit: could you align those last two strings?
>
>
> > +};
> > +
> >  /*
> >   * Parse the arguments given in the command line of the application.
> >   */
> > @@ -477,19 +504,31 @@ us_vhost_parse_args(int argc, char **argv)
> >         unsigned i;
> >         const char *prgname = argv[0];
> >         static struct option long_option[] = {
> > -               {"vm2vm", required_argument, NULL, 0},
> > -               {"rx-retry", required_argument, NULL, 0},
> > -               {"rx-retry-delay", required_argument, NULL, 0},
> > -               {"rx-retry-num", required_argument, NULL, 0},
> > -               {"mergeable", required_argument, NULL, 0},
> > -               {"stats", required_argument, NULL, 0},
> > -               {"socket-file", required_argument, NULL, 0},
> > -               {"tx-csum", required_argument, NULL, 0},
> > -               {"tso", required_argument, NULL, 0},
> > -               {"client", no_argument, &client_mode, 1},
> > -               {"builtin-net-driver", no_argument, &builtin_net_driver,
> 1},
> > -               {"dma-type", required_argument, NULL, 0},
> > -               {"dmas", required_argument, NULL, 0},
> > +               {OPT_VM2VM, required_argument,
> > +                               NULL, OPT_VM2VM_NUM},
> > +               {OPT_RX_RETRY, required_argument,
> > +                               NULL, OPT_RX_RETRY_NUM},
> > +               {OPT_RX_RETRY_DELAY, required_argument,
> > +                               NULL, OPT_RX_RETRY_DELAY_NUM},
> > +               {OPT_RX_RETRY_NUMB, required_argument,
> > +                               NULL, OPT_RX_RETRY_NUMB_NUM},
> > +               {OPT_MERGEABLE, required_argument,
> > +                               NULL, OPT_MERGEABLE_NUM},
> > +               {OPT_STATS, required_argument,
> > +                               NULL, OPT_STATS_NUM},
> > +               {OPT_SOCKET_FILE, required_argument,
> > +                               NULL, OPT_SOCKET_FILE_NUM},
> > +               {OPT_TX_CSUM, required_argument,
> > +                               NULL, OPT_TX_CSUM_NUM},
> > +               {OPT_TSO, required_argument,
> > +                               NULL, OPT_TSO_NUM},
> > +               {OPT_CLIENT, no_argument, &client_mode, 1},
> > +               {OPT_BUILTIN_NET_DRIVER, no_argument,
> > +                               &builtin_net_driver, 1},
>
> We will have an issue here.
>
> Quoting the manual:
> """
>        flag   specifies  how  results are returned for a long option.
> If flag is NULL, then getopt_long() returns val.  (For example, the
> calling program may set val to the equivalent short option charac‐
>               ter.)  Otherwise, getopt_long() returns 0, and flag
> points to a variable which is set to val if the option is found, but
> left unchanged if the option is not found.
> """
>
> $ build/examples/dpdk-vhost --no-huge -m 1024 -l 0,1 -a 0:0.0 --
> --builtin-net-driver --socket-file /tmp/sock0 -p 0x1
> EAL: Detected 8 lcore(s)
> EAL: Detected 1 NUMA nodes
> EAL: Multi-process socket /run/user/114840/dpdk/rte/mp_socket
> EAL: Selected IOVA mode 'VA'
> EAL: Probing VFIO support...
> VHOST_CONFIG: build/examples/dpdk-vhost [EAL options] -- -p PORTMASK
>         --vm2vm [0|1|2]
>         --rx_retry [0|1] --mergeable [0|1] --stats [0-N]
>         --socket-file <path>
>         --nb-devices ND
>         -p PORTMASK: Set mask for ports to be used by application
>         --vm2vm [0|1|2]: disable/software(default)/hardware vm2vm comms
>         --rx-retry [0|1]: disable/enable(default) retries on rx.
> Enable retry if destintation queue is full
>         --rx-retry-delay [0-N]: timeout(in usecond) between retries on
> RX. This makes effect only if retries on rx enabled
>         --rx-retry-num [0-N]: the number of retries on rx. This makes
> effect only if retries on rx enabled
>         --mergeable [0|1]: disable(default)/enable RX mergeable buffers
>         --stats [0-N]: 0: Disable stats, N: Time in seconds to print stats
>         --socket-file: The path of the socket file.
>         --tx-csum [0|1] disable/enable TX checksum offload.
>         --tso [0|1] disable/enable TCP segment offload.
>         --client register a vhost-user socket as client mode.
>         --dma-type register dma type for your vhost async driver. For
> example "ioat" for now.
>         --dmas register dma channel for specific vhost device.
> EAL: Error - exiting with code: 1
>   Cause: Invalid argument
>
> This is because getopt_long returned 0.
>
> We have two solutions:
> - we keep this &builtin_net_driver flag use, but we must accept a 0
> return value from getopt_long.
> - we convert this option to the same scheme as the other and add a
> OPT_BUILTIN_NET_DRIVER_NUM like other options in this example,
>
> Maxime, opinion?
>
>
> --
> David Marchand
>
>

-- 
- Ibtisam

  reply	other threads:[~2020-12-31  7:00 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-24 12:32 [dpdk-dev] [PATCH 1/7] examples/fips_validation: " Ibtisam Tariq
2020-11-24 12:32 ` [dpdk-dev] [PATCH 2/7] examples/l3fwd-acl: " Ibtisam Tariq
2020-11-24 12:32 ` [dpdk-dev] [PATCH 3/7] examples/packet_ordering: " Ibtisam Tariq
2020-11-24 12:32 ` [dpdk-dev] [PATCH 4/7] examples/performance-thread/l3fwd-thread: " Ibtisam Tariq
2020-11-24 12:32 ` [dpdk-dev] [PATCH 5/7] examples/qos_sched: " Ibtisam Tariq
2020-11-24 12:32 ` [dpdk-dev] [PATCH 6/7] examples/vhost: " Ibtisam Tariq
2020-12-07 10:51   ` David Marchand
2020-12-31  7:00     ` Ibtisam Tariq [this message]
2020-11-24 12:32 ` [dpdk-dev] [PATCH 7/7] examples/vhost_crypto: " Ibtisam Tariq
2020-12-07 10:50 ` [dpdk-dev] [PATCH 1/7] examples/fips_validation: " David Marchand
  -- strict thread matches above, loose matches on Subject: below --
2020-11-11  8:15 Ibtisam Tariq
2020-11-11  8:15 ` [dpdk-dev] [PATCH 6/7] examples/vhost: " Ibtisam Tariq

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=CA+8bGBsFF2Fn5vtSXdeQGnKFoV67YV32eR_TNSqt-WZzZ+NK9A@mail.gmail.com \
    --to=ibtisam.tariq@emumba.com \
    --cc=chenbo.xia@intel.com \
    --cc=cristian.dumitrescu@intel.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=huawei.xie@intel.com \
    --cc=jasvinder.singh@intel.com \
    --cc=jiayu.hu@intel.com \
    --cc=john.mcnamara@intel.com \
    --cc=konstantin.ananyev@intel.com \
    --cc=marko.kovacevic@intel.com \
    --cc=maxime.coquelin@redhat.com \
    --cc=reshma.pattan@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;
as well as URLs for NNTP newsgroup(s).