From: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
To: Yuval Shaia <yuval.shaia@oracle.com>,
dmitry.fleytman@gmail.com, jasowang@redhat.com,
eblake@redhat.com, armbru@redhat.com, pbonzini@redhat.com,
qemu-devel@nongnu.org, shamir.rabinovitch@oracle.com,
cohuck@redhat.com
Subject: Re: [Qemu-devel] [PATCH v5 24/24] docs: Update pvrdma device documentation
Date: Mon, 26 Nov 2018 12:34:41 +0200 [thread overview]
Message-ID: <25d71e93-c1ab-01c3-8f6c-145606d11a84@gmail.com> (raw)
In-Reply-To: <8b89bfaf-be29-e043-32fa-9615fb4ea0f7@gmail.com>
Re-sending the comments, some of the recipients didn't get it,
Thanks,
Marcel
On 11/25/18 9:51 AM, Marcel Apfelbaum wrote:
>
>
> On 11/22/18 2:14 PM, Yuval Shaia wrote:
>> Interface with the device is changed with the addition of support for
>> MAD packets.
>> Adjust documentation accordingly.
>>
>> While there fix a minor mistake which may lead to think that there is a
>> relation between using RXE on host and the compatibility with bare-metal
>> peers.
>>
>> Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
>> ---
>> docs/pvrdma.txt | 103 +++++++++++++++++++++++++++++++++++++++---------
>> 1 file changed, 84 insertions(+), 19 deletions(-)
>>
>> diff --git a/docs/pvrdma.txt b/docs/pvrdma.txt
>> index 5599318159..f82b2a69d2 100644
>> --- a/docs/pvrdma.txt
>> +++ b/docs/pvrdma.txt
>> @@ -9,8 +9,9 @@ It works with its Linux Kernel driver AS IS, no need
>> for any special guest
>> modifications.
>> While it complies with the VMware device, it can also communicate
>> with bare
>> -metal RDMA-enabled machines and does not require an RDMA HCA in the
>> host, it
>> -can work with Soft-RoCE (rxe).
>> +metal RDMA-enabled machines as peers.
>> +
>> +It does not require an RDMA HCA in the host, it can work with
>> Soft-RoCE (rxe).
>> It does not require the whole guest RAM to be pinned allowing memory
>> over-commit and, even if not implemented yet, migration support
>> will be
>> @@ -78,29 +79,93 @@ the required RDMA libraries.
>> 3. Usage
>> ========
>> +
>> +
>> +3.1 VM Memory settings
>> +======+++=============
>> Currently the device is working only with memory backed RAM
>> and it must be mark as "shared":
>> -m 1G \
>> -object memory-backend-ram,id=mb1,size=1G,share \
>> -numa node,memdev=mb1 \
>> -The pvrdma device is composed of two functions:
>> - - Function 0 is a vmxnet Ethernet Device which is redundant in Guest
>> - but is required to pass the ibdevice GID using its MAC.
>> - Examples:
>> - For an rxe backend using eth0 interface it will use its mac:
>> - -device vmxnet3,addr=<slot>.0,multifunction=on,mac=<eth0 MAC>
>> - For an SRIOV VF, we take the Ethernet Interface exposed by it:
>> - -device vmxnet3,multifunction=on,mac=<RoCE eth MAC>
>> - - Function 1 is the actual device:
>> - -device
>> pvrdma,addr=<slot>.1,backend-dev=<ibdevice>,backend-gid-idx=<gid>,backend-port=<port>
>> - where the ibdevice can be rxe or RDMA VF (e.g. mlx5_4)
>> - Note: Pay special attention that the GID at backend-gid-idx matches
>> vmxnet's MAC.
>> - The rules of conversion are part of the RoCE spec, but since manual
>> conversion
>> - is not required, spotting problems is not hard:
>> - Example: GID: fe80:0000:0000:0000:7efe:90ff:fecb:743a
>> - MAC: 7c:fe:90:cb:74:3a
>> - Note the difference between the first byte of the MAC and the GID.
>> +
>> +3.2 MAD Multiplexer
>> +===================
>> +MAD Multiplexer is a service that exposes MAD-like interface for VMs in
>> +order to overcome the limitation where only single entity can
>> register with
>> +MAD layer to send and receive RDMA-CM MAD packets.
>> +
>> +To build rdmacm-mux run
>> +# make rdmacm-mux
>> +
>> +The application accepts 3 command line arguments and exposes a UNIX
>> socket
>> +to pass control and data to it.
>> +-s unix-socket-path Path to unix socket to listen on
>> + (default /var/run/rdmacm-mux)
>> +-d rdma-device-name Name of RDMA device to register with
>> + (default rxe0)
>
> I would not default it to rxe0, but request to specify a RDMA interface.
> One can think the multiplexer may select the best available device
> and finish with an rxe instance instead of a bare-metal one...
>
>> +-p rdma-device-port Port number of RDMA device to register with
>> + (default 1)
>> +The final UNIX socket file name is a concatenation of the 3
>> arguments so
>> +for example for device mlx5_0 on port 2 this
>> /var/run/rdmacm-mux-mlx5_0-2
>> +will be created.
>> +
>> +Please refer to contrib/rdmacm-mux for more details.
>> +
>> +
>> +3.3 PCI devices settings
>> +========================
>> +RoCE device exposes two functions - an Ethernet and RDMA.
>> +To support it, pvrdma device is composed of two PCI functions, an
>> Ethernet
>> +device of type vmxnet3 on PCI slot 0 and a PVRDMA device on PCI slot
>> 1. The
>> +Ethernet function can be used for other Ethernet purposes such as IP.
>
> Nice !
>
>> +
>> +
>> +3.4 Device parameters
>> +=====================
>> +- netdev: Specifies the Ethernet device on host. For Soft-RoCE (rxe)
>> this
>> + would be the Ethernet device used to create it. For any other
>> physical
>> + RoCE device this would be the netdev name of the device.
>
> I don't fully understand the above explanation. Can you elaborate
> or give an exmaple?
>
>> +- ibdev: The IB device name on host for example rxe0, mlx5_0 etc.
>> +- mad-chardev: The name of the MAD multiplexer char device.
>> +- ibport: In case of multi-port device (such as Mellanox's HCA) this
>> + specify the port to use. If not set 1 will be used.
>> +- dev-caps-max-mr-size: The maximum size of MR.
>> +- dev-caps-max-qp: Maximum number of QPs.
>> +- dev-caps-max-sge: Maximum number of SGE elements in WR.
>> +- dev-caps-max-cq: Maximum number of CQs.
>> +- dev-caps-max-mr: Maximum number of MRs.
>> +- dev-caps-max-pd: Maximum number of PDs.
>> +- dev-caps-max-ah: Maximum number of AHs.
>> +
>> +Notes:
>> +- The first 3 parameters are mandatory settings, the rest have their
>> + defaults.
>> +- The last 8 parameters (the ones that prefixed by dev-caps) defines
>> the top
>> + limits but the final values is adjusted by the backend device
>> limitations.
>> +
>> +3.5 Example
>> +===========
>> +Define bridge device with vmxnet3 network backend:
>> +<interface type='bridge'>
>> + <mac address='56:b4:44:e9:62:dc'/>
>> + <source bridge='bridge1'/>
>> + <model type='vmxnet3'/>
>> + <address type='pci' domain='0x0000' bus='0x00' slot='0x10'
>> function='0x0' multifunction='on'/>
>> +</interface>
>> +
>> +Define pvrdma device:
>> +<qemu:commandline>
>> + <qemu:arg value='-object'/>
>> + <qemu:arg value='memory-backend-ram,id=mb1,size=1G,share'/>
>> + <qemu:arg value='-numa'/>
>> + <qemu:arg value='node,memdev=mb1'/>
>> + <qemu:arg value='-chardev'/>
>> + <qemu:arg value='socket,path=/var/run/rdmacm-mux-rxe0-1,id=mads'/>
>> + <qemu:arg value='-device'/>
>> + <qemu:arg
>> value='pvrdma,addr=10.1,ibdev=rxe0,netdev=bridge0,mad-chardev=mads'/>
>> +</qemu:commandline>
>
> Please be sure to emphasize that the pvrdma works only
> if the QEMU is operated by libvirt. The same about the multiplexer.
>
> Thanks,
> Marcel
>
>
next prev parent reply other threads:[~2018-11-26 10:34 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-22 12:13 [Qemu-devel] [PATCH v5 00/24] Add support for RDMA MAD Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 01/24] contrib/rdmacm-mux: Add implementation of RDMA User MAD multiplexer Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 02/24] hw/rdma: Add ability to force notification without re-arm Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 03/24] hw/rdma: Return qpn 1 if ibqp is NULL Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 04/24] hw/rdma: Abort send-op if fail to create addr handler Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 05/24] hw/rdma: Add support for MAD packets Yuval Shaia
2018-11-25 7:05 ` Marcel Apfelbaum
2018-11-25 7:27 ` Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 06/24] hw/pvrdma: Make function reset_device return void Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 07/24] hw/pvrdma: Make default pkey 0xFFFF Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 08/24] hw/pvrdma: Set the correct opcode for recv completion Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 09/24] hw/pvrdma: Set the correct opcode for send completion Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 10/24] qapi: Define new QMP message for pvrdma Yuval Shaia
2018-11-26 10:01 ` Markus Armbruster
2018-11-26 10:08 ` Yuval Shaia
2018-11-26 20:43 ` Eric Blake
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 11/24] hw/pvrdma: Add support to allow guest to configure GID table Yuval Shaia
2018-11-25 7:29 ` Marcel Apfelbaum
2018-11-25 9:10 ` Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 12/24] vmxnet3: Move some definitions to header file Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 13/24] hw/pvrdma: Make sure PCI function 0 is vmxnet3 Yuval Shaia
2018-11-25 7:31 ` Marcel Apfelbaum
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 14/24] hw/rdma: Initialize node_guid from vmxnet3 mac address Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 15/24] hw/pvrdma: Make device state depend on Ethernet function state Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 16/24] hw/pvrdma: Fill all CQE fields Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 17/24] hw/pvrdma: Fill error code in command's response Yuval Shaia
2018-11-25 7:40 ` Marcel Apfelbaum
2018-11-25 11:53 ` Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 18/24] hw/rdma: Remove unneeded code that handles more that one port Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 19/24] vl: Introduce shutdown_notifiers Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 20/24] hw/pvrdma: Clean device's resource when system is shutdown Yuval Shaia
2018-11-22 12:13 ` [Qemu-devel] [PATCH v5 21/24] hw/rdma: Do not use bitmap_zero_extend to free bitmap Yuval Shaia
2018-11-22 12:14 ` [Qemu-devel] [PATCH v5 22/24] hw/rdma: Do not call rdma_backend_del_gid on an empty gid Yuval Shaia
2018-11-22 12:14 ` [Qemu-devel] [PATCH v5 23/24] hw/pvrdma: Do not clean resources on shutdown Yuval Shaia
2018-11-25 7:30 ` Yuval Shaia
2018-11-25 7:41 ` Marcel Apfelbaum
2018-11-22 12:14 ` [Qemu-devel] [PATCH v5 24/24] docs: Update pvrdma device documentation Yuval Shaia
[not found] ` <8b89bfaf-be29-e043-32fa-9615fb4ea0f7@gmail.com>
2018-11-26 10:34 ` Marcel Apfelbaum [this message]
2018-11-26 13:05 ` Yuval Shaia
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=25d71e93-c1ab-01c3-8f6c-145606d11a84@gmail.com \
--to=marcel.apfelbaum@gmail.com \
--cc=armbru@redhat.com \
--cc=cohuck@redhat.com \
--cc=dmitry.fleytman@gmail.com \
--cc=eblake@redhat.com \
--cc=jasowang@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=shamir.rabinovitch@oracle.com \
--cc=yuval.shaia@oracle.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).