examples/ptpclient: delete wrong comments
Checks
Commit Message
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(-)
Comments
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());
>
>
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..
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..
> .
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
@@ -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());