At 2017-08-22 03:58:03, "Cong Wang" <xiyou.wangc...@gmail.com> wrote: >On Mon, Aug 21, 2017 at 10:47 AM, David Miller <da...@davemloft.net> wrote: >> From: gfree.w...@vip.163.com >> Date: Fri, 18 Aug 2017 15:23:24 +0800 >> >>> From: Gao Feng <gfree.w...@vip.163.com> >>> >>> Add the invalid handle "0" check to avoid unnecessary search, because >>> the qdisc uses the skb->priority as the handle value to look up, and >>> it is "0" usually. >>> >>> Signed-off-by: Gao Feng <gfree.w...@vip.163.com> >> >> Jamal, Cong, please review. >> >> If 'id' zero is never hashed into the tables, this change looks >> legitimate. > >Looks good to me. > >Gao, in the future please Cc maintainers directly, you can >use ./scripts/get_maintainer.pl. > >Thanks.
Hi Cong, Thanks your reminder firstly. But I had used the get_maintainer.pl actually before sent the patch. The following is the output. [fgao@ikuai8 net-next]#./scripts/get_maintainer.pl patch_ScheCheck/0001-net-sched-Add-the-invalid-handle-check-in-qdisc_clas.patch "David S. Miller" <da...@davemloft.net> (maintainer:NETWORKING [GENERAL]) netdev@vger.kernel.org (open list:NETWORKING [GENERAL]) linux-ker...@vger.kernel.org (open list) I don't know if it is an issue of the script "get_maintainer.pl" or my usage is wrong. Anyway, thanks you & Jamal's review. Best Regards Feng
