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

From: Moritz Fischer
Date: Wed Jan 15 2020 - 00:10:48 EST


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