From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cong Wang Subject: Re: [PATCH net] ipv6: no need to return rt->dst.error if it is not null entry. Date: Mon, 24 Jul 2017 12:57:43 -0700 Message-ID: References: <1500562286-14312-1-git-send-email-liuhangbin@gmail.com> <20170724030907.GC2938@leo.usersys.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: network dev , Roopa Prabhu To: Hangbin Liu Return-path: Received: from mail-wm0-f52.google.com ([74.125.82.52]:36244 "EHLO mail-wm0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752087AbdGXT6F (ORCPT ); Mon, 24 Jul 2017 15:58:05 -0400 Received: by mail-wm0-f52.google.com with SMTP id t201so28325095wmt.1 for ; Mon, 24 Jul 2017 12:58:04 -0700 (PDT) In-Reply-To: <20170724030907.GC2938@leo.usersys.redhat.com> Sender: netdev-owner@vger.kernel.org List-ID: On Sun, Jul 23, 2017 at 8:09 PM, Hangbin Liu wrote: > Do we still need this net->ipv6.ip6_null_entry check? How about remove all > the checks? I believe you only need to check for rt->dst.error, no need to check against NULL or ip6_null_entry. Take a look at other ip6_route_lookup() callers.