[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/1] kernel.h: Split out panic and oops helpers
- To: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
- From: Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx>
- Date: Sat, 10 Apr 2021 00:39:11 +0200
- Cc: Alexander Lobakin <alobakin@xxxxx>, Wei Liu <wei.liu@xxxxxxxxxx>, Rasmus Villemoes <linux@xxxxxxxxxxxxxxxxxx>, Arnd Bergmann <arnd@xxxxxxxx>, Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>, Christian Brauner <christian.brauner@xxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Kees Cook <keescook@xxxxxxxxxxxx>, Mike Rapoport <rppt@xxxxxxxxxx>, Corey Minyard <cminyard@xxxxxxxxxx>, Michael Ellerman <mpe@xxxxxxxxxxxxxx>, Vasily Gorbik <gor@xxxxxxxxxxxxx>, "Jason J. Herne" <jjherne@xxxxxxxxxxxxx>, Joerg Roedel <jroedel@xxxxxxx>, Michael Kelley <mikelley@xxxxxxxxxxxxx>, Joe Perches <joe@xxxxxxxxxxx>, Florian Fainelli <f.fainelli@xxxxxxxxx>, Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Scott Branden <scott.branden@xxxxxxxxxxxx>, Olof Johansson <olof@xxxxxxxxx>, Mihai Carabas <mihai.carabas@xxxxxxxxxx>, Wang Wenhu <wenhu.wang@xxxxxxxx>, Marek Czerski <ma.czerski@xxxxxxxxx>, Hongbo Yao <yaohongbo@xxxxxxxxxx>, Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>, Vineeth Vijayan <vneethv@xxxxxxxxxxxxx>, Heiko Carstens <hca@xxxxxxxxxxxxx>, Peter Oberparleiter <oberpar@xxxxxxxxxxxxx>, Alexander Egorenkov <egorenar@xxxxxxxxxxxxx>, Tetsuo Handa <penguin-kernel@xxxxxxxxxxxxxxxxxxx>, Vlastimil Babka <vbabka@xxxxxxx>, "Paul E. McKenney" <paulmck@xxxxxxxxxx>, "Steven Rostedt (VMware)" <rostedt@xxxxxxxxxxx>, linux-alpha@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, linux-mips@xxxxxxxxxxxxxxx, linux-parisc@xxxxxxxxxxxxxxx, linuxppc-dev@xxxxxxxxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx, sparclinux@xxxxxxxxxxxxxxx, linux-um@xxxxxxxxxxxxxxxxxxx, linux-hyperv@xxxxxxxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxxx, linux-xtensa@xxxxxxxxxxxxxxxx, openipmi-developer@xxxxxxxxxxxxxxxxxxxxx, linux-clk@xxxxxxxxxxxxxxx, linux-edac@xxxxxxxxxxxxxxx, coresight@xxxxxxxxxxxxxxxx, linux-leds@xxxxxxxxxxxxxxx, bcm-kernel-feedback-list@xxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, linux-pm@xxxxxxxxxxxxxxx, linux-remoteproc@xxxxxxxxxxxxxxx, linux-staging@xxxxxxxxxxxxxxx, dri-devel@xxxxxxxxxxxxxxxxxxxxx, linux-fbdev@xxxxxxxxxxxxxxx, linux-arch@xxxxxxxxxxxxxxx, kexec@xxxxxxxxxxxxxxxxxxx, rcu@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, Richard Henderson <rth@xxxxxxxxxxx>, Ivan Kokshaysky <ink@xxxxxxxxxxxxxxxxxxxx>, Matt Turner <mattst88@xxxxxxxxx>, Catalin Marinas <catalin.marinas@xxxxxxx>, Will Deacon <will@xxxxxxxxxx>, "James E.J. Bottomley" <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx>, Helge Deller <deller@xxxxxx>, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>, Paul Mackerras <paulus@xxxxxxxxx>, Christian Borntraeger <borntraeger@xxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, Jeff Dike <jdike@xxxxxxxxxxx>, Richard Weinberger <richard@xxxxxx>, Anton Ivanov <anton.ivanov@xxxxxxxxxxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, Borislav Petkov <bp@xxxxxxxxx>, x86@xxxxxxxxxx, "H. Peter Anvin" <hpa@xxxxxxxxx>, "K. Y. Srinivasan" <kys@xxxxxxxxxxxxx>, Haiyang Zhang <haiyangz@xxxxxxxxxxxxx>, Stephen Hemminger <sthemmin@xxxxxxxxxxxxx>, Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Chris Zankel <chris@xxxxxxxxxx>, Max Filippov <jcmvbkbc@xxxxxxxxx>, Corey Minyard <minyard@xxxxxxx>, Paul Walmsley <paul.walmsley@xxxxxxxxxx>, Michael Turquette <mturquette@xxxxxxxxxxxx>, Stephen Boyd <sboyd@xxxxxxxxxx>, Dinh Nguyen <dinguyen@xxxxxxxxxx>, Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>, Tony Luck <tony.luck@xxxxxxxxx>, James Morse <james.morse@xxxxxxx>, Robert Richter <rric@xxxxxxxxxx>, Suzuki K Poulose <suzuki.poulose@xxxxxxx>, Mike Leach <mike.leach@xxxxxxxxxx>, Leo Yan <leo.yan@xxxxxxxxxx>, Alexander Shishkin <alexander.shishkin@xxxxxxxxxxxxxxx>, Pavel Machek <pavel@xxxxxx>, Alex Elder <elder@xxxxxxxxxx>, Jakub Kicinski <kuba@xxxxxxxxxx>, Sebastian Reichel <sre@xxxxxxxxxx>, Ohad Ben-Cohen <ohad@xxxxxxxxxx>, Jens Frederich <jfrederich@xxxxxxxxx>, Daniel Drake <dsd@xxxxxxxxxx>, Jon Nettleton <jon.nettleton@xxxxxxxxx>, Eric Biederman <ebiederm@xxxxxxxxxxxx>, Josh Triplett <josh@xxxxxxxxxxxxxxxx>, Mathieu Desnoyers <mathieu.desnoyers@xxxxxxxxxxxx>, Lai Jiangshan <jiangshanlai@xxxxxxxxx>, Joel Fernandes <joel@xxxxxxxxxxxxxxxxx>, Luis Chamberlain <mcgrof@xxxxxxxxxx>, Iurii Zaikin <yzaikin@xxxxxxxxxx>, Mike Rapoport <rppt@xxxxxxxxxxxxx>
- Delivery-date: Sat, 10 Apr 2021 10:13:01 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On Fri, Apr 09, 2021 at 01:02:50PM +0300, Andy Shevchenko wrote:
> kernel.h is being used as a dump for all kinds of stuff for a long time.
> Here is the attempt to start cleaning it up by splitting out panic and
> oops helpers.
>
> There are several purposes of doing this:
> - dropping dependency in bug.h
> - dropping a loop by moving out panic_notifier.h
> - unload kernel.h from something which has its own domain
>
> At the same time convert users tree-wide to use new headers, although
> for the time being include new header back to kernel.h to avoid twisted
> indirected includes for existing users.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
> Acked-by: Mike Rapoport <rppt@xxxxxxxxxxxxx>
> Acked-by: Corey Minyard <cminyard@xxxxxxxxxx>
> Acked-by: Christian Brauner <christian.brauner@xxxxxxxxxx>
> Acked-by: Arnd Bergmann <arnd@xxxxxxxx>
> Acked-by: Kees Cook <keescook@xxxxxxxxxxxx>
> Acked-by: Wei Liu <wei.liu@xxxxxxxxxx>
> Acked-by: Rasmus Villemoes <linux@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> ---
> v2:
> - fixed all errors with allmodconfig on x86_64 (Andrew)
> - checked with allyesconfig on x86_64
> - additionally grepped source code for panic notifier list usage
> and converted all users
> - elaborated commit message (Luis)
> - collected given tags (incl. Andrew's SoB, see below)
>
> I added Andrew's SoB since part of the fixes I took from him. Andrew,
> feel free to amend or tell me how you want me to do.
>
> arch/mips/kernel/relocate.c | 1 +
> arch/mips/sgi-ip22/ip22-reset.c | 1 +
> arch/mips/sgi-ip32/ip32-reset.c | 1 +
Acked-by: Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx>
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]
|