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: <20200115051040.GA1389@epycbox.lan>
Date:   Tue, 14 Jan 2020 21:10:40 -0800
From:   Moritz Fischer <mdf@...nel.org>
To:     gregkh@...uxfoundation.org
Cc:     mdf@...nel.org, 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, gregkh@...uxfoundation.org, yilun.xu@...el.com
Subject: Re: [PATCH v6 0/2] add performance reporting support to FPGA DFL
 drivers

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?

As always thanks for your help!

Cheers,
Moritz