From owner-freebsd-net@freebsd.org Sat Mar 25 01:16:21 2017 Return-Path: Delivered-To: freebsd-net@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 854F9D1BCA4 for ; Sat, 25 Mar 2017 01:16:21 +0000 (UTC) (envelope-from ermal.luci@gmail.com) Received: from mail-it0-x22e.google.com (mail-it0-x22e.google.com [IPv6:2607:f8b0:4001:c0b::22e]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 46BD3767 for ; Sat, 25 Mar 2017 01:16:21 +0000 (UTC) (envelope-from ermal.luci@gmail.com) Received: by mail-it0-x22e.google.com with SMTP id w124so25299217itb.1 for ; Fri, 24 Mar 2017 18:16:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=PdG3eNwz1rpGClmp0vQcXq5dRugazymcvWnv4ViEj0M=; b=qwB1+FU29xitgjD/ItNbeWCOjw/FCkOulURn8aUDdgVtInmtZunxhaSVUAvhBYXIAW x20Lte/JNFmLJCXpBnQBk0kQXLF6mrpNmWCZ0inw/WyhgauE18RC6cC5ntI1y6r/KK+1 C0yZJMEEkS4O9ts4QBeXBEBBmwuDcVTTfezIOiD3MYH2zIhgFS2qOcpwKRj2qkvXUwO4 bIgkG1uyCas/Ht4PzerOrwl4Thebr9MSqACBlzLSQUhyVlDVEghD2lx2soxOGvNtXNUm p1RFzTi2LSe8bbQEkAcvtCs+9+bbKaRZOoxhmBvpw5uHh6fio2WcXgkSByHzczMHsZGo 1Qtw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc; bh=PdG3eNwz1rpGClmp0vQcXq5dRugazymcvWnv4ViEj0M=; b=ZHHz8jYT+5MifEAqrjHFfmzfT8XNh0AlaVMxb+CmgdcBa8rVpY58FBKoytSFI2awKe 1o+Hbn9n3PQoxZUuF6JKkeVAwM1KFs+cvzJT1gyXnJrIpxEZFHgKpJirnWIUUPrUD61t dpb19YQ/LJFLevEA+6YX/tzfcgqwKepsaCiqsQLQNmwkeO/cSthap1YIlIaVpLGAQ2Fu 1XySYxEZ3ZbHyMyPUpxcmEVAwLMpJnMT/1Buzp3Vk1KAbIXixrDUDABsY3bU23E7dxFF rrQ9bbicG+lBWDA7+7kji2E6oAPzZZA82Zj+Dv4WD14gGYElquQcv+7NAu4SnJ/YOwMH r8Jg== X-Gm-Message-State: AFeK/H229rBa6Xu4sL/pYcIKOC3xEMF7LSWtOvReLtBxeMB5Zhb256xue94Xp5pjkss28fBberSOwqLCaX4ikQ== X-Received: by 10.107.171.67 with SMTP id u64mr10890615ioe.102.1490404580599; Fri, 24 Mar 2017 18:16:20 -0700 (PDT) MIME-Version: 1.0 Sender: ermal.luci@gmail.com Received: by 10.107.149.135 with HTTP; Fri, 24 Mar 2017 18:16:19 -0700 (PDT) In-Reply-To: References: From: =?UTF-8?Q?Ermal_Lu=C3=A7i?= Date: Fri, 24 Mar 2017 18:16:19 -0700 X-Google-Sender-Auth: yDFrEH3pGm2gDrKqJ6XdpD3iYl8 Message-ID: Subject: Re: if_epair altq support problem To: =?UTF-8?B?w5Z6a2FuIEtJUklL?= Cc: "freebsd-net@freebsd.org" Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.23 X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 25 Mar 2017 01:16:21 -0000 On Fri, Mar 24, 2017 at 5:48 PM, =C3=96zkan KIRIK w= rote: > Hi again, > This patch works perfectly also. > Thank you so much. > Is it possible to commit this patch to repo? > https://svnweb.freebsd.org/changeset/base/315877 > > On Thu, Mar 23, 2017 at 10:57 PM, Ermal Lu=C3=A7i wrote= : > >> >> >> On Thu, Mar 23, 2017 at 12:16 PM, =C3=96zkan KIRIK >> wrote: >> >>> Ermal thank you again, >>> It compiles but pf still says that "pfctl: epair0b: driver does not >>> support altq" >>> :/ >>> >> >> The only way to get that error is by not applying the patch properly :) >> >> >>> >>> On Thu, Mar 23, 2017 at 9:46 PM, Ermal Lu=C3=A7i wrot= e: >>> >>>> >>>> On Thu, Mar 23, 2017 at 11:06 AM, =C3=96zkan KIRIK >>>> wrote: >>>> >>>>> Thank you, I'm waiting for 10.3 fix :) >>>>> have a nice day >>>>> >>>> >>>> >>>> This should work for 10.3++ >>>> >>>> diff --git a/sys/net/if_epair.c b/sys/net/if_epair.c >>>> index 540f06c..d028a95 100644 >>>> --- a/sys/net/if_epair.c >>>> +++ b/sys/net/if_epair.c >>>> @@ -829,7 +829,8 @@ epair_clone_create(struct if_clone *ifc, char >>>> *name, size_t len, caddr_t params) >>>> ifp->if_start =3D epair_start; >>>> ifp->if_ioctl =3D epair_ioctl; >>>> ifp->if_init =3D epair_init; >>>> - ifp->if_snd.ifq_maxlen =3D ifqmaxlen; >>>> + IFQ_SET_MAXLEN(&ifp->if_snd, ifqmaxlen); >>>> + IFQ_SET_READY(&ifp->if_snd); >>>> /* Assign a hopefully unique, locally administered etheraddr. = */ >>>> eaddr[0] =3D 0x02; >>>> eaddr[3] =3D (ifp->if_index >> 8) & 0xff; >>>> @@ -855,7 +856,8 @@ epair_clone_create(struct if_clone *ifc, char >>>> *name, size_t len, caddr_t params) >>>> ifp->if_start =3D epair_start; >>>> ifp->if_ioctl =3D epair_ioctl; >>>> ifp->if_init =3D epair_init; >>>> - ifp->if_snd.ifq_maxlen =3D ifqmaxlen; >>>> + IFQ_SET_MAXLEN(&ifp->if_snd, ifqmaxlen); >>>> + IFQ_SET_READY(&ifp->if_snd); >>>> /* We need to play some tricks here for the second interface. = */ >>>> strlcpy(name, epairname, len); >>>> error =3D if_clone_create(name, len, (caddr_t)scb); >>>> >>>> >>>> >>>> On Wed, Mar 22, 2017 at 11:44 PM, Ermal Lu=C3=A7i wr= ote: >>>>> >>>>>> >>>>>> On Wed, Mar 22, 2017 at 10:50 AM, =C3=96zkan KIRIK >>>>>> wrote: >>>>>> >>>>>>> Sorry, I mistested on 10.3RELENG. >>>>>>> it works on 11 RELENG, But at 10.3RELENG it throws >>>>>>> >>>>>>> /usr/src/sys/modules/if_epair/../../net/if_epair.c:830:2: error: >>>>>>> implicit declaration of function 'if_setstartfn' is invalid in C99 >>>>>>> [-Werror,-Wimplicit-function-declaration] >>>>>>> >>>>>>> >>>>>>> On Wed, Mar 22, 2017 at 8:18 PM, =C3=96zkan KIRIK >>>>>>> wrote: >>>>>>> >>>>>>>> Thank You Ermal ! >>>>>>>> >>>>>>>> It works perfectly, can you commit this patch to 11.0 RELENG and >>>>>>>> MFC to 10.3 RELENG ? >>>>>>>> >>>>>>>> >>>>>> Thanks, for confirming that it fixes your issues. >>>>>> Yeah, on 10.3 its almost the same fix i will deal with it. >>>>>> >>>>>> >>>>>>> Regards >>>>>>>> >>>>>>>> On Wed, Mar 22, 2017 at 6:59 AM, Ermal Lu=C3=A7i >>>>>>>> wrote: >>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> On Tue, Mar 21, 2017 at 5:26 AM, =C3=96zkan KIRIK < >>>>>>>>> ozkan.kirik@gmail.com> wrote: >>>>>>>>> >>>>>>>>>> Hello, >>>>>>>>>> >>>>>>>>>> I sent this email also to freebsd-pf list. But I think that the >>>>>>>>>> main >>>>>>>>>> problem is belongs to sys/net/if_epair.c. >>>>>>>>>> >>>>>>>>>> I'm using FreeBSD 10.3-p17 amd64. epair pseudo device is listed = as >>>>>>>>>> supperted deviced at the Man page of altq(4). >>>>>>>>>> From man page of altq : >>>>>>>>>> >>>>>>>>>> *SUPPORTED DEVICES >>>>>>>>> an.cgi?query=3Daltq#end>* >>>>>>>>>> >>>>>>>>>> The driver modifications described in altq(9) >>>>>>>>>> >>>>>>>>> ropos=3D0&manpath=3DFreeBSD+11.0-RELEASE+and+Ports> >>>>>>>>>> are required to use a cer- >>>>>>>>>> tain network card with *ALTQ*. They have been applie= d >>>>>>>>>> to the following >>>>>>>>>> hardware drivers: ae(4) >>>>>>>>>> >>>>>>>>> pos=3D0&manpath=3DFreeBSD+11.0-RELEASE+and+Ports>, >>>>>>>>>> age(4) >>>>>>>>> an.cgi?query=3Dage&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> alc(4) >>>>>>>>> an.cgi?query=3Dalc&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> ale(4) >>>>>>>>> an.cgi?query=3Dale&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> an(4) >>>>>>>>> an.cgi?query=3Dan&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.0= -REL >>>>>>>>>> EASE+and+Ports>, >>>>>>>>>> ath(4) >>>>>>>>> an.cgi?query=3Dath&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> aue(4) >>>>>>>>> an.cgi?query=3Daue&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> axe(4) >>>>>>>>> an.cgi?query=3Daxe&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> bce(4) >>>>>>>>> an.cgi?query=3Dbce&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> bfe(4) >>>>>>>>> an.cgi?query=3Dbfe&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> bge(4) >>>>>>>>> an.cgi?query=3Dbge&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> bxe(4) >>>>>>>>> an.cgi?query=3Dbxe&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> cas(4) >>>>>>>>> an.cgi?query=3Dcas&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.= 0-RE >>>>>>>>>> LEASE+and+Ports>, >>>>>>>>>> cxgbe(4) >>>>>>>>> an.cgi?query=3Dcxgbe&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+1= 1.0- >>>>>>>>>> RELEASE+and+Ports>, >>>>>>>>>> dc(4) >>>>>>>>> an.cgi?query=3Ddc&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.0= -REL >>>>>>>>>> EASE+and+Ports>, >>>>>>>>>> de(4) >>>>>>>>> an.cgi?query=3Dde&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.0= -REL >>>>>>>>>> EASE+and+Ports>, >>>>>>>>>> ed(4) >>>>>>>>> an.cgi?query=3Ded&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.0= -REL >>>>>>>>>> EASE+and+Ports>, >>>>>>>>>> em(4) >>>>>>>>> an.cgi?query=3Dem&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.0= -REL >>>>>>>>>> EASE+and+Ports>, >>>>>>>>>> ep(4) >>>>>>>>> an.cgi?query=3Dep&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+11.0= -REL >>>>>>>>>> EASE+and+Ports>, >>>>>>>>>> epair(4) >>>>>>>>> an.cgi?query=3Depair&sektion=3D4&apropos=3D0&manpath=3DFreeBSD+1= 1.0- >>>>>>>>>> RELEASE+and+Ports>, >>>>>>>>>> >>>>>>>>>> .... >>>>>>>>>> >>>>>>>>>> But while trying to use it the system says that it's not >>>>>>>>>> suppoerted. I >>>>>>>>>> tried on FreeBSD 11 also. The output is below: >>>>>>>>>> >>>>>>>>>> pf.conf : >>>>>>>>>> altq on epair0b hfsc bandwidth 1Mb queue { ftp, ssh, icmp, other= } >>>>>>>>>> queue ftp bandwidth 30% priority 0 hfsc (upperlimit 99%) >>>>>>>>>> queue ssh bandwidth 30% priority 2 hfsc (upperlimit 99%) >>>>>>>>>> queue icmp bandwidth 10% priority 2 hfsc (upperlimit 99%) >>>>>>>>>> queue other bandwidth 30% priority 1 hfsc (default upperlimit 99= %) >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> # ifconfig epair0 create >>>>>>>>>> # ifconfig epair0a up >>>>>>>>>> # ifconfig epair0b up >>>>>>>>>> # pfctl -f pf.conf >>>>>>>>>> pfctl: epair0b: driver does not support altq >>>>>>>>>> >>>>>>>>>> # sysctl -a | grep ALTQ >>>>>>>>>> options ALTQ_NOPCC >>>>>>>>>> options ALTQ_PRIQ >>>>>>>>>> options ALTQ_CDNR >>>>>>>>>> options ALTQ_HFSC >>>>>>>>>> options ALTQ_RIO >>>>>>>>>> options ALTQ_RED >>>>>>>>>> options ALTQ_CBQ >>>>>>>>>> options ALTQ >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> I have a look on /usr/src/sys/net/if_epair.c, and found the ALTQ >>>>>>>>>> section: >>>>>>>>>> >>>>>>>>>> 514 #ifdef ALTQ >>>>>>>>>> 515 /* Support ALTQ via the clasic if_start() path. */ >>>>>>>>>> 516 IF_LOCK(&ifp->if_snd); >>>>>>>>>> 517 if (ALTQ_IS_ENABLED(&ifp->if_snd)) { >>>>>>>>>> 518 ALTQ_ENQUEUE(&ifp->if_snd, m, NULL, error); >>>>>>>>>> 519 if (error) >>>>>>>>>> 520 ifp->if_snd.ifq_drops++; >>>>>>>>>> 521 IF_UNLOCK(&ifp->if_snd); >>>>>>>>>> 522 if (!error) { >>>>>>>>>> 523 ifp->if_obytes +=3D len; >>>>>>>>>> 524 if (mflags & (M_BCAST|M_MCAST)) >>>>>>>>>> 525 ifp->if_omcasts++; >>>>>>>>>> 526 >>>>>>>>>> 527 if ((ifp->if_drv_flags & >>>>>>>>>> IFF_DRV_OACTIVE) =3D=3D 0) >>>>>>>>>> 528 epair_start_locked(ifp); >>>>>>>>>> 529 else >>>>>>>>>> 530 (void)epair_add_ifp_for_dra= in >>>>>>>>>> ing(ifp); >>>>>>>>>> 531 } >>>>>>>>>> 532 return (error); >>>>>>>>>> 533 } >>>>>>>>>> 534 IF_UNLOCK(&ifp->if_snd); >>>>>>>>>> 535 #endif >>>>>>>>>> >>>>>>>>>> >>>>>>>>> >>>>>>>>> Just apply manually this patch to make it work. >>>>>>>>> >>>>>>>>> diff --git a/sys/net/if_epair.c b/sys/net/if_epair.c >>>>>>>>> index 540f06c..04733a5 100644 >>>>>>>>> --- a/sys/net/if_epair.c >>>>>>>>> +++ b/sys/net/if_epair.c >>>>>>>>> @@ -827,9 +827,11 @@ epair_clone_create(struct if_clone *ifc, cha= r >>>>>>>>> *name, size_t len, caddr_t params) >>>>>>>>> ifp->if_capabilities =3D IFCAP_VLAN_MTU; >>>>>>>>> ifp->if_capenable =3D IFCAP_VLAN_MTU; >>>>>>>>> ifp->if_start =3D epair_start; >>>>>>>>> + if_setstartfn(ifp, epair_start); >>>>>>>>> + if_setsendqlen(ifp, ifqmaxlen); >>>>>>>>> + if_setsendqready(ifp); >>>>>>>>> ifp->if_ioctl =3D epair_ioctl; >>>>>>>>> ifp->if_init =3D epair_init; >>>>>>>>> - ifp->if_snd.ifq_maxlen =3D ifqmaxlen; >>>>>>>>> /* Assign a hopefully unique, locally administered >>>>>>>>> etheraddr. */ >>>>>>>>> eaddr[0] =3D 0x02; >>>>>>>>> eaddr[3] =3D (ifp->if_index >> 8) & 0xff; >>>>>>>>> @@ -852,10 +854,11 @@ epair_clone_create(struct if_clone *ifc, >>>>>>>>> char *name, size_t len, caddr_t params) >>>>>>>>> ifp->if_flags =3D IFF_BROADCAST | IFF_SIMPLEX | >>>>>>>>> IFF_MULTICAST; >>>>>>>>> ifp->if_capabilities =3D IFCAP_VLAN_MTU; >>>>>>>>> ifp->if_capenable =3D IFCAP_VLAN_MTU; >>>>>>>>> - ifp->if_start =3D epair_start; >>>>>>>>> + if_setstartfn(ifp, epair_start); >>>>>>>>> + if_setsendqlen(ifp, ifqmaxlen); >>>>>>>>> + if_setsendqready(ifp); >>>>>>>>> ifp->if_ioctl =3D epair_ioctl; >>>>>>>>> ifp->if_init =3D epair_init; >>>>>>>>> - ifp->if_snd.ifq_maxlen =3D ifqmaxlen; >>>>>>>>> /* We need to play some tricks here for the second >>>>>>>>> interface. */ >>>>>>>>> strlcpy(name, epairname, len); >>>>>>>>> error =3D if_clone_create(name, len, (caddr_t)scb); >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>>> I have no idea that why it says that it doesn't support altq >>>>>>>>>> altough the >>>>>>>>>> source code contains ALTQ section. >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> Regards >>>>>>>>>> =C3=96zkan KIRIK >>>>>>>>>> _______________________________________________ >>>>>>>>>> freebsd-net@freebsd.org mailing list >>>>>>>>>> https://lists.freebsd.org/mailman/listinfo/freebsd-net >>>>>>>>>> To unsubscribe, send any mail to "freebsd-net-unsubscribe@freeb >>>>>>>>>> sd.org" >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> -- >>>>>>>>> Ermal >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>> >>>>>> >>>>>> >>>>>> -- >>>>>> Ermal >>>>>> >>>>> >>>>> >>>> >>>> >>>> -- >>>> Ermal >>>> >>> >>> >> >> >> -- >> Ermal >> > > --=20 Ermal