From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EDD9870 for ; Wed, 19 May 2021 12:23:56 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 0886860FE8; Wed, 19 May 2021 12:23:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1621427036; bh=qf5m2Nia+4DoUNrww0bmk5cQtJEm5cWAINMR8yRdz/A=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=YfCZvauDRbYUmSAlgtaMrkiKYI1UYEyinYzXVIPR7xRh1CtNh0+9OVD9OrNltg37F ahwtG1OzCfm+TPgAP2KCQ/seEJB4ce2EP9qVSocsEdEVaEDy5NMlr35rgUFUsut7E7 UvuvRtU/TJAcTlN9lW3pCF7Im0grQPZmNY7SFmII= Date: Wed, 19 May 2021 14:23:54 +0200 From: Greg Kroah-Hartman To: Paolo Bonzini Cc: Anup Patel , Anup Patel , Palmer Dabbelt , Palmer Dabbelt , Paul Walmsley , Albert Ou , Jonathan Corbet , Alexander Graf , Atish Patra , Alistair Francis , Damien Le Moal , KVM General , kvm-riscv@lists.infradead.org, linux-riscv , linux-doc@vger.kernel.org, "linux-kernel@vger.kernel.org List" , linux-staging@lists.linux.dev Subject: Re: [PATCH v18 00/18] KVM RISC-V Support Message-ID: References: <20210519033553.1110536-1-anup.patel@wdc.com> X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Wed, May 19, 2021 at 01:18:44PM +0200, Paolo Bonzini wrote: > On 19/05/21 12:47, Greg Kroah-Hartman wrote: > > > It is not a dumping ground for stuff that arch maintainers can not seem > > > to agree on, and it is not a place where you can just randomly play > > > around with user/kernel apis with no consequences. > > > > > > So no, sorry, not going to take this code at all. > > > > And to be a bit more clear about this, having other subsystem > > maintainers drop their unwanted code on this subsystem,_without_ even > > asking me first is just not very nice. All of a sudden I am now > responsible for this stuff, without me even being asked about it. > > Should I start throwing random drivers into the kvm subsystem for them > > to maintain because I don't want to?:) > > (I did see the smiley), I'm on board with throwing random drivers in > arch/riscv. :) > > The situation here didn't seem very far from what process/2.Process.rst says > about staging: > > - "a way to keep track of drivers that aren't up to standards", though in > this case the issue is not coding standards or quality---the code is very > good---and which people "may want to use" Exactly, this is different. And it's not self-contained, which is another requirement for staging code that we have learned to enforce over the years (makes it easier to rip out if no one is willing to maintain it.) > - the code could be removed if there's no progress on either changing the > RISC-V acceptance policy or ratifying the spec I really do not understand the issue here, why can this just not be merged normally? Is the code somehow not ok? Is it relying on hardware in ways that breaks other users? Does it cause problems for different users? Is it a user api that you don't like or think is the "proper" one? All staging drivers need a TODO list that shows what needs to be done in order to get it out of staging. All I can tell so far is that the riscv maintainers do not want to take this for "unknown reasons" so let's dump it over here for now where we don't have to see it. And that's not good for developers or users, so perhaps the riscv rules are not very good? > Of course there should have been a TODO file explaining the situation. But > if you think this is not the right place, I totally understand; if my > opinion had any weight in this, I would just place it in arch/riscv/kvm. > > The RISC-V acceptance policy as is just doesn't work, and the fact that > people are trying to work around it is proving it. There are many ways to > improve it: What is this magical acceptance policy that is preventing working code from being merged? And why is it suddenly the rest of the kernel developer's problems because of this? thanks, greg k-h