[dpdk-dev] [PATCH] examples/ptpclient: delete wrong comments

[dpdk-dev] [PATCH] examples/ptpclient: delete wrong comments

[Min Hu (Connor)]

This patch deletes the comments which are wrong and unnecessary.

Fixes: ab129e9065a5 ("examples/ptpclient: add minimal PTP client")
Cc: stable@dpdk.org

Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
---
 examples/ptpclient/ptpclient.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/examples/ptpclient/ptpclient.c b/examples/ptpclient/ptpclient.c
index 33b297e..878d1a0 100644
--- a/examples/ptpclient/ptpclient.c
+++ b/examples/ptpclient/ptpclient.c
@@ -606,10 +606,6 @@ lcore_main(void)
 	unsigned nb_rx;
 	struct rte_mbuf *m;
 
-	/*
-	 * Check that the port is on the same NUMA node as the polling thread
-	 * for best performance.
-	 */
 	printf("\nCore %u Waiting for SYNC packets. [Ctrl+C to quit]\n",
 			rte_lcore_id());
 
-- 
2.7.4

Re: [dpdk-dev] [PATCH] examples/ptpclient: delete wrong comments

[Min Hu (Connor)]

Hi, Ferruh, kirill and all,
	Any comments about this patch?

在 2021/3/27 15:36, Min Hu (Connor) 写道:
> This patch deletes the comments which are wrong and unnecessary.
> 
> Fixes: ab129e9065a5 ("examples/ptpclient: add minimal PTP client")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
> ---
>   examples/ptpclient/ptpclient.c | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/examples/ptpclient/ptpclient.c b/examples/ptpclient/ptpclient.c
> index 33b297e..878d1a0 100644
> --- a/examples/ptpclient/ptpclient.c
> +++ b/examples/ptpclient/ptpclient.c
> @@ -606,10 +606,6 @@ lcore_main(void)
>   	unsigned nb_rx;
>   	struct rte_mbuf *m;
>   
> -	/*
> -	 * Check that the port is on the same NUMA node as the polling thread
> -	 * for best performance.
> -	 */
>   	printf("\nCore %u Waiting for SYNC packets. [Ctrl+C to quit]\n",
>   			rte_lcore_id());
>   
> 

Re: [dpdk-dev] [PATCH] examples/ptpclient: delete wrong comments

[Ferruh Yigit]

On 4/10/2021 10:11 AM, Min Hu (Connor) wrote:
> Hi, Ferruh, kirill and all,
>      Any comments about this patch?
> 
> 在 2021/3/27 15:36, Min Hu (Connor) 写道:
>> This patch deletes the comments which are wrong and unnecessary.
>>
>> Fixes: ab129e9065a5 ("examples/ptpclient: add minimal PTP client")
>> Cc: stable@dpdk.org
>>
>> Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
>> ---
>>   examples/ptpclient/ptpclient.c | 4 ----
>>   1 file changed, 4 deletions(-)
>>
>> diff --git a/examples/ptpclient/ptpclient.c b/examples/ptpclient/ptpclient.c
>> index 33b297e..878d1a0 100644
>> --- a/examples/ptpclient/ptpclient.c
>> +++ b/examples/ptpclient/ptpclient.c
>> @@ -606,10 +606,6 @@ lcore_main(void)
>>       unsigned nb_rx;
>>       struct rte_mbuf *m;
>> -    /*
>> -     * Check that the port is on the same NUMA node as the polling thread
>> -     * for best performance.
>> -     */
>>       printf("\nCore %u Waiting for SYNC packets. [Ctrl+C to quit]\n",
>>               rte_lcore_id());
>>

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

The comment seems left over from the example this example cloned from..

Re: [dpdk-dev] [PATCH] examples/ptpclient: delete wrong comments

[Min Hu (Connor)]

Hi, Ferruh and all,
	If  no other comments, could your please merge it into the
branch？
	Thanks.

在 2021/4/12 16:04, Ferruh Yigit 写道:
> On 4/10/2021 10:11 AM, Min Hu (Connor) wrote:
>> Hi, Ferruh, kirill and all,
>>      Any comments about this patch?
>>
>> 在 2021/3/27 15:36, Min Hu (Connor) 写道:
>>> This patch deletes the comments which are wrong and unnecessary.
>>>
>>> Fixes: ab129e9065a5 ("examples/ptpclient: add minimal PTP client")
>>> Cc: stable@dpdk.org
>>>
>>> Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
>>> ---
>>>   examples/ptpclient/ptpclient.c | 4 ----
>>>   1 file changed, 4 deletions(-)
>>>
>>> diff --git a/examples/ptpclient/ptpclient.c 
>>> b/examples/ptpclient/ptpclient.c
>>> index 33b297e..878d1a0 100644
>>> --- a/examples/ptpclient/ptpclient.c
>>> +++ b/examples/ptpclient/ptpclient.c
>>> @@ -606,10 +606,6 @@ lcore_main(void)
>>>       unsigned nb_rx;
>>>       struct rte_mbuf *m;
>>> -    /*
>>> -     * Check that the port is on the same NUMA node as the polling 
>>> thread
>>> -     * for best performance.
>>> -     */
>>>       printf("\nCore %u Waiting for SYNC packets. [Ctrl+C to quit]\n",
>>>               rte_lcore_id());
>>>
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 
> The comment seems left over from the example this example cloned from..
> .

Re: [dpdk-dev] [PATCH] examples/ptpclient: delete wrong comments

[Thomas Monjalon]

12/04/2021 10:04, Ferruh Yigit:
> On 4/10/2021 10:11 AM, Min Hu (Connor) wrote:
> >> This patch deletes the comments which are wrong and unnecessary.
> >>
> >> Fixes: ab129e9065a5 ("examples/ptpclient: add minimal PTP client")
> >> Cc: stable@dpdk.org
> >>
> >> Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
> >> ---
> >> --- a/examples/ptpclient/ptpclient.c
> >> +++ b/examples/ptpclient/ptpclient.c
> >> -    /*
> >> -     * Check that the port is on the same NUMA node as the polling thread
> >> -     * for best performance.
> >> -     */
> >>       printf("\nCore %u Waiting for SYNC packets. [Ctrl+C to quit]\n",
> >>               rte_lcore_id());
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 
> The comment seems left over from the example this example cloned from..

Applied, thanks