On 2/23/2018 2:55 PM, Thomas Monjalon wrote:
23/02/2018 07:25, Hemant Agrawal:
Hi Thomas,
On 2/23/2018 4:23 AM, Thomas Monjalon wrote:
The bus and mempool dependencies should be declared after the PMD
libraries needing them.
Moreover there is no need to disable the PMDs at the Makefile level,
in case the dependencies are not met.
Such dependencies should be handled at configuration time.
The other side effect of this clean-up is to take into account
the mempool option CONFIG_RTE_LIBRTE_DPAA2_MEMPOOL.
Signed-off-by: Thomas Monjalon <tho...@monjalon.net>
---
mk/rte.app.mk | 20 ++++----------------
1 file changed, 4 insertions(+), 16 deletions(-)
This patch looks good, however I think we (NXP) need to also set some
dependency check in makefile to avoid dpaaX PMD compilations, if bus is
not available. I am working on a patch for the same.
Why do you think such check is necessary?
If the PMD is enable but not the bus, it is a configuration error.
A good configuration system would resolve it automatically,
but currently we do the configuration manually, so the user must fix
its configuration file.
I agree with your comment. That is the ideal approach.
However, just to make it convenient and to support cases like VFIO
disable, I have added few checks in my v2.
