public inbox for netdev@vger.kernel.org
 help / color / mirror / Atom feed
From: "D. Wythe" <alibuda@linux.alibaba.com>
To: Alexandra Winter <wintera@linux.ibm.com>
Cc: Tony Lu <tonylu@linux.alibaba.com>,
	Karsten Graul <kgraul@linux.ibm.com>,
	kuba@kernel.org, davem@davemloft.net, netdev@vger.kernel.org,
	linux-s390@vger.kernel.org, linux-rdma@vger.kernel.org
Subject: Re: [RFC net-next] net/smc:introduce 1RTT to SMC
Date: Thu, 2 Jun 2022 11:26:18 +0800	[thread overview]
Message-ID: <20220602032618.GA96227@e02h04389.eu6sqa> (raw)
In-Reply-To: <7fb28436-1fca-ba4c-7745-ca88d83c657b@linux.ibm.com>

On Wed, Jun 01, 2022 at 01:35:52PM +0200, Alexandra Winter wrote:
> 
> 
> On 01.06.22 11:24, Tony Lu wrote:
> > On Wed, Jun 01, 2022 at 02:33:09PM +0800, D. Wythe wrote:
> >>
> >> 在 2022/5/25 下午9:42, Alexandra Winter 写道:
> >>
> >>> We need to carefully evaluate them and make sure everything is compatible
> >>> with the existing implementations of SMC-D and SMC-R v1 and v2. In the
> >>> typical s390 environment ROCE LAG is propably not good enough, as the card
> >>> is still a single point of failure. So your ideas need to be compatible
> >>> with link redundancy. We also need to consider that the extension of the
> >>> protocol does not block other desirable extensions.
> >>>
> >>> Your prototype is very helpful for the understanding. Before submitting any
> >>> code patches to net-next, we should agree on the details of the protocol
> >>> extension. Maybe you could formulate your proposal in plain text, so we can
> >>> discuss it here?
> >>>
> >>> We also need to inform you that several public holidays are upcoming in the
> >>> next weeks and several of our team will be out for summer vacation, so please
> >>> allow for longer response times.
> >>>
> >>> Kind regards
> >>> Alexandra Winter
> >>>
> >>
> >> Hi alls,
> >>
> >> In order to achieve signle-link compatibility, we must
> >> complete at least once negotiation. We wish to provide
> >> higher scalability while meeting this feature. There are
> >> few ways to reach this.
> >>
> >> 1. Use the available reserved bits. According to
> >> the SMC v2 protocol, there are at least 28 reserved octets
> >> in PROPOSAL MESSAGE and at least 10 reserved octets in
> >> ACCEPT MESSAGE are available. We can define an area in which
> >> as a feature area, works like bitmap. Considering the subsequent
> >> scalability, we MAY use at least 2 reserved ctets, which can support
> >> negotiation of at least 16 features.
> >>
> >> 2. Unify all the areas named extension in current
> >> SMC v2 protocol spec without reinterpreting any existing field
> >> and field offset changes, including 'PROPOSAL V1 IP Subnet Extension',
> >> 'PROPOSAL V2 Extension', 'PROPOSAL SMC-DV2 EXTENSION' .etc. And provides
> >> the ability to grow dynamically as needs expand. This scheme will use
> >> at least 10 reserved octets in the PROPOSAL MESSAGE and at least 4 reserved
> >> octets in ACCEPT MESSAGE and CONFIRM MESSAGE. Fortunately, we only need to
> >> use reserved fields, and the current reserved fields are sufficient. And
> >> then we can easily add a new extension named SIGNLE LINK. Limited by space,
> >> the details will be elaborated after the scheme is finalized.
> > 
> > After reading this and latest version of protocol, I agree with that the
> > idea to provide a more flexible extension facilities. And, it's a good
> > chance for us to set here talking about the protocol extension.
> > 
> > There are some potential scenarios that need flexible extensions in my
> > mind:
> > - other protocols support, such as iWARP / IB or new version protocol,
> > - dozens of feature flags in the future, like this proposal. With the
> >   growth of new feature, it could overflow bitmap.
> > 
> > Actually, this extension facilities are very similar to TCP options.
> > 
> > So what about your opinions about the solution of this? If there are
> > some existed approaches for the future extensions, maybe this can get
> > involved in it. Or we can start a discuss about this as this mail
> > mentioned.
> > 
> > Also, I am wondering if there is plan to update the RFC7609, add the
> > latest v2 support?
> > 
> > Thanks,
> > Tony Lu
> 
> We have asked the SMC protocol owners about their opinion about using the
> reserved fields for new options in particular, and about where and how to
> discuss this in general. (including where to document the versions).
> Please allow some time for us to come back to you.
> 
> Kind regards
> Alexandra

Thank you for the information. Before we officially push the document update,
if you had any suggestions for the two schemes we are mentioned above,
or which one you prefer, please keep us informed.

Best wishes.
D. Wyther


  reply	other threads:[~2022-06-02  3:26 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-24  6:52 [RFC net-next] net/smc:introduce 1RTT to SMC D. Wythe
2022-05-24  7:49 ` Tony Lu
2022-05-25 13:42   ` Alexandra Winter
2022-05-26  3:47     ` D. Wythe
2022-05-26  7:04     ` Tony Lu
2022-06-01  6:33     ` D. Wythe
2022-06-01  9:24       ` Tony Lu
2022-06-01 11:35         ` Alexandra Winter
2022-06-02  3:26           ` D. Wythe [this message]
2022-06-16 13:49       ` D. Wythe
2022-06-23 11:59         ` Alexandra Winter
2022-06-23 12:50           ` D. Wythe

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=20220602032618.GA96227@e02h04389.eu6sqa \
    --to=alibuda@linux.alibaba.com \
    --cc=davem@davemloft.net \
    --cc=kgraul@linux.ibm.com \
    --cc=kuba@kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=tonylu@linux.alibaba.com \
    --cc=wintera@linux.ibm.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