Soft Patch Panel
 help / color / mirror / Atom feed
From: Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp>
To: spp@dpdk.org, ferruh.yigit@intel.com
Cc: srv-apl-arch@lab.ntt.co.jp
Subject: Re: [spp] [RFC] assign global unique port ID
Date: Mon, 29 Jan 2018 21:44:20 +0900	[thread overview]
Message-ID: <ae7b8316-d7d2-5d3e-1cae-8fe53c5ba05c@lab.ntt.co.jp> (raw)
In-Reply-To: <093f4d79-1238-3be8-f242-062bca27a933@lab.ntt.co.jp>

Hi,

To assign port with unique ID, define a struct 'porttype_map' for 
managing a combination of port type and alias of it. I also add a parser 
to convert unique ID to local ID of each of secondaries.

This update is applied to spp_nfv, spp_vm and spp.py which checks if 
user input of unique ID is valid.

Thanks,
Yasufumi

On 2017/10/03 15:34, Yasufumi Ogawa wrote:
> Hi, Gerald, Sy Jong,
> 
> In spp, port ID is assigned as a number incrementally and the number of 
> ID can be different from each sec processes. For instance, ring 0 is 
> added to sec 1 at first while it is added to sec 2 after ring 1, ring 0 
> is referred as port 2 from sec 1 and port 3 from sec 2.
> 
>    spp > sec 1;status
>    ...
>    port id: 0,on,PHY,outport: -99
>    port id: 1,on,PHY,outport: -99
>    port id: 2,on,RING(0),outport: -99  # ring 0 is referred as port 2
> 
>    spp > sec 2;status
>    ...
>    port id: 0,on,PHY,outport: -99
>    port id: 1,on,PHY,outport: -99
>    port id: 2,on,RING(1),outport: -99
>    port id: 3,on,RING(0),outport: -99  # ring 0 is referred as port 3
> 
> User always has to check the status any time patching to avoid mistake. 
> However, it must be annoying and an error is possibly happened.
> 
> I propose to assign global unique ID for port. In this update, port ID 
> is referred as a combination of PMD type and its ID, not a number. For 
> instance,
> 
>    (Before) spp> patch 0 2
>    (After)  spp> patch phy:0 ring:0  # means to patch from phy 0 to ring 0
> 
> If you agree, I would like to send patches for update.
> 
> Thanks,
> Yasufumi
> 


-- 
Yasufumi Ogawa
NTT Network Service Systems Labs

      reply	other threads:[~2018-01-29 12:45 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-03  6:34 Yasufumi Ogawa
2018-01-29 12:44 ` Yasufumi Ogawa [this message]

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=ae7b8316-d7d2-5d3e-1cae-8fe53c5ba05c@lab.ntt.co.jp \
    --to=ogawa.yasufumi@lab.ntt.co.jp \
    --cc=ferruh.yigit@intel.com \
    --cc=spp@dpdk.org \
    --cc=srv-apl-arch@lab.ntt.co.jp \
    /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).