On Thu, May 22, 2008 at 3:42 PM, Bruce M. Simpson <[EMAIL PROTECTED]> wrote:
> Hi,
>
> It looks like this patch will cause gratuitous ARP to be queued even when
> the interface is not IFF_UP, is this intentional?
>
> Niki Denev wrote:
>>
>> I think arp_gratuit() needs a better name.
>>
>
> arp_anno
Hi,
It looks like this patch will cause gratuitous ARP to be queued even
when the interface is not IFF_UP, is this intentional?
Niki Denev wrote:
I think arp_gratuit() needs a better name.
arp_announce() ?
Is if_ethersubr.c:ether_ifattach() good place to register the EVENT hook?
On Thu, May 22, 2008 at 1:41 AM, Max Laier <[EMAIL PROTECTED]> wrote:
> I think if_ether.c would be good, where arp_ifinit() is.
>
Here is what i've come up with (some comments after the patch) :
diff -ur /usr/src/.zfs/snapshot/orig/sys/net/if.c /usr/src/sys/net/if.c
--- /usr/src/.zfs/snapshot/or
On Thursday 22 May 2008 00:14:37 Niki Denev wrote:
> On Thu, May 22, 2008 at 12:32 AM, Max Laier <[EMAIL PROTECTED]> wrote:
> > On Wednesday 21 May 2008 23:22:52 Niki Denev wrote:
> >> On Thu, May 22, 2008 at 12:05 AM, Max Laier <[EMAIL PROTECTED]>
wrote:
> >> > Looks good, though I'd probably mov
On Thu, May 22, 2008 at 12:32 AM, Max Laier <[EMAIL PROTECTED]> wrote:
> On Wednesday 21 May 2008 23:22:52 Niki Denev wrote:
>> On Thu, May 22, 2008 at 12:05 AM, Max Laier <[EMAIL PROTECTED]> wrote:
>> > Looks good, though I'd probably move up the _INVOKE to before the
>> > ARPs are sent out. Prob
On Wednesday 21 May 2008 23:22:52 Niki Denev wrote:
> On Thu, May 22, 2008 at 12:05 AM, Max Laier <[EMAIL PROTECTED]> wrote:
> > Looks good, though I'd probably move up the _INVOKE to before the
> > ARPs are sent out. Probably between twiddling the hardware and
> > sending ARPs (though that needs
On Thu, May 22, 2008 at 12:05 AM, Max Laier <[EMAIL PROTECTED]> wrote:
> Looks good, though I'd probably move up the _INVOKE to before the ARPs are
> sent out. Probably between twiddling the hardware and sending ARPs
> (though that needs an else-case if the interface is still down). In fact
> the
On Wednesday 21 May 2008 22:44:42 Niki Denev wrote:
> On Wed, May 21, 2008 at 8:44 PM, Max Laier <[EMAIL PROTECTED]> wrote:
> >> It doesn't (and shouldn't have to). I'd simply add an
> >> EVENTHANDLER_INVOKE(ifaddr_event, ifp) to if_setlladdr() - we do
> >> that for INET[6] address already. Then
On Wed, May 21, 2008 at 8:44 PM, Max Laier <[EMAIL PROTECTED]> wrote:
>> It doesn't (and shouldn't have to). I'd simply add an
>> EVENTHANDLER_INVOKE(ifaddr_event, ifp) to if_setlladdr() - we do that for
>> INET[6] address already. Then vlan (and any other device interested in
>> LLaddress change
On Wed, May 21, 2008 at 8:44 PM, Max Laier <[EMAIL PROTECTED]> wrote:
> On Wednesday 21 May 2008 19:31:46 Niki Denev wrote:
>> If one tries to use lagg0.2 style vlans on lagg0 interface configured
>> from rc.conf it does't work.
>> The problem is that all of the cloned interfaces (lagg0 , lagg0.2,
On Wednesday 21 May 2008 19:31:46 Niki Denev wrote:
> If one tries to use lagg0.2 style vlans on lagg0 interface configured
> from rc.conf it does't work.
> The problem is that all of the cloned interfaces (lagg0 , lagg0.2, etc)
> are created before any other interface configuration is done,
> and
If one tries to use lagg0.2 style vlans on lagg0 interface configured
from rc.conf it does't work.
The problem is that all of the cloned interfaces (lagg0 , lagg0.2, etc)
are created before any other interface configuration is done,
and in this case lagg0 is created, then lagg0.2 is created.
But be
12 matches
Mail list logo
