Linux-Next Archive on lore.kernel.org
 help / color / Atom feed
From: Ben Dooks <ben@fluff.org>
To: Mark Brown <broonie@kernel.org>
Cc: Linus Walleij <linus.walleij@linaro.org>,
	Robin Murphy <robin.murphy@arm.com>,
	Naresh Kamboju <naresh.kamboju@linaro.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	Stephen Rothwell <sfr@canb.auug.org.au>,
	Arnd Bergmann <arnd@arndb.de>,
	open list <linux-kernel@vger.kernel.org>,
	lkft-triage@lists.linaro.org,
	John Stultz <john.stultz@linaro.org>,
	Linux-Next Mailing List <linux-next@vger.kernel.org>
Subject: Re: Linux-next-20191106 : arm64: Internal error: Oops: 96000007
Date: Thu, 7 Nov 2019 11:59:54 +0000
Message-ID: <20191107115954.7sfjflcxwejcolpb@hetzy.fluff.org> (raw)
In-Reply-To: <20191107114553.GA6159@sirena.co.uk>

On Thu, Nov 07, 2019 at 11:45:53AM +0000, Mark Brown wrote:
> On Thu, Nov 07, 2019 at 09:00:25AM +0100, Linus Walleij wrote:
> 
> > I wonder if it's worth to look at the static checkers like checkpatch
> > to warn for this?
> 
> I noticed this due to the warning emitted in the build process so I'm
> not sure there'd be much more chance that people would notice (though
> admittedly the warning during the build doesn't suggest a boot problem).
> There's hundreds of other examples in the tree that don't trigger
> crashes for whatever reason, I think this one was just noticable because
> it got built in.

Is it worth hving a attribute for 'can't be initdata' to add to sparse to
catch things like this. Not sure if checkpatch would be able to do the sort
of full level of catch for this.



-- 
Ben Dooks, ben@fluff.org, http://www.fluff.org/ben/

Large Hadron Colada: A large Pina Colada that makes the universe disappear.


  reply index

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-06 15:56 Naresh Kamboju
2019-11-06 16:07 ` Robin Murphy
2019-11-06 16:16   ` Mark Rutland
2019-11-06 16:17   ` Mark Brown
2019-11-07  8:00     ` Linus Walleij
2019-11-07 11:45       ` Mark Brown
2019-11-07 11:59         ` Ben Dooks [this message]
2019-11-06 17:19 ` Russell King - ARM Linux admin

Reply instructions:

You may reply publically 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=20191107115954.7sfjflcxwejcolpb@hetzy.fluff.org \
    --to=ben@fluff.org \
    --cc=arnd@arndb.de \
    --cc=broonie@kernel.org \
    --cc=catalin.marinas@arm.com \
    --cc=john.stultz@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-next@vger.kernel.org \
    --cc=lkft-triage@lists.linaro.org \
    --cc=naresh.kamboju@linaro.org \
    --cc=robin.murphy@arm.com \
    --cc=sfr@canb.auug.org.au \
    --cc=will@kernel.org \
    /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

Linux-Next Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-next/0 linux-next/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-next linux-next/ https://lore.kernel.org/linux-next \
		linux-next@vger.kernel.org
	public-inbox-index linux-next

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-next


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git