Re: [ovs-dev] [PATCH V10 02/33] tc: Introduce tc module

[Roi Dayan]

On 08/06/2017 17:30, Simon Horman wrote:

On Thu, Jun 08, 2017 at 02:46:19PM +0300, Roi Dayan wrote:

Add tc module to expose tc operations to be used by other modules.
Move some tc related functions from netdev-linux.c to tc.c
This patch doesn't change any functionality.

Signed-off-by: Paul Blakey 
Co-authored-by: Roi Dayan 


Hi Roi,

as your name appears in the From field (as the author) I think
that Paul's name rather than yours should be in the Co-authored-by tag.
If you agree please consider responding to this email with the correct tag.


Signed-off-by: Roi Dayan 
Acked-by: Joe Stringer 
Acked-by: Flavio Leitner 


...



Hi,

In this case Paul should be the author as he did most of the work in
this new module while I did some updates afterwards.
The author change is maybe because of the work of splitting the commit.
Does this block? should I submit again with fixed author?
or replying with a tag should suffice?

From: Paul Blakey 

Thanks,
Roi
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH V10 02/33] tc: Introduce tc module

[Simon Horman]

On Thu, Jun 08, 2017 at 02:46:19PM +0300, Roi Dayan wrote:
> Add tc module to expose tc operations to be used by other modules.
> Move some tc related functions from netdev-linux.c to tc.c
> This patch doesn't change any functionality.
> 
> Signed-off-by: Paul Blakey 
> Co-authored-by: Roi Dayan 

Hi Roi,

as your name appears in the From field (as the author) I think
that Paul's name rather than yours should be in the Co-authored-by tag.
If you agree please consider responding to this email with the correct tag.

> Signed-off-by: Roi Dayan 
> Acked-by: Joe Stringer 
> Acked-by: Flavio Leitner 

...
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev