Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Date:   Sat, 4 Nov 2023 14:28:37 +0100
From:   Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To:     Thomas =?iso-8859-1?Q?Wei=DFschuh?= <linux@weissschuh.net>
Cc:     Arnd Bergmann <arnd@arndb.de>, linux-kernel@vger.kernel.org,
        Zhangjin Wu <falcon@tinylab.org>, Willy Tarreau <w@1wt.eu>,
        Yuan Tan <tanyuan@tinylab.org>
Subject: Re: [PATCH RFC] misc/pvpanic: add support for normal shutdowns
Message-ID: <2023110431-pacemaker-pruning-0e4c@gregkh>
References: <20231104-pvpanic-shutdown-v1-1-5ee7c9b3e301@weissschuh.net>
 <2023110407-unselect-uptight-b96d@gregkh>
 <365bbe1f-5ee8-40fe-bec0-53d9e7395c18@t-8ch.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <365bbe1f-5ee8-40fe-bec0-53d9e7395c18@t-8ch.de>
Precedence: bulk

On Sat, Nov 04, 2023 at 02:16:53PM +0100, Thomas Wei?schuh wrote:
> On 2023-11-04 14:05:02+0100, Greg Kroah-Hartman wrote:
> > On Sat, Nov 04, 2023 at 12:29:30PM +0100, Thomas Wei?schuh wrote:
> > > Shutdown requests are normally hardware dependent.
> > > By extending pvpanic to also handle shutdown requests, guests can
> > > submit such requests with an easily implementable and cross-platform
> > > mechanism.
> > > 
> > > Signed-off-by: Thomas Wei?schuh <linux@weissschuh.net>
> > > ---
> > > The corresponding patch to qemu has also been submitted[0].
> > > General discussions about the feature should happen on the other thread.
> > > 
> > > [0] https://lore.kernel.org/qemu-devel/20231104-pvpanic-shutdown-v1-0-02353157891b@t-8ch.de/
> > > ---
> > >  drivers/misc/pvpanic/pvpanic.c | 19 +++++++++++++++++--
> > >  include/uapi/misc/pvpanic.h    |  1 +
> > >  2 files changed, 18 insertions(+), 2 deletions(-)
> 
> [..]
> 
> > > diff --git a/include/uapi/misc/pvpanic.h b/include/uapi/misc/pvpanic.h
> > > index 54b7485390d3..82fc618bfbcf 100644
> > > --- a/include/uapi/misc/pvpanic.h
> > > +++ b/include/uapi/misc/pvpanic.h
> > > @@ -5,5 +5,6 @@
> > >  
> > >  #define PVPANIC_PANICKED	(1 << 0)
> > >  #define PVPANIC_CRASH_LOADED	(1 << 1)
> > > +#define PVPANIC_SHUTDOWN	(1 << 2)
> > 
> > Why are these in a uapi file?
> 
> They are ABI between qemu and its guest.

But there's no interaction between Linux and userspace for these values,
so I would just drop them from here.

> The specification for these values is part of qemu but for some reason
> the header is part of Linux which is then imported back into qemu.
> 
> I guess this has historical reasons, maybe because qemu doesn't really
> ship ABI headers and for Linux it's natural.

That feels odd, are there other in-kernel examples of the Linux uapi
files being abused like this?

> The real reason probably doesn't matter today as the header propably
> can't be dropped from Linux anyways for compatibility reasons.
> 
> > And if they need to be here, why not use the proper BIT() macro for it?
> 
> This was for uniformity with the existing code.
> I can send a (standalone?) patch to fix it up.

If we keep it, sure, that would be nice.  But let's try to drop it if
possible :)

thanks,

greg k-h