Doesnt seem like i responded to this.
On Sat, 2005-28-05 at 14:07 +0200, Thomas Graf wrote:
> * jamal <1117244567.6251.34.camel@xxxxxxxxxxxxxxxxxxxxx> 2005-05-27 21:42
> > On Fri, 2005-27-05 at 18:35 +0200, Thomas Graf wrote:
> >
> > > I do NOT agree on moving gc_ into this architecture as well,
> > > it doesn't belong there.
> >
> > Well, you do realize they are part of the in_dev ? ;->
>
> Huh? They cannot be device specific, there is only one gc
> per neighbour table. So even _iff_ there was a device specific
> setting it wouldn't make much sense ;->
>
I keep forgeting some of these tables are system wide.
But i think it doesnt matter as long as we comply to the abstration
thats in the kernel; in other words, config access is still via the
in_devxx. i.e if you supply any ifindex (since all devices _at the
moment_ point to the same ARP/ndisc table), it can be accessed and
configured.
> > I see a little main service header with ifindex always no different than
> > IFA or IFLINK etc followed by appropriate TLVs which are nested.
>
> I guess we could simply move NDTPA_PARMS into the devconfig family.
Probably a good start. This would still be missing the gc thresholds
etc, no?
> > Well, if you look at the structure there is no reason they should really
> > be separate; infact theres a comment:
> > -----------
> > struct neigh_parms parms;
> > /* HACK. gc_* shoul follow parms without a gap! */
> > int gc_interval;
> > int gc_thresh1;
> > int gc_thresh2;
> > ----------
>
> You do realize the reason for that comment? ;-> The author of
> the neighbour procfs code was simply too lazy to put these
> into a separate place so he introduced a nasty hack.
>
I think you are correct.
> What about this, we move the device specific part into the new devconfig
> layer but leave the main configuration, statistics, and gc parameters
> in RTM_NEIGHTBL? i.e.
>
> arp_cache entries 2 reachable-time 23s 993msec retransmit-time 1s
> key-len 4 entry-size 152 last-flush 55m 44s 572msec
> gc threshold 128/512/1024 interval 30s chain-position 3
> hash-rand 0x69650257/0x00000003 last-rand 1m 44s 571msec
> ? refcnt 1 pending-queue-limit 3 proxy-delayed-queue-limit 64
> ? num-userspace-probes 0 num-unicast-probes 3 num-multicast-probes 3
> ? min-age 1s base-reachable-time 30s stale-check-interval 1m
> ? initial-probe-delay 5s answer-delay 1s proxy-answer-delay 800msec
> lookups 195 hits 190 failed 0 allocations 3 destroys 1
> hash-grows 1 forced-gc-runs 0 periodic-gc-runs 910
> rcv-unicast-probes 0 rcv-multicast-probes 0
>
> Xarp_cache<eth0> reachable-time 35s 967msec retransmit-time 1s
> X refcnt 3 pending-queue-limit 3 proxy-delayed-queue-limit 64
> X num-userspace-probes 0 num-unicast-probes 3 num-multicast-probes 3
> X min-age 1s base-reachable-time 30s stale-check-interval 1m
> X initial-probe-delay 5s answer-delay 1s proxy-answer-delay 800msec
>
>
> Everything marked with X is clearly device specific so it will move.
> Everything marked with '?' is the default parameter set, we can either
> leave it or move it as well and put it under into a 'default' ifindex.
> I don't care about the latter, either is fine with me.
>
All "stuffs" (your bases is mine) accessible via in_devxx abstraction
should be devconfig configurable. I do concur that some of it may not
make sense - but that should be the exception rules...
Ok, lets say these tables are one such exception, but note:
In the future there could be multiple ARP tables for example (very
likely given all the virtualization approaches happening). In other
words different devices may point to different tables...
cheers,
jamal
|