From mboxrd@z Thu Jan 1 00:00:00 1970 From: Veaceslav Falico Subject: Re: [PATCH net-next 0/3] bonding: slight optimization and cleanup Date: Wed, 7 May 2014 15:34:19 +0200 Message-ID: <20140507133419.GS6295@redhat.com> References: <1399434319-7300-1-git-send-email-dingtianhong@huawei.com> <20140507132758.GQ6295@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Cc: fubar@us.ibm.com, andy@greyhouse.net, davem@davemloft.net, netdev@vger.kernel.org To: Ding Tianhong Return-path: Received: from mx1.redhat.com ([209.132.183.28]:2212 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932919AbaEGNek (ORCPT ); Wed, 7 May 2014 09:34:40 -0400 Content-Disposition: inline In-Reply-To: <20140507132758.GQ6295@redhat.com> Sender: netdev-owner@vger.kernel.org List-ID: On Wed, May 07, 2014 at 03:27:58PM +0200, Veaceslav Falico wrote: >On Wed, May 07, 2014 at 11:45:16AM +0800, Ding Tianhong wrote: >>Ding Tianhong (3): >> bonding: remove the unnecessary struct bond_net >> bonding: simplify the slave_do_arp_validate_only() >> bonding: remove the unused macro > > >Except that remark about first patch - looks good to me. Also, please update Maintainer's email addresses, get them from MAINTAINERS file. Jay's bounces, mine still not :) but I'm filtering them out. > >> >>drivers/net/bonding/bond_main.c | 4 ++-- >>drivers/net/bonding/bonding.h | 23 +++++++++-------------- >>2 files changed, 11 insertions(+), 16 deletions(-) >> >>-- >>1.8.0 >> >>