From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 86A67C4332F for ; Wed, 19 Oct 2022 13:37:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230281AbiJSNhU (ORCPT ); Wed, 19 Oct 2022 09:37:20 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56754 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231750AbiJSNgb (ORCPT ); Wed, 19 Oct 2022 09:36:31 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 27C73188106 for ; Wed, 19 Oct 2022 06:25:20 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 2F918B822E9 for ; Wed, 19 Oct 2022 10:49:57 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7F2B2C433C1; Wed, 19 Oct 2022 10:49:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1666176595; bh=Nm8juoAlBJxv8Z2ax+Ipj61nzQdsz4StDeouEjkeAoA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=i6AjiLRY+kT4BVItzL1zsT8OiDGTBhSxXLI3DCcvfvMRJJa0FsGtpx8UeNcuIiGWo R/fFXbFAp6prWcRiEvv6uG2cNewa+SGQeprGAHholT7w18B43Ipfh0JEC0mY09HSTV 4IXXJoMj2tpS1M6LmvoK0YOpeySjPJw3FjROZmqs= Date: Wed, 19 Oct 2022 12:49:53 +0200 From: Greg Kroah-Hartman To: Alexander Atanasov Cc: Nadav Amit , VMware PV-Drivers Reviewers , Arnd Bergmann , kernel@openvz.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH v5 6/8] drivers: vmware: balloon - report inflated memory Message-ID: References: <20221019095620.124909-1-alexander.atanasov@virtuozzo.com> <20221019095620.124909-7-alexander.atanasov@virtuozzo.com> <2ae8251a-456d-abdd-ed1a-1bfc04356741@virtuozzo.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2ae8251a-456d-abdd-ed1a-1bfc04356741@virtuozzo.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 19, 2022 at 01:38:13PM +0300, Alexander Atanasov wrote: > On 19.10.22 13:26, Greg Kroah-Hartman wrote: > > On Wed, Oct 19, 2022 at 12:56:18PM +0300, Alexander Atanasov wrote: > > > Update the inflated memory in the mm core on change. > > > > That says what this does, but not why it is needed. > > > > Please expand on this. > > > > Also, is this actually fixing a bug? Is it a new feature? Something > > else? > > The whole series is about adding a new feature - providing access to the > balloon inflated memory amount - it's in the cover letter. Should I repeat > it for every driver that implements it? Each commit needs to justify why it is needed on its own. You do not provide the needed information here at all to be able to review and understand if this commit is even correct or needed. thanks, greg k-h