Huazhong Tan <tanhuazh...@huawei.com> wrote: >From: Peng Li <lipeng...@huawei.com> > >Remove repeated word "that". > >Signed-off-by: Peng Li <lipeng...@huawei.com> >Signed-off-by: Huazhong Tan <tanhuazh...@huawei.com> >--- > drivers/net/bonding/bond_alb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/drivers/net/bonding/bond_alb.c b/drivers/net/bonding/bond_alb.c >index c3091e0..3455f2c 100644 >--- a/drivers/net/bonding/bond_alb.c >+++ b/drivers/net/bonding/bond_alb.c >@@ -1098,7 +1098,7 @@ static void alb_fasten_mac_swap(struct bonding *bond, >struct slave *slave1, > * If @slave's permanent hw address is different both from its current > * address and from @bond's address, then somewhere in the bond there's > * a slave that has @slave's permanet address as its current address. >- * We'll make sure that that slave no longer uses @slave's permanent address. >+ * We'll make sure that slave no longer uses @slave's permanent address.
This is actually correct as written, but I can see that it's a bit confusing. Rather than removing the second that, I'd suggest changing it to "... make sure the other slave no longer uses ..." to be clearer. -J --- -Jay Vosburgh, jay.vosbu...@canonical.com