Thanks you for your review Jerin.  See my responses are inline.

On 2020-03-20 06:24, Jerin Jacob wrote:
On Fri, Feb 21, 2020 at 4:22 AM David Wilder <dwil...@us.ibm.com> wrote:

If --no-huge is set and iova-mode has not been specified force VA mode.
If --no-huge and --iova-mode=PA is requested error out as this is
an impossible configuration.

Signed-off-by: David Wilder <dwil...@us.ibm.com>
---
 lib/librte_eal/linux/eal/eal.c | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/lib/librte_eal/linux/eal/eal.c b/lib/librte_eal/linux/eal/eal.c
index 9530ee55f..d3a0a1731 100644
--- a/lib/librte_eal/linux/eal/eal.c
+++ b/lib/librte_eal/linux/eal/eal.c
@@ -1062,9 +1062,16 @@ rte_eal_init(int argc, char **argv)

/* if no EAL option "--iova-mode=<pa|va>", use bus IOVA scheme */
        if (internal_config.iova_mode == RTE_IOVA_DC) {
+
                /* autodetect the IOVA mapping mode */
enum rte_iova_mode iova_mode = rte_bus_get_iommu_class();

+ if (iova_mode == RTE_IOVA_PA && !rte_eal_has_hugepages()) {
+                       iova_mode = RTE_IOVA_VA;


What if igb_uio or vfio_nommu has been loaded(i.e no iommu support
enabled from the driver)? This would fail.

Yes they would fail. If igb_uio or vfio_nommu (or any driver) cant be forced to VA mode it cant be used with out hugepages. Drivers can be available but not used therefor we print a warning message.


+ RTE_LOG(WARNING, EAL, "Some buses want 'PA' but forcing 'VA' because --no-huge is requested.\n"); + RTE_LOG(WARNING, EAL, "Not all buses may be able to initialize.\n");
+               }
+
                if (iova_mode == RTE_IOVA_DC) {
RTE_LOG(DEBUG, EAL, "Buses did not request a specific IOVA mode.\n");

@@ -1111,6 +1118,13 @@ rte_eal_init(int argc, char **argv)
                        internal_config.iova_mode;
        }

+       if (rte_eal_iova_mode() == RTE_IOVA_PA &&
+           rte_eal_has_hugepages() == 0) {
+ rte_eal_init_alert("Cannot use IOVA as 'PA' with --no-huge");

Top of the tree already detecting this case. am I missing anything?

[master]dell[dpdk.org] $ sudo ./build/app/test/dpdk-test  -c 0x3
--no-huge --iova-mode=pa
EAL: Detected 56 lcore(s)
EAL: Detected 2 NUMA nodes
EAL: Static memory layout is selected, amount of reserved memory can
be adjusted with -m or --socket-mem
EAL: Multi-process socket /var/run/dpdk/rte/mp_socket
EAL: FATAL: Cannot use IOVA as 'PA' since physical addresses are not available
EAL: Cannot use IOVA as 'PA' since physical addresses are not available


The check you reference is reporting that physical address are not available, for example no permissions to read /proc/self/pagemap. In this case, if --no-huge is set then PA mode is not allowed. There is no guarantee that physical address are persistent with out using hugepages.


+               rte_errno = EINVAL;
+               return -1;
+       }
+
        if (rte_eal_iova_mode() == RTE_IOVA_PA && !phys_addrs) {
rte_eal_init_alert("Cannot use IOVA as 'PA' since physical addresses are not available");
                rte_errno = EINVAL;
--
2.25.0

Reply via email to