From owner-freebsd-net@FreeBSD.ORG Tue Apr 2 13:19:14 2013 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id 0364FBAD for ; Tue, 2 Apr 2013 13:19:14 +0000 (UTC) (envelope-from fodillemlinkarim@gmail.com) Received: from mail-ia0-x22f.google.com (mail-ia0-x22f.google.com [IPv6:2607:f8b0:4001:c02::22f]) by mx1.freebsd.org (Postfix) with ESMTP id C45301F3 for ; Tue, 2 Apr 2013 13:19:13 +0000 (UTC) Received: by mail-ia0-f175.google.com with SMTP id e16so305481iaa.34 for ; Tue, 02 Apr 2013 06:19:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:message-id:date:from:user-agent:mime-version:to:cc :subject:references:in-reply-to:content-type :content-transfer-encoding; bh=olx90bvkItxoosMhqetXTgxaFWA8gBX9mwPq9k6xSGQ=; b=oy3rqH6pSBLfMJApwAvOjkBAYAxn+WsMXBNPfMuSoUP8KRxwDQHlxeCuI3IpxhqOZw xwHITnWnN6cjmptAyxOl82MWM166V0+CQkj2E2H4ePdpJnnm3P8sYkLHKIO6OFfvNbbk 5ucLhz3ucq8Fin6e8ekmnNSBhMWpGLihMtNGqAs8Et9sm7Cm9NOOaFp4uuO3KkL9OyK/ QrBKNdYqjtylYr5HLNGpaMMLzdazF/zniVrliNYRGGaoha/uDLS93EKVVVYHEEqYglum YZzymfzBbwuTmCkdB5qoH45SUHQVv/WKbs8lKPGDm3UBlvBL4sMXi8BxI/cbOYUa7oMi ZPMA== X-Received: by 10.50.183.233 with SMTP id ep9mr5160279igc.87.1364908753395; Tue, 02 Apr 2013 06:19:13 -0700 (PDT) Received: from [192.168.1.73] ([208.85.112.101]) by mx.google.com with ESMTPS id px9sm5446742igc.0.2013.04.02.06.19.10 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Tue, 02 Apr 2013 06:19:11 -0700 (PDT) Message-ID: <515ADACD.8010108@gmail.com> Date: Tue, 02 Apr 2013 09:19:09 -0400 From: Karim Fodil-Lemelin User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130307 Thunderbird/17.0.4 MIME-Version: 1.0 To: Nick Rogers Subject: Re: igb and ALTQ in 9.1-rc3 References: <1355177348.71087.YahooMailClassic@web121601.mail.ne1.yahoo.com> <50C6656E.3020601@gmail.com> <20121211075853.GU48639@FreeBSD.org> <5159AB72.1050202@gmail.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: "freebsd-net@freebsd.org" X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 02 Apr 2013 13:19:14 -0000 Hi Nick, Can you verify that you have at least one of those options in your kernel config file: ALTQ_CBQ ALTQ_PRIQ ALTQ_HFSC Regards, Karim. On 01/04/2013 8:22 PM, Nick Rogers wrote: > On Mon, Apr 1, 2013 at 8:44 AM, Karim Fodil-Lemelin > wrote: >> Hi Jack, >> >> I think this would help M. Rogers case as we have done something similar >> here to circumvent the issue and it seems to work well. I would also add >> that when using ALTQ we found it much more stable to set the number of >> queues to 1: >> >> static int igb_num_queues = 1; >> >> Our approach consisted in keeping igb_start() defined and using >> igb_mq_start_locked() inside it instead of igb_start_locked(). >> >> Regards, >> >> Karim. > Thanks for the pointer. > > I've been working with Jack to get a fix for this in that downgrades > the stack to the older if_start/non-multiqueue interface. See the > following two commits he made to HEAD. > > http://svnweb.freebsd.org/base/head/sys/dev/e1000/if_igb.c?revision=248906&view=markup > http://svnweb.freebsd.org/base/head/sys/dev/e1000/if_igb.h?revision=248908&view=markup > > I've applied these changes to latest 9.1-STABLE src and included the > IGB_LEGACY_TX in the e1000 Makefile. So far I am not having any luck > getting pfctl to think igb is supported. > > i.e. I am still getting the following when loading my PF config: > pfctl -f /etc/pf.conf > pfctl: igb0: driver does not support altq > > Can anyone comment on if the above referenced changes that Jack made > should be sufficient to get ALTQ working with igb? > > The error is coming from src/contrib/pf/pfctl/pfctl.c. There seems to > be a function that checks if the driver is supported or not but I > cannot figure out why the ioctl is not returning a device. > > Here is the device check code for reference: > > int > pfctl_add_altq(struct pfctl *pf, struct pf_altq *a) > { > if (altqsupport && > (loadopt & PFCTL_FLAG_ALTQ) != 0) { > memcpy(&pf->paltq->altq, a, sizeof(struct pf_altq)); > if ((pf->opts & PF_OPT_NOACTION) == 0) { > if (ioctl(pf->dev, DIOCADDALTQ, pf->paltq)) { > if (errno == ENXIO) > errx(1, "qtype not configured"); > else if (errno == ENODEV) > errx(1, "%s: driver does not support " > "altq", a->ifname); > else > err(1, "DIOCADDALTQ"); > } > } > pfaltq_store(&pf->paltq->altq); > } > return (0); > } > > > >> >> >> On 28/03/2013 7:16 PM, Jack Vogel wrote: >>> Have been kept fairly busy with other matters, one thing I could do short >>> term is >>> change the defines in igb the way I did in the em driver so you could >>> still >>> define >>> the older if_start entry. Right now those are based on OS version and so >>> you will >>> automatically get if_transmit, but I could change it to be IGB_LEGACY_TX >>> or >>> so, >>> and that could be defined in the Makefile. >>> >>> Would this help? >>> >>> Jack >>> >>> >>> On Thu, Mar 28, 2013 at 2:31 PM, Nick Rogers wrote: >>> >>>> On Tue, Dec 11, 2012 at 1:09 AM, Jack Vogel wrote: >>>>> On Mon, Dec 10, 2012 at 11:58 PM, Gleb Smirnoff >>>> wrote: >>>>>> On Mon, Dec 10, 2012 at 03:31:19PM -0800, Jack Vogel wrote: >>>>>> J> UH, maybe asking the owner of the driver would help :) >>>>>> J> >>>>>> J> ... and no, I've never been aware of doing anything to stop >>>> supporting >>>>>> altq >>>>>> J> so you wouldn't see any commits. If there's something in the altq >>>> code >>>>>> or >>>>>> J> support (which I have nothing to do with) that caused this no-one >>>>>> informed >>>>>> J> me. >>>>>> >>>>>> Switching from if_start to if_transmit effectively disables ALTQ >>>> support. >>>>>> AFAIR, there is some magic implemented in other drivers that makes them >>>>>> modern (that means using if_transmit), but still capable to switch to >>>>>> queueing >>>>>> mode if SIOCADDALTQ was casted upon them. >>>>>> >>>>>> >>>>> Oh, hmmm, I'll look into the matter after my vacation. >>>>> >>>>> Jack >>>> Has there been any progress on resolving this issue? I recently ran >>>> into this problem upgrading my servers from 8.3 to 9.1-RELEASE and am >>>> wondering what the latest recommendation is. I've used ALTQ and igb >>>> successfully for years and it is unfortunate it no longer works. >>>> Appreciate any advice. >>>> >>>>> _______________________________________________ >>>>> freebsd-net@freebsd.org mailing list >>>>> http://lists.freebsd.org/mailman/listinfo/freebsd-net >>>>> To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org" >>> _______________________________________________ >>> freebsd-net@freebsd.org mailing list >>> http://lists.freebsd.org/mailman/listinfo/freebsd-net >>> To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org" >> >> _______________________________________________ >> freebsd-net@freebsd.org mailing list >> http://lists.freebsd.org/mailman/listinfo/freebsd-net >> To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org"