From: Yasufumi Ogawa <yasufum.o@gmail.com> To: spp@dpdk.org, ferruh.yigit@intel.com, yasufum.o@gmail.com Subject: [spp] [PATCH 06/29] tools/sppc: update dev options of l3fwd-acl Date: Tue, 25 Feb 2020 19:34:23 +0900 Message-ID: <20200225103446.8243-7-yasufum.o@gmail.com> (raw) In-Reply-To: <20200225103446.8243-1-yasufum.o@gmail.com> Update options for vdevs of l3fwd-acl container app. Signed-off-by: Yasufumi Ogawa <yasufum.o@gmail.com> --- tools/sppc/app/l3fwd-acl.py | 59 +++++++++++++++++++++---------------- 1 file changed, 33 insertions(+), 26 deletions(-) diff --git a/tools/sppc/app/l3fwd-acl.py b/tools/sppc/app/l3fwd-acl.py index d1e2178..46163f4 100755 --- a/tools/sppc/app/l3fwd-acl.py +++ b/tools/sppc/app/l3fwd-acl.py @@ -66,7 +66,7 @@ def parse_args(): def check_config_format(config_opt, nof_queues): - """Check if config format is valid + """Check if config format is valid. Config options is for Determining which queues from which ports are mapped to which cores. @@ -106,8 +106,8 @@ def check_config_format(config_opt, nof_queues): i = i + 1 if nof_tx_queues > nof_queues: - print('Error: {}={} should be equal or less than {}={}!'.format( - 'tx_queues', nof_tx_queues, 'nof_queues', nof_queues)) + print('Error: {0:s}={1:d} should be equal or less than {2:s}={3:d}!'. + format('tx_queues', nof_tx_queues, 'nof_queues', nof_queues)) print("\tnof_queues is defiend with '-nq' or '--nof-queues' option") return False @@ -115,7 +115,7 @@ def check_config_format(config_opt, nof_queues): def check_jumbo_opt(enable_jumbo, max_pkt_len): - """Check if jumbo frame option is valid + """Check if jumbo frame option is valid. Jumbo frame is enabled with '--enable-jumbo' and max packet size is defined with '--max-pkt-len'. @@ -128,8 +128,8 @@ def check_jumbo_opt(enable_jumbo, max_pkt_len): if max_pkt_len is not None: if (max_pkt_len < 64) or (max_pkt_len > 9600): - print('Error: --max-pkt-len {} should be {}-{}'.format( - max_pkt_len, 64, 9600)) + print('Error: --max-pkt-len {0:d} should be {1:d}-{2:d}'. + format(max_pkt_len, 64, 9600)) return False return True @@ -149,39 +149,46 @@ def main(): # Check for other mandatory opitons. if args.port_mask is None: common.error_exit('--port-mask') - if args.dev_ids is None: - common.error_exit('--dev-ids') - # Setup for vhost devices with given device IDs. - dev_ids_list = app_helper.dev_ids_to_list(args.dev_ids) - sock_files = app_helper.sock_files(dev_ids_list) + # Setup devices with given device UIDs. + dev_uids = None + sock_files = None + if args.dev_uids is not None: + if app_helper.is_valid_dev_uids(args.dev_uids) is False: + print('Invalid option: {}'.format(args.dev_uids)) + exit() + + dev_uids_list = args.dev_uids.split(',') + sock_files = app_helper.sock_files(dev_uids_list) # Setup docker command. docker_cmd = ['sudo', 'docker', 'run', '\\'] docker_opts = app_helper.setup_docker_opts( args, container_image, sock_files) - # Parse vhost device IDs and Check the number of devices is - # sufficient for port mask. - if app_helper.is_sufficient_dev_ids( - args.dev_ids, args.port_mask) is not True: - print("Error: Cannot reserve ports '{} (= 0b{})' on '{}'.".format( - args.port_mask, - format(int(args.port_mask, 16), 'b'), - args.dev_ids)) + # Check given number of ports is enough for portmask. + if (args.port_mask is None) or (args.dev_uids is None): + pass + elif app_helper.is_sufficient_ports(args) is not True: + print("Error: Not enough ports, {0:d} devs for '{1:s}(=0b{2:s})'.". + format(len(args.dev_uids.split(',')), args.port_mask, + format(int(args.port_mask, 16), 'b'))) exit() # Setup l3fwd-acl command runs on container. - cmd_path = '{}/examples/l3fwd-acl/{}/l3fwd-acl'.format( + cmd_path = '{0:s}/examples/l3fwd-acl/{1:s}/l3fwd-acl'.format( env.RTE_SDK, env.RTE_TARGET) l3fwd_cmd = [cmd_path, '\\'] # Setup EAL options. - file_prefix = 'spp-l3fwd-acl-container{}'.format(dev_ids_list[0]) + if args.name is not None: + file_prefix = app_helper.gen_sppc_file_prefix(args.name) + else: + file_prefix = app_helper.gen_sppc_file_prefix('l3fwd-acl') eal_opts = app_helper.setup_eal_opts(args, file_prefix) - # Setup l3fwd options. + # Setup l3fwd-acl options. l3fwd_opts = ['-p', args.port_mask, '\\'] if args.config is None: @@ -190,7 +197,7 @@ def main(): print('Invalid config: {}'.format(args.config)) exit() else: - l3fwd_opts += ['--config', '"{}"'.format(args.config), '\\'] + l3fwd_opts += ['--config', '"{:s}"'.format(args.config), '\\'] jumbo_opt_valid = False if args.enable_jumbo is True: @@ -202,13 +209,13 @@ def main(): if args.rule_ipv4 is not None: if os.path.exists(args.rule_ipv4): - l3fwd_opts += ['--rule_ipv4', '"{}"'.format(args.rule_ipv4), '\\'] + l3fwd_opts += ['--rule_ipv4', '"{:s}"'.format(args.rule_ipv4), '\\'] else: print('Error: "{}" does not exist'.format(args.rule_ipv4)) exit() if args.rule_ipv6 is not None: if os.path.exists(args.rule_ipv6): - l3fwd_opts += ['--rule_ipv6', '"{}"'.format(args.rule_ipv6), '\\'] + l3fwd_opts += ['--rule_ipv6', '"{:s}"'.format(args.rule_ipv6), '\\'] else: print('Error: "{}" does not exist'.format(args.rule_ipv6)) exit() @@ -220,7 +227,7 @@ def main(): if (args.enable_jumbo is not None) and (jumbo_opt_valid is True): l3fwd_opts += ['--enable-jumbo', '\\'] if args.max_pkt_len is not None: - l3fwd_opts += ['--max-pkt-len {}'.format(args.max_pkt_len), '\\'] + l3fwd_opts += ['--max-pkt-len {:d}'.format(args.max_pkt_len), '\\'] if args.no_numa is True: l3fwd_opts += ['--no-numa', '\\'] -- 2.17.1
next prev parent reply other threads:[~2020-02-25 10:35 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-02-25 10:34 [spp] [PATCH 00/29] Update SPP Container tools Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 01/29] tools/sppc: update options for assigning devices Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 02/29] tools/sppc: update dev options of l2fwd Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 03/29] tools/sppc: add container name option Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 04/29] tools/sppc: update l2fwd app for " Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 05/29] tools/sppc: update dev options of l3fwd Yasufumi Ogawa 2020-02-25 10:34 ` Yasufumi Ogawa [this message] 2020-02-25 10:34 ` [spp] [PATCH 07/29] tools/sppc: update dev options of testpmd Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 08/29] tools/sppc: update dev options of pktgen Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 09/29] tools/sppc: update dev options of load-balancer Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 10/29] tools/sppc: version checker for container DPDK ver Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 11/29] tools/sppc: check DPDK ver in load-balancer Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 12/29] tools/sppc: setup spp_pri opts in app_helper Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 13/29] tools/sppc: define file prefix for SPP Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 14/29] tools/sppc: update dev options of spp_primary Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 15/29] tools/sppc: setup with docker opts in SPP pri Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 16/29] tools/sppc: update calling setup_docker_opts() Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 17/29] tools/sppc: update dev options of helloworld Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 18/29] tools/sppc: update dev options of suricata Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 19/29] tools/sppc: update dev options of spp_nfv Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 20/29] tools/sppc: change to gen EAL opts with app name Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 21/29] tools/sppc: remove nouse variable Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 22/29] bin: remove sock files created by docker Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 23/29] tools/sppc: skip checking rule file if dry run Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 24/29] docs: revise examples in sppc Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 25/29] docs: update versions in " Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 26/29] docs: update old example in spp_primary container Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 27/29] tools/sppc: python3 support for sppc build tool Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 28/29] docs: update app container help msg Yasufumi Ogawa 2020-02-25 10:34 ` [spp] [PATCH 29/29] docs: update howto define app container guide Yasufumi Ogawa
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=20200225103446.8243-7-yasufum.o@gmail.com \ --to=yasufum.o@gmail.com \ --cc=ferruh.yigit@intel.com \ --cc=spp@dpdk.org \ /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
Soft Patch Panel This inbox may be cloned and mirrored by anyone: git clone --mirror https://inbox.dpdk.org/spp/0 spp/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 spp spp/ https://inbox.dpdk.org/spp \ spp@dpdk.org public-inbox-index spp Example config snippet for mirrors. Newsgroup available over NNTP: nntp://inbox.dpdk.org/inbox.dpdk.spp AGPL code for this site: git clone https://public-inbox.org/public-inbox.git