* [dpdk-dev] Shared compilation issue - mempool/octeontx: add build and log infrastructure
@ 2017-10-14 6:07 Hemant Agrawal
2017-10-14 15:27 ` santosh
0 siblings, 1 reply; 5+ messages in thread
From: Hemant Agrawal @ 2017-10-14 6:07 UTC (permalink / raw)
To: Santosh Shukla, olivier.matz, dev; +Cc: thomas, jerin.jacob
Santosh/jerin,
Your patch "mempool/octeontx: add build and log infrastructure" is causing shared build failure for NXP's patch "event/dpaa2: support event eth adapter", which is still under review.
http://dpdk.org/dev/patchwork/patch/30248/
In your patch, you have added mempool to be dependent on event, which is not correct.
"+DEPDIRS-mempool := bus event"
Ideally, if you have common code, you should put that in bus or mempool . This way you can make event, net or sec etc dependent on bus or mempool and not the otherway around.
With the addition of ethernet adapter in the eventdev, eventdev drivers have dependency on the net driver.
e.g dpaa2 event dev is dependent on dpaa2 net driver, but your patch causes cyclic dependency.
You may also face this issue, once you implement the ethernet adapter support in eventdev.
Please look into this issue.
I suggest you to move the common code to mempool itself or to bus.
Regards,
Hemant
diff --git a/drivers/Makefile b/drivers/Makefile
index 7fef66d..bc2d082 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -32,13 +32,13 @@
include $(RTE_SDK)/mk/rte.vars.mk
DIRS-y += bus
+DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
+DEPDIRS-event := bus
DIRS-y += mempool
-DEPDIRS-mempool := bus
+DEPDIRS-mempool := bus event
DIRS-y += net
DEPDIRS-net := bus mempool
DIRS-$(CONFIG_RTE_LIBRTE_CRYPTODEV) += crypto
DEPDIRS-crypto := mempool
-DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
-DEPDIRS-event := bus
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [dpdk-dev] Shared compilation issue - mempool/octeontx: add build and log infrastructure
2017-10-14 6:07 [dpdk-dev] Shared compilation issue - mempool/octeontx: add build and log infrastructure Hemant Agrawal
@ 2017-10-14 15:27 ` santosh
2017-10-16 6:48 ` Hemant Agrawal
0 siblings, 1 reply; 5+ messages in thread
From: santosh @ 2017-10-14 15:27 UTC (permalink / raw)
To: Hemant Agrawal, olivier.matz, dev; +Cc: thomas, jerin.jacob
On Saturday 14 October 2017 11:37 AM, Hemant Agrawal wrote:
> Santosh/jerin,
> Your patch "mempool/octeontx: add build and log infrastructure" is causing shared build failure for NXP's patch "event/dpaa2: support event eth adapter", which is still under review.
> http://dpdk.org/dev/patchwork/patch/30248/
>
> In your patch, you have added mempool to be dependent on event, which is not correct.
>
> "+DEPDIRS-mempool := bus event"
Yes, Octeontx needed mbox definitions and it was defined at event area.
But You are right that adding event dependency in mempool may create issues
for future feature like you mentioned.
> Ideally, if you have common code, you should put that in bus or mempool . This way you can make event, net or sec etc dependent on bus or mempool and not the otherway around.
Right.
Proposed change for makefile will be:
diff --git a/drivers/Makefile b/drivers/Makefile
index 3a5b22342..0467250aa 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -32,13 +32,13 @@
include $(RTE_SDK)/mk/rte.vars.mk
DIRS-y += bus
-DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
-DEPDIRS-event := bus
DIRS-y += mempool
-DEPDIRS-mempool := bus event
+DEPDIRS-mempool := bus
DIRS-y += net
DEPDIRS-net := bus mempool
DIRS-$(CONFIG_RTE_LIBRTE_CRYPTODEV) += crypto
DEPDIRS-crypto := bus mempool
+DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
+DEPDIRS-event := bus mempool
include $(RTE_SDK)/mk/rte.subdir.mk
Can you give it a try and let me know your feedback.
Thanks.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [dpdk-dev] Shared compilation issue - mempool/octeontx: add build and log infrastructure
2017-10-14 15:27 ` santosh
@ 2017-10-16 6:48 ` Hemant Agrawal
2017-10-16 6:50 ` santosh
0 siblings, 1 reply; 5+ messages in thread
From: Hemant Agrawal @ 2017-10-16 6:48 UTC (permalink / raw)
To: santosh, olivier.matz, dev; +Cc: thomas, jerin.jacob
Hi Santosh,
On 10/14/2017 8:57 PM, santosh wrote:
>
> On Saturday 14 October 2017 11:37 AM, Hemant Agrawal wrote:
>> Santosh/jerin,
>> Your patch "mempool/octeontx: add build and log infrastructure" is causing shared build failure for NXP's patch "event/dpaa2: support event eth adapter", which is still under review.
>> http://dpdk.org/dev/patchwork/patch/30248/
>>
>> In your patch, you have added mempool to be dependent on event, which is not correct.
>>
>> "+DEPDIRS-mempool := bus event"
>
> Yes, Octeontx needed mbox definitions and it was defined at event area.
>
> But You are right that adding event dependency in mempool may create issues
> for future feature like you mentioned.
>
>> Ideally, if you have common code, you should put that in bus or mempool . This way you can make event, net or sec etc dependent on bus or mempool and not the otherway around.
>
> Right.
> Proposed change for makefile will be:
>
> diff --git a/drivers/Makefile b/drivers/Makefile
> index 3a5b22342..0467250aa 100644
> --- a/drivers/Makefile
> +++ b/drivers/Makefile
> @@ -32,13 +32,13 @@
> include $(RTE_SDK)/mk/rte.vars.mk
>
> DIRS-y += bus
> -DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
> -DEPDIRS-event := bus
> DIRS-y += mempool
> -DEPDIRS-mempool := bus event
> +DEPDIRS-mempool := bus
> DIRS-y += net
> DEPDIRS-net := bus mempool
> DIRS-$(CONFIG_RTE_LIBRTE_CRYPTODEV) += crypto
> DEPDIRS-crypto := bus mempool
> +DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
> +DEPDIRS-event := bus mempool
>
> include $(RTE_SDK)/mk/rte.subdir.mk
>
> Can you give it a try and let me know your feedback.
>
This change breaks the octonetx compilation.
You may also have to move the the common code from event to mempool.
Regards,
Hemant
> Thanks.
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [dpdk-dev] Shared compilation issue - mempool/octeontx: add build and log infrastructure
2017-10-16 6:48 ` Hemant Agrawal
@ 2017-10-16 6:50 ` santosh
2017-10-16 13:47 ` santosh
0 siblings, 1 reply; 5+ messages in thread
From: santosh @ 2017-10-16 6:50 UTC (permalink / raw)
To: Hemant Agrawal, olivier.matz, dev; +Cc: thomas, jerin.jacob
On Monday 16 October 2017 12:18 PM, Hemant Agrawal wrote:
> Hi Santosh,
>
> On 10/14/2017 8:57 PM, santosh wrote:
>>
>> On Saturday 14 October 2017 11:37 AM, Hemant Agrawal wrote:
>>> Santosh/jerin,
>>> Your patch "mempool/octeontx: add build and log infrastructure" is causing shared build failure for NXP's patch "event/dpaa2: support event eth adapter", which is still under review.
>>> http://dpdk.org/dev/patchwork/patch/30248/
>>>
>>> In your patch, you have added mempool to be dependent on event, which is not correct.
>>>
>>> "+DEPDIRS-mempool := bus event"
>>
>> Yes, Octeontx needed mbox definitions and it was defined at event area.
>>
>> But You are right that adding event dependency in mempool may create issues
>> for future feature like you mentioned.
>>
>>> Ideally, if you have common code, you should put that in bus or mempool . This way you can make event, net or sec etc dependent on bus or mempool and not the otherway around.
>>
>> Right.
>> Proposed change for makefile will be:
>>
>> diff --git a/drivers/Makefile b/drivers/Makefile
>> index 3a5b22342..0467250aa 100644
>> --- a/drivers/Makefile
>> +++ b/drivers/Makefile
>> @@ -32,13 +32,13 @@
>> include $(RTE_SDK)/mk/rte.vars.mk
>>
>> DIRS-y += bus
>> -DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
>> -DEPDIRS-event := bus
>> DIRS-y += mempool
>> -DEPDIRS-mempool := bus event
>> +DEPDIRS-mempool := bus
>> DIRS-y += net
>> DEPDIRS-net := bus mempool
>> DIRS-$(CONFIG_RTE_LIBRTE_CRYPTODEV) += crypto
>> DEPDIRS-crypto := bus mempool
>> +DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
>> +DEPDIRS-event := bus mempool
>>
>> include $(RTE_SDK)/mk/rte.subdir.mk
>>
>> Can you give it a try and let me know your feedback.
>>
>
> This change breaks the octonetx compilation.
> You may also have to move the the common code from event to mempool.
>
Yes, I missed mentioning that part in earlier mail.
We'll post series soon.
Thanks.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [dpdk-dev] Shared compilation issue - mempool/octeontx: add build and log infrastructure
2017-10-16 6:50 ` santosh
@ 2017-10-16 13:47 ` santosh
0 siblings, 0 replies; 5+ messages in thread
From: santosh @ 2017-10-16 13:47 UTC (permalink / raw)
To: Hemant Agrawal, olivier.matz, dev; +Cc: thomas, jerin.jacob
Hi Hemant,
On Monday 16 October 2017 12:20 PM, santosh wrote:
> On Monday 16 October 2017 12:18 PM, Hemant Agrawal wrote:
>> Hi Santosh,
>>
>> On 10/14/2017 8:57 PM, santosh wrote:
>>> On Saturday 14 October 2017 11:37 AM, Hemant Agrawal wrote:
>>>> Santosh/jerin,
>>>> Your patch "mempool/octeontx: add build and log infrastructure" is causing shared build failure for NXP's patch "event/dpaa2: support event eth adapter", which is still under review.
>>>> http://dpdk.org/dev/patchwork/patch/30248/
>>>>
>>>> In your patch, you have added mempool to be dependent on event, which is not correct.
>>>>
>>>> "+DEPDIRS-mempool := bus event"
>>> Yes, Octeontx needed mbox definitions and it was defined at event area.
>>>
>>> But You are right that adding event dependency in mempool may create issues
>>> for future feature like you mentioned.
>>>
>>>> Ideally, if you have common code, you should put that in bus or mempool . This way you can make event, net or sec etc dependent on bus or mempool and not the otherway around.
>>> Right.
>>> Proposed change for makefile will be:
>>>
>>> diff --git a/drivers/Makefile b/drivers/Makefile
>>> index 3a5b22342..0467250aa 100644
>>> --- a/drivers/Makefile
>>> +++ b/drivers/Makefile
>>> @@ -32,13 +32,13 @@
>>> include $(RTE_SDK)/mk/rte.vars.mk
>>>
>>> DIRS-y += bus
>>> -DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
>>> -DEPDIRS-event := bus
>>> DIRS-y += mempool
>>> -DEPDIRS-mempool := bus event
>>> +DEPDIRS-mempool := bus
>>> DIRS-y += net
>>> DEPDIRS-net := bus mempool
>>> DIRS-$(CONFIG_RTE_LIBRTE_CRYPTODEV) += crypto
>>> DEPDIRS-crypto := bus mempool
>>> +DIRS-$(CONFIG_RTE_LIBRTE_EVENTDEV) += event
>>> +DEPDIRS-event := bus mempool
>>>
>>> include $(RTE_SDK)/mk/rte.subdir.mk
>>>
>>> Can you give it a try and let me know your feedback.
>>>
>> This change breaks the octonetx compilation.
>> You may also have to move the the common code from event to mempool.
>>
> Yes, I missed mentioning that part in earlier mail.
>
> We'll post series soon.
>
> Thanks.
Sent fix [1] and [2]. Pl. test with your series.
Thanks.
[1] http://dpdk.org/dev/patchwork/patch/30430/
[2] http://dpdk.org/dev/patchwork/patch/30431/
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-10-16 13:47 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-14 6:07 [dpdk-dev] Shared compilation issue - mempool/octeontx: add build and log infrastructure Hemant Agrawal
2017-10-14 15:27 ` santosh
2017-10-16 6:48 ` Hemant Agrawal
2017-10-16 6:50 ` santosh
2017-10-16 13:47 ` santosh
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).