All of lore.kernel.org
 help / color / mirror / Atom feed
From: Doug Berger <opendmb@gmail.com>
To: Jakub Kicinski <kuba@kernel.org>, kernel test robot <lkp@intel.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	ntfs3@lists.linux.dev, netdev@vger.kernel.org,
	linuxppc-dev@lists.ozlabs.org, linux-scsi@vger.kernel.org,
	linux-mm@kvack.org, linux-mediatek@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	Florian Fainelli <f.fainelli@gmail.com>
Subject: Re: [linux-next:master] BUILD REGRESSION 89bf6e28373beef9577fa71f996a5f73a569617c
Date: Tue, 25 Oct 2022 16:04:15 -0700	[thread overview]
Message-ID: <8fbc9d02-3c73-5990-85af-82eecb6d64e3@gmail.com> (raw)
In-Reply-To: <20221025154150.729bbbd0@kernel.org>

On 10/25/2022 3:41 PM, Jakub Kicinski wrote:
> On Wed, 26 Oct 2022 01:17:48 +0800 kernel test robot wrote:
>> drivers/net/ethernet/broadcom/genet/bcmgenet.c:1497:5-13: ERROR: invalid reference to the index variable of the iterator on line 1475
> 
> CC Doug
Thanks for highlighting this for me, but I happened to catch it from the 
linux-mm list and was just looking into it.

It looks to me like a false positive since I am initializing the 
loc_rule variable in all paths outside of the list_for_each_entry() loop 
prior to its use on line 1497.

If desired I can submit a new patch to make coccinelle happy.
-Doug


WARNING: multiple messages have this Message-ID (diff)
From: Doug Berger <opendmb@gmail.com>
To: Jakub Kicinski <kuba@kernel.org>, kernel test robot <lkp@intel.com>
Cc: Florian Fainelli <f.fainelli@gmail.com>,
	linux-scsi@vger.kernel.org, netdev@vger.kernel.org,
	linux-mm@kvack.org, linux-mediatek@lists.infradead.org,
	Andrew Morton <akpm@linux-foundation.org>,
	ntfs3@lists.linux.dev, linuxppc-dev@lists.ozlabs.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [linux-next:master] BUILD REGRESSION 89bf6e28373beef9577fa71f996a5f73a569617c
Date: Tue, 25 Oct 2022 16:04:15 -0700	[thread overview]
Message-ID: <8fbc9d02-3c73-5990-85af-82eecb6d64e3@gmail.com> (raw)
In-Reply-To: <20221025154150.729bbbd0@kernel.org>

On 10/25/2022 3:41 PM, Jakub Kicinski wrote:
> On Wed, 26 Oct 2022 01:17:48 +0800 kernel test robot wrote:
>> drivers/net/ethernet/broadcom/genet/bcmgenet.c:1497:5-13: ERROR: invalid reference to the index variable of the iterator on line 1475
> 
> CC Doug
Thanks for highlighting this for me, but I happened to catch it from the 
linux-mm list and was just looking into it.

It looks to me like a false positive since I am initializing the 
loc_rule variable in all paths outside of the list_for_each_entry() loop 
prior to its use on line 1497.

If desired I can submit a new patch to make coccinelle happy.
-Doug

WARNING: multiple messages have this Message-ID (diff)
From: Doug Berger <opendmb@gmail.com>
To: Jakub Kicinski <kuba@kernel.org>, kernel test robot <lkp@intel.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	ntfs3@lists.linux.dev, netdev@vger.kernel.org,
	linuxppc-dev@lists.ozlabs.org, linux-scsi@vger.kernel.org,
	linux-mm@kvack.org, linux-mediatek@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	Florian Fainelli <f.fainelli@gmail.com>
Subject: Re: [linux-next:master] BUILD REGRESSION 89bf6e28373beef9577fa71f996a5f73a569617c
Date: Tue, 25 Oct 2022 16:04:15 -0700	[thread overview]
Message-ID: <8fbc9d02-3c73-5990-85af-82eecb6d64e3@gmail.com> (raw)
In-Reply-To: <20221025154150.729bbbd0@kernel.org>

On 10/25/2022 3:41 PM, Jakub Kicinski wrote:
> On Wed, 26 Oct 2022 01:17:48 +0800 kernel test robot wrote:
>> drivers/net/ethernet/broadcom/genet/bcmgenet.c:1497:5-13: ERROR: invalid reference to the index variable of the iterator on line 1475
> 
> CC Doug
Thanks for highlighting this for me, but I happened to catch it from the 
linux-mm list and was just looking into it.

It looks to me like a false positive since I am initializing the 
loc_rule variable in all paths outside of the list_for_each_entry() loop 
prior to its use on line 1497.

If desired I can submit a new patch to make coccinelle happy.
-Doug

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2022-10-25 23:05 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-25 17:17 [linux-next:master] BUILD REGRESSION 89bf6e28373beef9577fa71f996a5f73a569617c kernel test robot
2022-10-25 17:17 ` kernel test robot
2022-10-25 17:17 ` kernel test robot
2022-10-25 22:41 ` Jakub Kicinski
2022-10-25 22:41   ` Jakub Kicinski
2022-10-25 22:41   ` Jakub Kicinski
2022-10-25 23:04   ` Doug Berger [this message]
2022-10-25 23:04     ` Doug Berger
2022-10-25 23:04     ` Doug Berger
2022-10-25 23:17     ` Jakub Kicinski
2022-10-25 23:17       ` Jakub Kicinski
2022-10-25 23:17       ` Jakub Kicinski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8fbc9d02-3c73-5990-85af-82eecb6d64e3@gmail.com \
    --to=opendmb@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=f.fainelli@gmail.com \
    --cc=kuba@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=lkp@intel.com \
    --cc=netdev@vger.kernel.org \
    --cc=ntfs3@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.