Re: [PATCH v6 0/2] add performance reporting support to FPGA DFL drivers
From: Wu Hao <hidden>
Date: 2020-01-16 03:14:00
Also in:
linux-fpga, lkml
On Wed, Jan 15, 2020 at 09:14:00AM +0100, Greg KH wrote:
On Tue, Jan 14, 2020 at 09:10:40PM -0800, Moritz Fischer wrote:quoted
Hi Greg, On Tue, Jan 14, 2020 at 01:56:05PM +0800, Wu Hao wrote:quoted
On Mon, Jan 06, 2020 at 10:37:42AM +0800, Wu Hao wrote:quoted
On Mon, Dec 16, 2019 at 09:01:04AM +0800, Xu Yilum wrote:quoted
On Mon, Dec 09, 2019 at 10:45:27AM +0800, Wu Hao wrote:quoted
On Mon, Nov 25, 2019 at 04:08:39PM +0800, Wu Hao wrote:quoted
On Mon, Nov 25, 2019 at 08:01:28AM +0000, Will Deacon wrote:quoted
On Mon, Nov 25, 2019 at 11:34:12AM +0800, Wu Hao wrote:quoted
Hi Will and Mark, Could you please help us on review this patchset? as this patchset mainly introduced a new perf driver following the similar way as drivers/perf/*.Why is it not under drivers/perf/, then?Hi Will Thanks for the quick response. This is one sub feature for DFL based FPGAs, and we plan to put this sub feature together with others, including related documentation. It only registers a standard perf pmu for its userspace interfaces.quoted
quoted
This patchset has been submitted for a long time but didn't receive any comment after v4. we appreciate any review comments! thanks in advance. :)Hmm, not sure I saw the previous versions. Guessing I wasn't on cc?We switched to perf API from v4, and started ccing you and Mark from v5. :)Hi Will Did you get a chance to look into this patchset? Thanks HaoHi Will Did you have time to look into this patchset? We have done review work for FPGA part. And as a perf driver, we appreciate your comments. Thanks YilunHi Will Did you get a chance to look into this patchset these days? Actually we didn't receive any comments for a long time, if you are busy and don't have enough time on this, do you know if someone else could help with review and ack from perf driver point of view, or any other things we can do to speed up this? Thanks in advance!Hi Moritz Looks like still no response from Will. :( Do you know someone else could help?Do you have some feedback? I'm a bit confused on what to do in such a situation, do I just take the patch if the maintainer doesn't respond for a while?Resend it and say something like "please review" or the like. With the holidays and catching up from the holidays, this time of year is usually very backlogged for lots of reviewers.
Sure, will resend this patchset soon. Thanks! Hao
greg k-h