qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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
>
>

  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).