Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 20 Aug 2020 00:21:08 +0530
From:   Vaibhav Gupta <vaibhavgupta40@gmail.com>
To:     Jens Axboe <axboe@kernel.dk>
Cc:     Bjorn Helgaas <helgaas@kernel.org>,
        Bjorn Helgaas <bhelgaas@google.com>,
        Bjorn Helgaas <bjorn@helgaas.com>,
        Vaibhav Gupta <vaibhav.varodek@gmail.com>,
        Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
        Kai-Heng Feng <kai.heng.feng@canonical.com>,
        Christoph Hellwig <hch@lst.de>,
        "Gustavo A . R . Silva" <gustavoars@kernel.org>,
        Hans de Goede <hdegoede@redhat.com>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Prabhakar Kushwaha <pkushwaha@marvell.com>,
        Alex Shi <alex.shi@linux.alibaba.com>,
        =?utf-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= <mirq-linux@rere.qmqm.pl>,
        Thomas Gleixner <tglx@linutronix.de>,
        Jiri Slaby <jslaby@suse.cz>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Paul Gortmaker <paul.gortmaker@windriver.com>,
        Jia-Ju Bai <baijiaju1990@gmail.com>,
        Javier Martinez Canillas <javier@osg.samsung.com>,
        Zhang Rui <rui.zhang@intel.com>, linux-ide@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        linux-kernel-mentees@lists.linuxfoundation.org,
        Shuah Khan <skhan@linuxfoundation.org>
Subject: Re: [PATCH v3] ata: use generic power management
Message-ID: <20200819184738.GA108131@gmail.com>
References: <20200817092245.10478-1-vaibhavgupta40@gmail.com>
 <20200817092511.GA10566@gmail.com>
 <11190a0b-1009-caf8-9cbe-6cf296b614bf@kernel.dk>
 <20200818130329.GA39194@gmail.com>
 <a1937b38-055f-0fc7-0d08-16af7c42660b@kernel.dk>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <a1937b38-055f-0fc7-0d08-16af7c42660b@kernel.dk>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Aug 19, 2020 at 12:40:20PM -0600, Jens Axboe wrote:
> On 8/18/20 6:03 AM, Vaibhav Gupta wrote:
> > On Mon, Aug 17, 2020 at 07:06:03AM -0700, Jens Axboe wrote:
> >> On 8/17/20 2:25 AM, Vaibhav Gupta wrote:
> >>> Hello,
> >>>
> >>> I am working to upgrade power management framework support for PCI drivers, as
> >>> my project under the Linux Kernel Mentorship Program.
> >>>
> >>> The ultimate goal is to completely remove the legacy framework. And for this we
> >>> need to remove ".suspend" and ".resume" pointers from "struct pci_driver".
> >>>
> >>> The patch is doing the change for the same. The actual suspend() and
> >>> resume() callbacks are defined in drivers/ata/libata-core.c and are exported.
> >>> Which are then used by other files.
> >>>
> >>> Thus removing those pointers, included 54 files in this patch. Although, the
> >>> actual changes are done in few files only. The changes should work fine as I
> >>> have done similar changes for other drivers too which made their way into the
> >>> kernel. Still, tests on few ata devices are necessary.
> >>
> >> Didn't we agree to split this up into separate patches??
> >>
> >>> I tried but unfortunately, I couldn't find or arrange devices to test upon. I
> >>> have added the authors of the previous commit(s) for respective drivers as
> >>> recipients. It would be very helpful if someone can test it on a device.
> >>
> >> Does qemu support any of the affected drivers?
> >>
> > Hello Jens,
> > Yes we discussed about splitting it and I will surely do it. I just thought that
> > still it has not got tested and in that case keeping check on patch-series with
> > 55+ patches may become somewhat messy. Moreover, to test any driver one will
> > have to apply the patch for libata-core and then that of the driver.
> > So, I sent this v3 in order to get some help for the testing purpose. Once it
> > goes successful, I can split it and send it for the submission.
> > 
> > I am not sure about qemu, I haven't checked for it. Qemu will be appropriate
> > for testing power management for .suspend() and .resume() callbacks?
> 
> Well, that's your homework then, it seems pretty ideal for testing that
> kind of thing and way easier than getting your hands on hardware. It
> won't have support for everything, but so far you are at 0 things
> tested, so any piix and ahci testing would be a win as far as I'm
> concerned.
> 
Okay. Thanks for hinting about qemu. I will get started with it then.

Vaibhav Gupta
> -- 
> Jens Axboe
>