Hi Gleb & Craig,
I am already working on the issues that need to be fixed, namely:
1) all the changes from projects/pf
2) unloading the module
3) one purge thread per pf instance will remain until we have
more experience
It's almost ready, I am trying to fix unloading the module.
As I am not at home this week I will send you a patch for review on
Monday.
Best regards,
Nikos
On 02/16/15 09:02, Craig Rodrigues wrote:
On Sun, Feb 15, 2015 at 11:46 PM, Gleb Smirnoff <gleb...@freebsd.org> wrote:
On Sun, Feb 15, 2015 at 11:36:17PM -0800, Craig Rodrigues wrote:
C> On Sun, Feb 15, 2015 at 5:25 PM, Gleb Smirnoff <gleb...@freebsd.org>
wrote:
C>
C> > On Sun, Feb 15, 2015 at 01:33:19PM -0800, Craig Rodrigues wrote:
C> > C> By the way, it would be helpful if you could provide feedback in
C> > C> Phabricator. When I created those Phabricator reviews, I added
C> > C> you as a reviewer to all of them, so you can't say that you didn't
see
C> > C> the patches.
C> > C> You did not provide feedback on any of them:
C> > C>
C> > C> https://reviews.freebsd.org/D1309
C> > C> https://reviews.freebsd.org/D1312
C> > C> https://reviews.freebsd.org/D1313
C> > C> https://reviews.freebsd.org/D1315
C> > C>
C> > C> Please take some time to go and provide feedback in those
C> > C> reviews, so that a better patch can be made that makes you happy.
C> >
C> > Did you address all problems that arised after code was committed?
C> >
C> > Please do, otherwise my review would require me to cut-n-paste from
C> > my own emails.
C> >
C>
C> Yes, please cut and paste from your e-mails, and put in the reviews.
No, I will not do this. You know right now that you have problems
in the aforementioned phab URLs, and you are asking me to look at
patches at to point them out to you. This is your task, not mine.
Please address all already known problems and update the phab revisions.
C> It's easier to follow in the individual reviews because there are
different
C> changes
C> in each review, rather than one big revert, which is what you did.
I'm already starting to repeat myself. I did a big revert, because
the first change wasn't compilable, later changes fixed compilation
failures, but introduced unacceptable bugs. That's why I was forced
to back out all chain.
It's not clear to me what parts you find unacceptable,
and what parts you find acceptable. I added you to the reviews
in Phabricator from the beginning, so it would have been nice if you could
have
provided the review feedback there. Since you are slow to provide
feedback when I asked you to be a reviewer in Phabricator, but you
are quick to revert changes, and provide unclear feedback during your
reversion,
this makes it very hard to make forward progress in this area.
In the past, multiple like Nikos and Martin Matuska have been providing
patches in your
projects/pf branch, but you don't seem to have the time/interest to push
these patches back into head.
I've seen that Martin Matuska stopped waiting for you to merge the pf
branch to head, and started cherry
--
Craig
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
