linux-kernel - Re: [PATCH v6 0/2] add performance reporting support to FPGA DFL drivers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20200115081400.GA2978927@kroah.com>
Date:   Wed, 15 Jan 2020 09:14:00 +0100
From:   Greg KH <gregkh@...uxfoundation.org>
To:     Moritz Fischer <mdf@...nel.org>
Cc:     Will Deacon <will@...nel.org>, mark.rutland@....com,
        linux-fpga@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-api@...r.kernel.org, atull@...nel.org, yilun.xu@...el.com
Subject: Re: [PATCH v6 0/2] add performance reporting support to FPGA DFL
 drivers

On Tue, Jan 14, 2020 at 09:10:40PM -0800, Moritz Fischer wrote:
> Hi Greg,
> 
> On Tue, Jan 14, 2020 at 01:56:05PM +0800, Wu Hao wrote:
> > On Mon, Jan 06, 2020 at 10:37:42AM +0800, Wu Hao wrote:
> > > On Mon, Dec 16, 2019 at 09:01:04AM +0800, Xu Yilum wrote:
> > > > On Mon, Dec 09, 2019 at 10:45:27AM +0800, Wu Hao wrote:
> > > > > On Mon, Nov 25, 2019 at 04:08:39PM +0800, Wu Hao wrote:
> > > > > > On Mon, Nov 25, 2019 at 08:01:28AM +0000, Will Deacon wrote:
> > > > > > > On Mon, Nov 25, 2019 at 11:34:12AM +0800, Wu Hao wrote:
> > > > > > > > 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.
> > > > > > 
> > > > > > > 
> > > > > > > > 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
> > > > > Hao
> > > > 
> > > > Hi 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
> > > > Yilun
> > > 
> > > Hi 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.

greg k-h