From patchwork Tue May 10 14:21:02 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Sebastian.Poehn@Belden.com X-Patchwork-Id: 94993 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id E2368B6EF2 for ; Wed, 11 May 2011 00:21:16 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756466Ab1EJOVL (ORCPT ); Tue, 10 May 2011 10:21:11 -0400 Received: from mx1.belden.com ([12.161.118.90]:48791 "EHLO mx1.belden.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755514Ab1EJOVK convert rfc822-to-8bit (ORCPT ); Tue, 10 May 2011 10:21:10 -0400 X-AuditID: 0a01015a-b7b1eae0000036eb-84-4dc949d5b3eb Received: from bdcnotes2.beldencdt.com ( [10.1.1.72]) by mx1.belden.com (ESMTP Service Ready) with SMTP id 09.F8.14059.5D949CD4; Tue, 10 May 2011 10:21:09 -0400 (EDT) X-Disclaimed: 1 MIME-Version: 1.0 Importance: Normal X-Priority: 3 (Normal) In-Reply-To: References: Subject: [PATCH] ixgbe: RX ntuple feature must check num_rx_queues From: Sebastian.Poehn@Belden.com To: "netdev" Message-ID: Date: Tue, 10 May 2011 10:21:02 -0400 X-Mailer: Lotus Domino Web Server Release 8.5.2FP2 March 22, 2011 X-MIMETrack: Serialize by HTTP Server on BDCNotes2/BeldenCDT(Release 8.5.2FP2|March 22, 2011) at 05/10/2011 10:21:02 AM, Serialize complete at 05/10/2011 10:21:02 AM, Itemize by HTTP Server on BDCNotes2/BeldenCDT(Release 8.5.2FP2|March 22, 2011) at 05/10/2011 10:21:02 AM, Serialize by Router on BDCNotes2/BeldenCDT(Release 8.5.2FP2|March 22, 2011) at 05/10/2011 10:21:02 AM content-transfer-encoding: 8BIT MIME-Version: 1.0 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmpgleLIzCtJLcpLzFFi42LhYmT00L3qedLXoH29vMWxBWIOjB6fN8kF MEY1MNokJZaUBWem5+nb2STm5eWXJJakKqSkFifbKjml5qSk5im4ZBYn5yRm5qYW6XoG++ta WJhaKilkptgqGSkpFOQkJqfmpuaV2ColFhSk5qUo2XEpYAAboLLMPIXUvOT8lMy8dFslmDm6 hkoJT1ky9k1bxViwmq3i/7TrTA2Me1i7GDk5JARMJI72vWOHsMUkLtxbz9bFyMUhJLCCUWLZ ii5GiAS/xKc/rWwgNq+AoMTJmU9YQGxOASGJjqudrDDNE9b9Yu5i5ACK80pM+GcNEhYS4JF4 tWE/I0hYWMBJYm93LkiYTUBBYuKq42BrRQRUJb5v3M0EMT1UYsGtBWCbWIDiB1+fZYaY7ijx 6NkbRpDTJASamCW2z2kAa2AW0JZYtvA1M7rTmAX0JP7/PMU4gVF4FpLULCQtmOIQLQsYWVYx 8uVWGOolgWNGLzk/dxMjJH6jdjA+bVE4xCjAwajEw/uP+4SvEGtiWXFl7iFGCQ5mJRHeeT+B QrwpiZVVqUX58UWlOanFhxgtgH6ZyCzFnZwPTC15JfHGBgYoHCVx3vvGb7yFBNKBqSI7NbUg tQimlYmDE2Q0l5RIMTDaU4sSS0sy4kFpKb4YmJikGhjdlug2uQpwed+X+cH8ODBu6/OFLiJb dB9uW8I6n0tGMaX53c89aVp39Fhl+Zivq5fvuldfM8eNdc70y26yho/05m/dZyC8n/mhRtGS BfzuBp1v/Z/6fpaarWW8eKNp3tbpvvyxQmflzNdu317S/eFdFX+22rFd0jL7us92fctbdEav 9MEco0AlluKMREMt5qLiRAAcBlUuEwMAAA== Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The driver must check how much RX queues there are, not TX queues. Signed-off-by: Sebastian Poehn DISCLAIMER: Privileged and/or Confidential information may be contained in this message. If you are not the addressee of this message, you may not copy, use or deliver this message to anyone. In such event, you should destroy the message and kindly notify the sender by reply e-mail. It is understood that opinions or conclusions that do not relate to the official business of the company are neither given nor endorsed by the company. Thank You. --- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html --- ixgbe_ethtool.c.orig    2011-05-10 16:18:00.313745560 +0200 +++ ixgbe_ethtool.c    2011-05-10 16:18:23.285747635 +0200 @@ -2349,9 +2349,9 @@ static int ixgbe_set_rx_ntuple(struct ne        /*       * Don't allow programming if the action is a queue greater than -     * the number of online Tx queues. +     * the number of online Rx queues.       */ -    if ((fs->action >= adapter->num_tx_queues) || +    if ((fs->action >= adapter->num_rx_queues) ||          (fs->action < ETHTOOL_RXNTUPLE_ACTION_DROP))          return -EINVAL;