From: Jason Gunthorpe <jgg@ziepe.ca>
To: Dennis Dalessandro <dennis.dalessandro@intel.com>
Cc: dledford@redhat.com, linux-rdma@vger.kernel.org,
Mike Marciniszyn <mike.marciniszyn@intel.com>,
Dennis Dalessandro <dennis.alessandro@intel.com>,
Gary Leshner <Gary.S.Leshner@intel.com>,
Kaike Wan <kaike.wan@intel.com>
Subject: Re: [PATCH for-next 13/16] IB/{hfi1, ipoib, rdma}: Broadcast ping sent packets which exceeded mtu size
Date: Fri, 21 Feb 2020 19:32:49 -0400 [thread overview]
Message-ID: <20200221233249.GM31668@ziepe.ca> (raw)
In-Reply-To: <2c91a053-add3-a7f9-2da1-f56f4c70381d@intel.com>
On Fri, Feb 21, 2020 at 02:40:28PM -0500, Dennis Dalessandro wrote:
> On 2/18/2020 7:42 PM, Jason Gunthorpe wrote:
> > On Mon, Feb 10, 2020 at 08:19:44AM -0500, Dennis Dalessandro wrote:
> > > From: Gary Leshner <Gary.S.Leshner@intel.com>
> > >
> > > When in connected mode ipoib sent broadcast pings which exceeded the mtu
> > > size for broadcast addresses.
> > >
> > > Add an mtu attribute to the rdma_netdev structure which ipoib sets to its
> > > mcast mtu size.
> > >
> > > The RDMA netdev uses this value to determine if the skb length is too long
> > > for the mtu specified and if it is, drops the packet and logs an error
> > > about the errant packet.
> >
> > I'm confused by this comment, connected mode is not able to use
> > rdma_netdev, for various technical reason, I thought?
> >
> > Is this somehow running a rdma_netdev concurrently with connected
> > mode? How?
>
> No, not concurrently. When ipoib is in connected mode, a broadcast request,
> something like:
>
> ping -s 2017 -i 0.001 -c 10 -M do -I ib0 -b 192.168.0.255
>
> will be sent down from user space to ipoib. At an mcast_mtu of 2048, the max
> payload size is 2016 (2048 - 28 - 4). If AIP is not being used then the
> datagram send function (ipoib_send()) does a check and drops the packet.
>
> However when AIP is enabled ipoib_send is of course not used and we land in
> rn->send function. Which needs to do the same check.
You just contradicted yourself: the first sentence was 'not
concurrently' and here you say we have connected mode turned on and
yet a packet is delivered to AIP, so what do you mean?
What I mean is if you can do connected mode you don't have a
rdma_netdev and you can't do AIP.
How are things in connected mode and a rdma_netdev is available?
Jason
next prev parent reply other threads:[~2020-02-21 23:32 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-10 13:18 [PATCH for-next 00/16] New hfi1 feature: Accelerated IP Dennis Dalessandro
2020-02-10 13:18 ` [PATCH for-next 01/16] IB/hfi1: Add accelerated IP capability bit Dennis Dalessandro
2020-02-10 13:18 ` [PATCH for-next 02/16] IB/hfi1: Add functions to transmit datagram ipoib packets Dennis Dalessandro
2020-02-10 13:18 ` [PATCH for-next 03/16] IB/hfi1: Add the transmit side of a datagram ipoib RDMA netdev Dennis Dalessandro
2020-02-10 13:18 ` [PATCH for-next 04/16] IB/hfi1: Remove module parameter for KDETH qpns Dennis Dalessandro
2020-02-10 13:18 ` [PATCH for-next 05/16] IB/{rdmavt, hfi1}: Implement creation of accelerated UD QPs Dennis Dalessandro
2020-02-10 13:18 ` [PATCH for-next 06/16] IB/hfi1: RSM rules for AIP Dennis Dalessandro
2020-02-10 13:18 ` [PATCH for-next 07/16] IB/ipoib: Increase ipoib Datagram mode MTU's upper limit Dennis Dalessandro
2020-02-19 11:01 ` Erez Shitrit
2020-02-21 19:40 ` Dennis Dalessandro
2020-02-10 13:18 ` [PATCH for-next 08/16] IB/hfi1: Rename num_vnic_contexts as num_netdev_contexts Dennis Dalessandro
2020-02-10 13:19 ` [PATCH for-next 09/16] IB/hfi1: Add functions to receive accelerated ipoib packets Dennis Dalessandro
2020-02-10 13:19 ` [PATCH for-next 10/16] IB/hfi1: Add interrupt handler functions for accelerated ipoib Dennis Dalessandro
2020-02-10 13:19 ` [PATCH for-next 11/16] IB/hfi1: Add rx functions for dummy netdev Dennis Dalessandro
2020-02-10 13:19 ` [PATCH for-next 12/16] IB/hfi1: Activate the " Dennis Dalessandro
2020-02-10 13:19 ` [PATCH for-next 13/16] IB/{hfi1, ipoib, rdma}: Broadcast ping sent packets which exceeded mtu size Dennis Dalessandro
2020-02-19 0:42 ` Jason Gunthorpe
2020-02-21 19:40 ` Dennis Dalessandro
2020-02-21 23:32 ` Jason Gunthorpe [this message]
2020-03-20 13:53 ` Dennis Dalessandro
2020-02-19 13:41 ` Erez Shitrit
2020-02-21 19:40 ` Dennis Dalessandro
2020-02-10 13:19 ` [PATCH for-next 14/16] IB/hfi1: Add packet histogram trace event Dennis Dalessandro
2020-02-10 13:19 ` [PATCH for-next 15/16] IB/ipoib: Add capability to switch between datagram and connected mode Dennis Dalessandro
2020-02-10 13:20 ` [PATCH for-next 16/16] IB/hfi1: Enable the transmit side of the datagram ipoib netdev Dennis Dalessandro
2020-02-10 13:31 ` [PATCH for-next 00/16] New hfi1 feature: Accelerated IP Jason Gunthorpe
2020-02-10 17:36 ` Dennis Dalessandro
2020-02-10 18:32 ` Jason Gunthorpe
2020-02-11 21:58 ` Dennis Dalessandro
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=20200221233249.GM31668@ziepe.ca \
--to=jgg@ziepe.ca \
--cc=Gary.S.Leshner@intel.com \
--cc=dennis.alessandro@intel.com \
--cc=dennis.dalessandro@intel.com \
--cc=dledford@redhat.com \
--cc=kaike.wan@intel.com \
--cc=linux-rdma@vger.kernel.org \
--cc=mike.marciniszyn@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).