Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933240AbbLRQ7Y (ORCPT ); Fri, 18 Dec 2015 11:59:24 -0500 Received: from mail2-relais-roc.national.inria.fr ([192.134.164.83]:39084 "EHLO mail2-relais-roc.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932264AbbLRQ7W (ORCPT ); Fri, 18 Dec 2015 11:59:22 -0500 X-IronPort-AV: E=Sophos;i="5.20,447,1444687200"; d="scan'208";a="193022089" Date: Fri, 18 Dec 2015 17:59:18 +0100 (CET) From: Julia Lawall X-X-Sender: jll@hadrien To: Geliang Tang cc: "David S. Miller" , Alexey Kuznetsov , James Morris , Hideaki YOSHIFUJI , Patrick McHardy , Geliang Tang , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, kbuild-all@01.org Subject: Re: [PATCH 03/14] ipv4, ipv6: use list_for_each_entry* In-Reply-To: <201512190031.S65NXNNd%fengguang.wu@intel.com> Message-ID: References: <201512190031.S65NXNNd%fengguang.wu@intel.com> User-Agent: Alpine 2.10 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2788 Lines: 57 I don't think this code can work. list_for_each_entry uses answer to get from one element to the next. julia On Sat, 19 Dec 2015, kbuild test robot wrote: > CC: kbuild-all@01.org > In-Reply-To: <0167bba2ecf8c4fcb6b0b3135a4e957309986498.1450451516.git.geliangtang@163.com> > TO: Geliang Tang > CC: "David S. Miller" , Alexey Kuznetsov , James Morris , Hideaki YOSHIFUJI , Patrick McHardy > CC: Geliang Tang , netdev@vger.kernel.org, linux-kernel@vger.kernel.org > > Hi Geliang, > > [auto build test WARNING on net/master] > [also build test WARNING on v4.4-rc5 next-20151218] > > url: https://github.com/0day-ci/linux/commits/Geliang-Tang/Bluetooth-use-list_for_each_entry/20151218-234306 > :::::: branch date: 69 minutes ago > :::::: commit date: 69 minutes ago > > >> net/ipv6/af_inet6.c:589:1-20: iterator with update on line 597 > > git remote add linux-review https://github.com/0day-ci/linux > git remote update linux-review > git checkout c5e8d791cacac62eeec48e00a1a14a6a350670f4 > vim +589 net/ipv6/af_inet6.c > > ^1da177e Linus Torvalds 2005-04-16 583 goto out_illegal; > ^1da177e Linus Torvalds 2005-04-16 584 > ^1da177e Linus Torvalds 2005-04-16 585 /* If we are trying to override a permanent protocol, bail. */ > ^1da177e Linus Torvalds 2005-04-16 586 answer = NULL; > 87c3efbf Daniel Lezcano 2007-12-11 587 ret = -EPERM; > ^1da177e Linus Torvalds 2005-04-16 588 last_perm = &inetsw6[p->type]; > c5e8d791 Geliang Tang 2015-12-18 @589 list_for_each_entry(answer, &inetsw6[p->type], list) { > ^1da177e Linus Torvalds 2005-04-16 590 /* Check only the non-wild match. */ > ^1da177e Linus Torvalds 2005-04-16 591 if (INET_PROTOSW_PERMANENT & answer->flags) { > ^1da177e Linus Torvalds 2005-04-16 592 if (protocol == answer->protocol) > ^1da177e Linus Torvalds 2005-04-16 593 break; > c5e8d791 Geliang Tang 2015-12-18 594 last_perm = &answer->list; > ^1da177e Linus Torvalds 2005-04-16 595 } > ^1da177e Linus Torvalds 2005-04-16 596 > ^1da177e Linus Torvalds 2005-04-16 @597 answer = NULL; > ^1da177e Linus Torvalds 2005-04-16 598 } > ^1da177e Linus Torvalds 2005-04-16 599 if (answer) > ^1da177e Linus Torvalds 2005-04-16 600 goto out_permanent; > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/pipermail/kbuild-all Intel Corporation > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/