DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
@ 2020-04-27  6:11 asomalap
  2020-04-27  6:14 ` Kumar, Ravi1
  0 siblings, 1 reply; 9+ messages in thread
From: asomalap @ 2020-04-27  6:11 UTC (permalink / raw)
  To: dev; +Cc: Ravi1.Kumar

From: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>

Ownership change.

Signed-off-by: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
---
 MAINTAINERS | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index d31a80929..c3dcedbb3 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -566,7 +566,7 @@ F: doc/guides/nics/ena.rst
 F: doc/guides/nics/features/ena.ini
 
 AMD axgbe
-M: Ravi Kumar <ravi1.kumar@amd.com>
+M: Somalapuram Amaranath <asomalap@amd.com>
 F: drivers/net/axgbe/
 F: doc/guides/nics/axgbe.rst
 F: doc/guides/nics/features/axgbe.ini
@@ -953,7 +953,7 @@ T: git://dpdk.org/next/dpdk-next-crypto
 F: doc/guides/cryptodevs/features/default.ini
 
 AMD CCP Crypto
-M: Ravi Kumar <ravi1.kumar@amd.com>
+M: Somalapuram Amaranath <asomalap@amd.com>
 F: drivers/crypto/ccp/
 F: doc/guides/cryptodevs/ccp.rst
 F: doc/guides/cryptodevs/features/ccp.ini
-- 
2.17.1


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
  2020-04-27  6:11 [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto asomalap
@ 2020-04-27  6:14 ` Kumar, Ravi1
  2020-04-27 10:58   ` Ferruh Yigit
  0 siblings, 1 reply; 9+ messages in thread
From: Kumar, Ravi1 @ 2020-04-27  6:14 UTC (permalink / raw)
  To: Somalapuram, Amaranath, dev

[AMD Public Use]

Acked-by: Ravi Kumar <ravi1.kumar@amd.com>
>-----Original Message-----
>From: Somalapuram, Amaranath <Amaranath.Somalapuram@amd.com> 
>Sent: Monday, April 27, 2020 11:42 AM
>To: dev@dpdk.org
>Cc: Kumar, Ravi1 <Ravi1.Kumar@amd.com>
>Subject: [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
>
>From: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>
>Ownership change.
>
>Signed-off-by: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>---
> MAINTAINERS | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/MAINTAINERS b/MAINTAINERS
>index d31a80929..c3dcedbb3 100644
>--- a/MAINTAINERS
>+++ b/MAINTAINERS
>@@ -566,7 +566,7 @@ F: doc/guides/nics/ena.rst
> F: doc/guides/nics/features/ena.ini
> 
> AMD axgbe
>-M: Ravi Kumar <ravi1.kumar@amd.com>
>+M: Somalapuram Amaranath <asomalap@amd.com>
> F: drivers/net/axgbe/
> F: doc/guides/nics/axgbe.rst
> F: doc/guides/nics/features/axgbe.ini
>@@ -953,7 +953,7 @@ T: git://dpdk.org/next/dpdk-next-crypto
> F: doc/guides/cryptodevs/features/default.ini
> 
> AMD CCP Crypto
>-M: Ravi Kumar <ravi1.kumar@amd.com>
>+M: Somalapuram Amaranath <asomalap@amd.com>
> F: drivers/crypto/ccp/
> F: doc/guides/cryptodevs/ccp.rst
> F: doc/guides/cryptodevs/features/ccp.ini
>-- 
>2.17.1
>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
  2020-04-27  6:14 ` Kumar, Ravi1
@ 2020-04-27 10:58   ` Ferruh Yigit
  2020-04-27 11:24     ` Somalapuram, Amaranath
  2020-04-27 20:05     ` Thomas Monjalon
  0 siblings, 2 replies; 9+ messages in thread
From: Ferruh Yigit @ 2020-04-27 10:58 UTC (permalink / raw)
  To: Kumar, Ravi1, Somalapuram, Amaranath, dev

On 4/27/2020 7:14 AM, Kumar, Ravi1 wrote:
> [AMD Public Use]
> 
> Acked-by: Ravi Kumar <ravi1.kumar@amd.com>
>> -----Original Message-----
>> From: Somalapuram, Amaranath <Amaranath.Somalapuram@amd.com> 
>> Sent: Monday, April 27, 2020 11:42 AM
>> To: dev@dpdk.org
>> Cc: Kumar, Ravi1 <Ravi1.Kumar@amd.com>
>> Subject: [PATCH v1] maintainers: update for AMD xgbe and ccp crypto

I guess net driver is called 'axgbe', good to fix while merging.

>>
>> From: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>>
>> Ownership change.
>>
>> Signed-off-by: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>> ---
>> MAINTAINERS | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index d31a80929..c3dcedbb3 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -566,7 +566,7 @@ F: doc/guides/nics/ena.rst
>> F: doc/guides/nics/features/ena.ini
>>
>> AMD axgbe
>> -M: Ravi Kumar <ravi1.kumar@amd.com>
>> +M: Somalapuram Amaranath <asomalap@amd.com>
>> F: drivers/net/axgbe/
>> F: doc/guides/nics/axgbe.rst
>> F: doc/guides/nics/features/axgbe.ini
>> @@ -953,7 +953,7 @@ T: git://dpdk.org/next/dpdk-next-crypto
>> F: doc/guides/cryptodevs/features/default.ini

Thanks Ravi, Welcome Somalapuram.

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

>>
>> AMD CCP Crypto
>> -M: Ravi Kumar <ravi1.kumar@amd.com>
>> +M: Somalapuram Amaranath <asomalap@amd.com>
>> F: drivers/crypto/ccp/
>> F: doc/guides/cryptodevs/ccp.rst
>> F: doc/guides/cryptodevs/features/ccp.ini
>> -- 
>> 2.17.1
>>


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
  2020-04-27 10:58   ` Ferruh Yigit
@ 2020-04-27 11:24     ` Somalapuram, Amaranath
  2020-04-27 12:07       ` Ferruh Yigit
  2020-04-27 20:05     ` Thomas Monjalon
  1 sibling, 1 reply; 9+ messages in thread
From: Somalapuram, Amaranath @ 2020-04-27 11:24 UTC (permalink / raw)
  To: Ferruh Yigit, Kumar, Ravi1, dev

Hi Ferruh,

Do you want me to resubmit. 

I see this below errors email:

From: robot@bytheb.org

Test-Label: travis-robot
Test-Status:
https://nam11.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdpdk.org%2Fpatch%2F69359&amp;data=02%7C01%7CAmaranath.Somalapuram%40amd.com%7C3c14b94fb68e492d035608d7ea93e6a2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637235793258422338&amp;sdata=WcqAv8%2BRhMq4z0YRZ9S09NuV8x7kjgc2TtX3jmvJlzA%3D&amp;reserved=0

_Travis build: errored_
Build URL: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-ci.com%2Fovsrobot%2Fdpdk%2Fbuilds%2F162163735&amp;data=02%7C01%7CAmaranath.Somalapuram%40amd.com%7C3c14b94fb68e492d035608d7ea93e6a2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637235793258422338&amp;sdata=plABOXG44ijGqIo5CHCHWerbirVOlf5hujOFiWkysb4%3D&amp;reserved=0

Regards,
S.Amarnath

-----Original Message-----
From: Ferruh Yigit <ferruh.yigit@intel.com> 
Sent: Monday, April 27, 2020 4:29 PM
To: Kumar, Ravi1 <Ravi1.Kumar@amd.com>; Somalapuram, Amaranath <Amaranath.Somalapuram@amd.com>; dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto

[CAUTION: External Email]

On 4/27/2020 7:14 AM, Kumar, Ravi1 wrote:
> [AMD Public Use]
>
> Acked-by: Ravi Kumar <ravi1.kumar@amd.com>
>> -----Original Message-----
>> From: Somalapuram, Amaranath <Amaranath.Somalapuram@amd.com>
>> Sent: Monday, April 27, 2020 11:42 AM
>> To: dev@dpdk.org
>> Cc: Kumar, Ravi1 <Ravi1.Kumar@amd.com>
>> Subject: [PATCH v1] maintainers: update for AMD xgbe and ccp crypto

I guess net driver is called 'axgbe', good to fix while merging.

>>
>> From: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>>
>> Ownership change.
>>
>> Signed-off-by: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>> ---
>> MAINTAINERS | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index d31a80929..c3dcedbb3 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -566,7 +566,7 @@ F: doc/guides/nics/ena.rst
>> F: doc/guides/nics/features/ena.ini
>>
>> AMD axgbe
>> -M: Ravi Kumar <ravi1.kumar@amd.com>
>> +M: Somalapuram Amaranath <asomalap@amd.com>
>> F: drivers/net/axgbe/
>> F: doc/guides/nics/axgbe.rst
>> F: doc/guides/nics/features/axgbe.ini
>> @@ -953,7 +953,7 @@ T: git://dpdk.org/next/dpdk-next-crypto
>> F: doc/guides/cryptodevs/features/default.ini

Thanks Ravi, Welcome Somalapuram.

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

>>
>> AMD CCP Crypto
>> -M: Ravi Kumar <ravi1.kumar@amd.com>
>> +M: Somalapuram Amaranath <asomalap@amd.com>
>> F: drivers/crypto/ccp/
>> F: doc/guides/cryptodevs/ccp.rst
>> F: doc/guides/cryptodevs/features/ccp.ini
>> --
>> 2.17.1
>>


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
  2020-04-27 11:24     ` Somalapuram, Amaranath
@ 2020-04-27 12:07       ` Ferruh Yigit
  2020-04-27 12:58         ` Aaron Conole
  0 siblings, 1 reply; 9+ messages in thread
From: Ferruh Yigit @ 2020-04-27 12:07 UTC (permalink / raw)
  To: Somalapuram, Amaranath, Kumar, Ravi1, dev, David Marchand, Aaron Conole
  Cc: Raslan Darawsheh, Ali Alnubani, Thomas Monjalon, dpdklab

On 4/27/2020 12:24 PM, Somalapuram, Amaranath wrote:
> Hi Ferruh,
> 
> Do you want me to resubmit. 

Hi Amaranath,

No need to send a new version.

> 
> I see this below errors email:
> 
> From: robot@bytheb.org
> 
> Test-Label: travis-robot
> Test-Status:
> https://nam11.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdpdk.org%2Fpatch%2F69359&amp;data=02%7C01%7CAmaranath.Somalapuram%40amd.com%7C3c14b94fb68e492d035608d7ea93e6a2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637235793258422338&amp;sdata=WcqAv8%2BRhMq4z0YRZ9S09NuV8x7kjgc2TtX3jmvJlzA%3D&amp;reserved=0
> 
> _Travis build: errored_
> Build URL: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-ci.com%2Fovsrobot%2Fdpdk%2Fbuilds%2F162163735&amp;data=02%7C01%7CAmaranath.Somalapuram%40amd.com%7C3c14b94fb68e492d035608d7ea93e6a2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637235793258422338&amp;sdata=plABOXG44ijGqIo5CHCHWerbirVOlf5hujOFiWkysb4%3D&amp;reserved=0

The travis error seems unrelated [1], cc'ed David and Aaron for it.

[1] Two issues reported
a) ninja: build stopped: Error writing to build log: Disk quota exceeded.
b) No output has been received in the last 10m0s, this potentially indicates a
stalled build or something wrong with the build itself.


Also community lab tests seems failing because of the mellanox performance
report, but this patch is not something that can affect the performance.
Cc'ed Raslan & Ali and lab for it.
https://lab.dpdk.org/results/dashboard/patchsets/10717/

> 
> Regards,
> S.Amarnath
> 
> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit@intel.com> 
> Sent: Monday, April 27, 2020 4:29 PM
> To: Kumar, Ravi1 <Ravi1.Kumar@amd.com>; Somalapuram, Amaranath <Amaranath.Somalapuram@amd.com>; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
> 
> [CAUTION: External Email]
> 
> On 4/27/2020 7:14 AM, Kumar, Ravi1 wrote:
>> [AMD Public Use]
>>
>> Acked-by: Ravi Kumar <ravi1.kumar@amd.com>
>>> -----Original Message-----
>>> From: Somalapuram, Amaranath <Amaranath.Somalapuram@amd.com>
>>> Sent: Monday, April 27, 2020 11:42 AM
>>> To: dev@dpdk.org
>>> Cc: Kumar, Ravi1 <Ravi1.Kumar@amd.com>
>>> Subject: [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
> 
> I guess net driver is called 'axgbe', good to fix while merging.
> 
>>>
>>> From: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>>>
>>> Ownership change.
>>>
>>> Signed-off-by: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>>> ---
>>> MAINTAINERS | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index d31a80929..c3dcedbb3 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -566,7 +566,7 @@ F: doc/guides/nics/ena.rst
>>> F: doc/guides/nics/features/ena.ini
>>>
>>> AMD axgbe
>>> -M: Ravi Kumar <ravi1.kumar@amd.com>
>>> +M: Somalapuram Amaranath <asomalap@amd.com>
>>> F: drivers/net/axgbe/
>>> F: doc/guides/nics/axgbe.rst
>>> F: doc/guides/nics/features/axgbe.ini
>>> @@ -953,7 +953,7 @@ T: git://dpdk.org/next/dpdk-next-crypto
>>> F: doc/guides/cryptodevs/features/default.ini
> 
> Thanks Ravi, Welcome Somalapuram.
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 
>>>
>>> AMD CCP Crypto
>>> -M: Ravi Kumar <ravi1.kumar@amd.com>
>>> +M: Somalapuram Amaranath <asomalap@amd.com>
>>> F: drivers/crypto/ccp/
>>> F: doc/guides/cryptodevs/ccp.rst
>>> F: doc/guides/cryptodevs/features/ccp.ini
>>> --
>>> 2.17.1
>>>
> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
  2020-04-27 12:07       ` Ferruh Yigit
@ 2020-04-27 12:58         ` Aaron Conole
  2020-04-27 15:00           ` Thomas Monjalon
  0 siblings, 1 reply; 9+ messages in thread
From: Aaron Conole @ 2020-04-27 12:58 UTC (permalink / raw)
  To: Ferruh Yigit
  Cc: Somalapuram, Amaranath, Kumar, Ravi1, dev, David Marchand,
	Raslan Darawsheh, Ali Alnubani, Thomas Monjalon, dpdklab

Ferruh Yigit <ferruh.yigit@intel.com> writes:

> On 4/27/2020 12:24 PM, Somalapuram, Amaranath wrote:
>> Hi Ferruh,
>> 
>> Do you want me to resubmit. 
>
> Hi Amaranath,
>
> No need to send a new version.
>
>> 
>> I see this below errors email:
>> 
>> From: robot@bytheb.org
>> 
>> Test-Label: travis-robot
>> Test-Status:
>> https://nam11.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdpdk.org%2Fpatch%2F69359&amp;data=02%7C01%7CAmaranath.Somalapuram%40amd.com%7C3c14b94fb68e492d035608d7ea93e6a2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637235793258422338&amp;sdata=WcqAv8%2BRhMq4z0YRZ9S09NuV8x7kjgc2TtX3jmvJlzA%3D&amp;reserved=0
>> 
>> _Travis build: errored_
>> Build URL:
>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-ci.com%2Fovsrobot%2Fdpdk%2Fbuilds%2F162163735&amp;data=02%7C01%7CAmaranath.Somalapuram%40amd.com%7C3c14b94fb68e492d035608d7ea93e6a2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637235793258422338&amp;sdata=plABOXG44ijGqIo5CHCHWerbirVOlf5hujOFiWkysb4%3D&amp;reserved=0
>
> The travis error seems unrelated [1], cc'ed David and Aaron for it.

I agree.

> [1] Two issues reported
> a) ninja: build stopped: Error writing to build log: Disk quota
> exceeded.

This occurs on some of the ARM64 builds.  Travis is aware of the issue,
but don't seem urgently fixing it. :-/  I think we have a series
outstanding that should be applied to drop those builds for now.

> b) No output has been received in the last 10m0s, this potentially indicates a
> stalled build or something wrong with the build itself.

This is a future enhancement I need to work on.  When a build has jobs
that are stalled / errored, we need to restart them (at least once).  Or
otherwise flag it.  It especially happens when internal travis
infrastructure fails.

>
> Also community lab tests seems failing because of the mellanox performance
> report, but this patch is not something that can affect the performance.
> Cc'ed Raslan & Ali and lab for it.
> https://lab.dpdk.org/results/dashboard/patchsets/10717/
>
>> 
>> Regards,
>> S.Amarnath
>> 
>> -----Original Message-----
>> From: Ferruh Yigit <ferruh.yigit@intel.com> 
>> Sent: Monday, April 27, 2020 4:29 PM
>> To: Kumar, Ravi1 <Ravi1.Kumar@amd.com>; Somalapuram, Amaranath
>> <Amaranath.Somalapuram@amd.com>; dev@dpdk.org
>> Subject: Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
>> 
>> [CAUTION: External Email]
>> 
>> On 4/27/2020 7:14 AM, Kumar, Ravi1 wrote:
>>> [AMD Public Use]
>>>
>>> Acked-by: Ravi Kumar <ravi1.kumar@amd.com>
>>>> -----Original Message-----
>>>> From: Somalapuram, Amaranath <Amaranath.Somalapuram@amd.com>
>>>> Sent: Monday, April 27, 2020 11:42 AM
>>>> To: dev@dpdk.org
>>>> Cc: Kumar, Ravi1 <Ravi1.Kumar@amd.com>
>>>> Subject: [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
>> 
>> I guess net driver is called 'axgbe', good to fix while merging.
>> 
>>>>
>>>> From: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>>>>
>>>> Ownership change.
>>>>
>>>> Signed-off-by: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
>>>> ---
>>>> MAINTAINERS | 4 ++--
>>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index d31a80929..c3dcedbb3 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -566,7 +566,7 @@ F: doc/guides/nics/ena.rst
>>>> F: doc/guides/nics/features/ena.ini
>>>>
>>>> AMD axgbe
>>>> -M: Ravi Kumar <ravi1.kumar@amd.com>
>>>> +M: Somalapuram Amaranath <asomalap@amd.com>
>>>> F: drivers/net/axgbe/
>>>> F: doc/guides/nics/axgbe.rst
>>>> F: doc/guides/nics/features/axgbe.ini
>>>> @@ -953,7 +953,7 @@ T: git://dpdk.org/next/dpdk-next-crypto
>>>> F: doc/guides/cryptodevs/features/default.ini
>> 
>> Thanks Ravi, Welcome Somalapuram.
>> 
>> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
>> 
>>>>
>>>> AMD CCP Crypto
>>>> -M: Ravi Kumar <ravi1.kumar@amd.com>
>>>> +M: Somalapuram Amaranath <asomalap@amd.com>
>>>> F: drivers/crypto/ccp/
>>>> F: doc/guides/cryptodevs/ccp.rst
>>>> F: doc/guides/cryptodevs/features/ccp.ini
>>>> --
>>>> 2.17.1
>>>>
>> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
  2020-04-27 12:58         ` Aaron Conole
@ 2020-04-27 15:00           ` Thomas Monjalon
  2020-04-28 13:31             ` Aaron Conole
  0 siblings, 1 reply; 9+ messages in thread
From: Thomas Monjalon @ 2020-04-27 15:00 UTC (permalink / raw)
  To: Ferruh Yigit, Aaron Conole, David Marchand
  Cc: Somalapuram, Amaranath, Kumar, Ravi1, dev, Raslan Darawsheh,
	Ali Alnubani, dpdklab

27/04/2020 14:58, Aaron Conole:
> Ferruh Yigit <ferruh.yigit@intel.com> writes:
> > [1] Two issues reported
> > a) ninja: build stopped: Error writing to build log: Disk quota
> > exceeded.
> 
> This occurs on some of the ARM64 builds.  Travis is aware of the issue,
> but don't seem urgently fixing it. :-/  I think we have a series
> outstanding that should be applied to drop those builds for now.

I saw some "Disk quota exceeded" on x86 static builds.
So I changed my mind, I am not sure dropping only Arm from Travis
is a solution, even temporary.


> > b) No output has been received in the last 10m0s, this potentially indicates a
> > stalled build or something wrong with the build itself.
> 
> This is a future enhancement I need to work on.  When a build has jobs
> that are stalled / errored, we need to restart them (at least once).  Or
> otherwise flag it.  It especially happens when internal travis
> infrastructure fails.

Yes Travis infrastructure fails. Always.
Travis is free, but it is not a reason to provide a randomly failing tool.
I suggest switching to other platforms like OBS or our community lab,
preferring the most reliable one.



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
  2020-04-27 10:58   ` Ferruh Yigit
  2020-04-27 11:24     ` Somalapuram, Amaranath
@ 2020-04-27 20:05     ` Thomas Monjalon
  1 sibling, 0 replies; 9+ messages in thread
From: Thomas Monjalon @ 2020-04-27 20:05 UTC (permalink / raw)
  To: Somalapuram, Amaranath; +Cc: Kumar, Ravi1, dev, Ferruh Yigit

27/04/2020 12:58, Ferruh Yigit:
> On 4/27/2020 7:14 AM, Kumar, Ravi1 wrote:
> > [AMD Public Use]
> > 
> > Acked-by: Ravi Kumar <ravi1.kumar@amd.com>
> >> -----Original Message-----
> >> From: Somalapuram, Amaranath <Amaranath.Somalapuram@amd.com> 
> >> Sent: Monday, April 27, 2020 11:42 AM
> >> To: dev@dpdk.org
> >> Cc: Kumar, Ravi1 <Ravi1.Kumar@amd.com>
> >> Subject: [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
> 
> I guess net driver is called 'axgbe', good to fix while merging.

Fixed

> >>
> >> From: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>

Changed email address to match the one filled in MAINTAINERS file
and in previous contributions.

> >>
> >> Ownership change.
> >>
> >> Signed-off-by: Amaranath Somalapuram <Amaranath.Somalapuram@amd.com>
> >> ---
> >> AMD axgbe
> >> -M: Ravi Kumar <ravi1.kumar@amd.com>
> >> +M: Somalapuram Amaranath <asomalap@amd.com>
> 
> Thanks Ravi, Welcome Somalapuram.
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 
> >> AMD CCP Crypto
> >> -M: Ravi Kumar <ravi1.kumar@amd.com>
> >> +M: Somalapuram Amaranath <asomalap@amd.com>

Applied



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto
  2020-04-27 15:00           ` Thomas Monjalon
@ 2020-04-28 13:31             ` Aaron Conole
  0 siblings, 0 replies; 9+ messages in thread
From: Aaron Conole @ 2020-04-28 13:31 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: Ferruh Yigit, David Marchand, Somalapuram, Amaranath, Kumar,
	Ravi1, dev, Raslan Darawsheh, Ali Alnubani, dpdklab

Thomas Monjalon <thomas@monjalon.net> writes:

> 27/04/2020 14:58, Aaron Conole:
>> Ferruh Yigit <ferruh.yigit@intel.com> writes:
>> > [1] Two issues reported
>> > a) ninja: build stopped: Error writing to build log: Disk quota
>> > exceeded.
>> 
>> This occurs on some of the ARM64 builds.  Travis is aware of the issue,
>> but don't seem urgently fixing it. :-/  I think we have a series
>> outstanding that should be applied to drop those builds for now.
>
> I saw some "Disk quota exceeded" on x86 static builds.
> So I changed my mind, I am not sure dropping only Arm from Travis
> is a solution, even temporary.

Okay.

>
>> > b) No output has been received in the last 10m0s, this potentially indicates a
>> > stalled build or something wrong with the build itself.
>> 
>> This is a future enhancement I need to work on.  When a build has jobs
>> that are stalled / errored, we need to restart them (at least once).  Or
>> otherwise flag it.  It especially happens when internal travis
>> infrastructure fails.
>
> Yes Travis infrastructure fails. Always.
> Travis is free, but it is not a reason to provide a randomly failing tool.

It has been quite useful - it shows us that we still have some
incomplete unit tests and also some racy unit tests.  I agree, it
shouldn't be failing the way it is.

> I suggest switching to other platforms like OBS or our community lab,
> preferring the most reliable one.

OBS and community lab isn't as easy for individual developers to
integrate with.  One thing that's been useful for me is to push to my
github repo and see the results of all the build configurations.  It
doesn't require much effort from my side to ensure that code works.  I
don't always do this (but that's my bad habit), but when I do it proves
very useful.

Maybe the new github-ci (I think it's called 'actions') would be good to
investigate.  Also, Cirrus-CI lets us test on windows and freebsd.  And
there's no reason we can't use all of them.


^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-04-28 13:31 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-27  6:11 [dpdk-dev] [PATCH v1] maintainers: update for AMD xgbe and ccp crypto asomalap
2020-04-27  6:14 ` Kumar, Ravi1
2020-04-27 10:58   ` Ferruh Yigit
2020-04-27 11:24     ` Somalapuram, Amaranath
2020-04-27 12:07       ` Ferruh Yigit
2020-04-27 12:58         ` Aaron Conole
2020-04-27 15:00           ` Thomas Monjalon
2020-04-28 13:31             ` Aaron Conole
2020-04-27 20:05     ` Thomas Monjalon

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