netdev - Re: [PATCH 1/2] bridge: Adjust min age inc for HZ

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] [day] [month] [year] [list]

Date:	Mon, 5 Mar 2012 13:30:00 -0800
From:	Stephen Hemminger <shemminger@...tta.com>
To:	Joakim Tjernlund <joakim.tjernlund@...nsmode.se>
Cc:	David Miller <davem@...emloft.net>, netdev@...r.kernel.org
Subject: Re: [PATCH 1/2] bridge: Adjust min age inc for HZ > 256

On Mon, 5 Mar 2012 09:47:52 +0100
Joakim Tjernlund <joakim.tjernlund@...nsmode.se> wrote:

> David Miller <davem@...emloft.net> wrote on 2012/03/05 09:24:10:
> >
> > From: Joakim Tjernlund <joakim.tjernlund@...nsmode.se>
> > Date: Mon, 5 Mar 2012 09:14:00 +0100
> >
> > > David Miller <davem@...emloft.net> wrote on 2012/03/05 06:09:21:
> > >>
> > >> From: Stephen Hemminger <shemminger@...tta.com>
> > >> Date: Sun, 4 Mar 2012 21:03:50 -0800
> > >>
> > >> > I don't understand why this is required.
> > >>
> > >> And I don't understand why it takes nearly a week to review a one line
> > >> patch like this.
> > >>
> > >> If you don't say anything I'm going to just apply it after a few days,
> > >> which is what I did here already.
> > >>
> > >> Me applying it should never be your trigger to review the patch, yet I
> > >> see this happen all the time.  What do you think I was waiting for
> > >> these past 5 days, divine intervention?
> > >
> > > hmm, does this mean you "unapplied" the patch now?
> >
> > I can't, it's now in the permanent record so I can't remove it.
> >
> > At best I can apply a revert patch, but you guys are still discussing
> > this so I have no idea whether that's even necessary or not.
> 
> OK, hopefully Stephen is happy with my explanation. In any case
> I won't be able to retest if only patch 2 is needed as our test window closed
> this weekend.
> 
>  Jocke
> 

I am okay with the change, but that whole section needs more comments
and explanation. 
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html