Message ID | 20170516182800.GA1067@felix-thinkpad.cavium.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show
Return-Path: <netdev-owner@vger.kernel.org> 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 3wS5Y64LMTz9s4q for <patchwork-incoming@ozlabs.org>; Wed, 17 May 2017 04:28:10 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=CAVIUMNETWORKS.onmicrosoft.com header.i=@CAVIUMNETWORKS.onmicrosoft.com header.b="IJm3wWqa"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751716AbdEPS2I (ORCPT <rfc822;patchwork-incoming@ozlabs.org>); Tue, 16 May 2017 14:28:08 -0400 Received: from mail-by2nam03on0051.outbound.protection.outlook.com ([104.47.42.51]:40342 "EHLO NAM03-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750853AbdEPS2G (ORCPT <rfc822;netdev@vger.kernel.org>); Tue, 16 May 2017 14:28:06 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=UwkYel78DmFhVt2N6s8Js0NdX9fCXfVrdZQdE0uzTc8=; b=IJm3wWqa/j/s9R8KDPBU8JDSEJFkqVhBiu/QfZGiwdz6i4R63nuOPye6ti/kfg1izGz0M3MZsYuHWoHBS6xd6uZauJ3C1Uaf4snKyXp4zGbXNgbq3/BQTIEBtya5cHMzxODz8A1u8nZ3qCDqSh4VQOApaudwQlZd8baCdTJTw1g= Authentication-Results: davemloft.net; dkim=none (message not signed) header.d=none; davemloft.net; dmarc=none action=none header.from=cavium.com; Received: from localhost (73.71.186.126) by CY4PR07MB3176.namprd07.prod.outlook.com (10.172.115.142) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1084.16; Tue, 16 May 2017 18:28:05 +0000 Date: Tue, 16 May 2017 11:28:00 -0700 From: Felix Manlunas <felix.manlunas@cavium.com> To: davem@davemloft.net Cc: netdev@vger.kernel.org, raghu.vatsavayi@cavium.com, derek.chickles@cavium.com, satananda.burla@cavium.com Subject: [PATCH net-next] liquidio: fix PF falsely indicating success at setting MAC address of a nonexistent VF Message-ID: <20170516182800.GA1067@felix-thinkpad.cavium.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.6.1 (2016-04-27) X-Originating-IP: [73.71.186.126] X-ClientProxiedBy: MWHPR1201CA0015.namprd12.prod.outlook.com (10.174.253.25) To CY4PR07MB3176.namprd07.prod.outlook.com (10.172.115.142) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: fd5f9f61-f766-415b-227b-08d49c894b4a X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(201703131423075)(201703031133081); SRVR:CY4PR07MB3176; X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3176; 3:QdBKKebEt4mjP9MqoEAMd/+lc4Erhu1hfeSq7JeChSsgbc7z/b8hcom+BzxhIXlfcl2CUXefM6sOi9ypa1YmSw70WXXrO2/FVM368wdhaM3s2Ijt42XdAue4JAZvZnINOXygnHGPTFYLihLb0DguJfKng4XKjLIWrK3dT62F1oz4KtocZNF4EguCm86Md4cXPVljF3V9lMjrUClCrrCf79Rjsg27LJBvKBhClTS1yuc9EgQ1s/BISFkzev2r+ErGzoDGvb48Zys3KbD6wKUCrsFgaPor4O76DCUQjxNNLtSSjkK/7YDQlaW8seTG3nJpIm1L6xfvZZRRz0tRbzrhUg==; 25:dPB0S2ISXh/wnyuWMs/prHt4ew05VGIdxn2VkA96bioiuLrtf6FcEHUAEEIEwa0PKH00WRQsKNvoybtQN8ps5egzaX6FgckNlQPjx4nbcHRdIK/YBBQUuV2uGjaUIQwriK1c4qMZROSsPQVFOAdO3Ozm/QevKwLAuqxwpPZzC83pDqGyvOQcY5nWenT3Re/QRMwGzY5+v/bb8EK/3FGTVQRs0szqKq8QXYpc5kBmRN6VcOxhQDEVfX7ra1CYauk+syGAwrAzisiHd0iazjdPOIJ+c5DM3Ks2mo74WjyQN4R+PX1DU63F47rDc4Ua2I4zmlQ8dN4N3BfMaQOzCyFMZaNf1IXGXGxgrQxEBKZ7JyEjRvoKHwbxZj5Vpz8jYapRrWucMC54f5AX9Gr1JiGcruT5eZIOj73gE89odZumYB9hlSBUJPbE+z6niBMclh5XsvxWPHXw28r+QZYztGT5yQtmye0G7YhNKDqDmuDFu9s= X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3176; 31:rzyGrIesThtmpLXXmhFRINNtuls8OBrxfQk2usPCph+dhW3CtrqSBSpQwD4rKarrd2CgOh3DoaK89Re/1HgKNLlGXxuQt61E9xq10loW+kIdvCGXTJr0XkQVAwOjV6hCRg2fAtvwAMs95rGMs0D6XfSlJYZlwe9/SKAEXbFyPAvziPPKSmiOtklMJnL4hyzbJAeZnB3Dq3K8cJLJ+zRKfj+zvNFskJeh4V4ct423Iod6sOnwv2WWIpcd2ScqekIXDJTpzLFcypX+JzcE/m/uHQ==; 20:zJ3EnA0Nd4CJE/snHWmPaUBK/UcPOg3GpP07KeY0XX2f3sdmj+cGMiCc5NHSbEZcj1AhdH/YyhuZ7EBKeroldrG+DgmLHtbvCHU68o2Pn2UIPmuJWWMJptH3+IW0J58E6RjyqXlksBtsy7cJQqT+ewO2Ya3MEl069eig5S8CuRImzi9akX/SQIwZg1xz1AnK1FqhAFCTXaczii6tXtbBWO2LLeLftt0cZvb29NqKRukPpyIMIr2E1piN4CszNlI9LR8/4HdoteXCpPgQakmyQV0WxXONEwRk+yHC262zev3goPCclgVSzg0hvcNN3JUAqIM0xP6p5l1H1+5jT5V4NWYEctqtb+RiNGRIE2ct9q9b+hYnLtSidCyZOQOEFXlE6wWK4F/fLx6IrJzkkz9ydSsAQxGbePdRD5TJy6CIZ7B4kqgLAUjjUNejZNrfpbYNcptB7XnePCzyAD5zu/wEVQJJn2AT+81dw1KjpP01yJ1jZI4ljPt9B28IPzVnBNQU X-Microsoft-Antispam-PRVS: <CY4PR07MB3176999A03B9839E61543E98E7E60@CY4PR07MB3176.namprd07.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040450)(601004)(2401047)(8121501046)(5005006)(93006095)(93001095)(3002001)(10201501046)(6041248)(20161123564025)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123560025)(20161123555025)(20161123558100)(6072148); SRVR:CY4PR07MB3176; BCL:0; PCL:0; RULEID:; SRVR:CY4PR07MB3176; X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3176; 4:9LhXw/JjEmkxtK3X1pEv/96rDasi/M0SfYPnO/w+36LloBMP81giB+nWO4ORIWc+dxnI6aDGBG0U9gPx65yAJgGUgouoWSdmQ2z1aqpvu9VSDyRRA92taoYI6JXqILZC75NjSXCSHb6dtvN+adSoPcqI7IZbeIucqnKnkW7eqGEarsVOzfT69rsHR74vt3TrRhWlL1CJa5cASs/ldkcJ8vIwVDg2wgX6OnjfNVYskBXwYW1b/GSAkATUBhbhDSc8DmaQdb1hcSsG7zPO5vQ+3aTedudxEp8JvJVPrJgIChTAAW48yhhyw0LAem0euysO42irg5Ze3Hb1+piDCIs6pnUrvyCki1C6QjtBdGmB/83ZYDWgFn/Dfdp+7Y4SPNtEaH+zpOSP9JDd9HHg6LJ+U8/GqGsUBSBALo7w9Q1zOscw/YsSMCjQtTEO/WQVqIZIv3bgNxUQxoULfYS857SRIAvupvlghzXQh8Lvx9EOwQ2jcSxFFPpTdG+LmY1qGoqI49BsWUTC/Z/yOPRDLBJcFC0K/38bn+qPgGVTKkaXfsEhxA3g36CJvuJf6IbNw+9Y5TUH1+vq/A6jWwXunrBiussL+1stW8/XtgoyGCcZjZayIpdwFBNXJG5a6wmZGy0ZTdnzfVgI3uhJZXUHToC7ii8cQp5QjPQZMVnK+aDXONaSupUxFyHLaUF3n5IjNSxWaaxnwftlg4MJHkL7Ju6o5+kn+YSiRTC1Qx68On6KmpbodPdPeNEJCZJWRv7WCt5UPCzMJXvq1SKLdhvzaUo5r7BlkZjQqqmN+lxUdsKsBgQ= X-Forefront-PRVS: 03094A4065 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6009001)(6069001)(39410400002)(39840400002)(39850400002)(39400400002)(39450400003)(4001350100001)(66066001)(4326008)(47776003)(76506005)(54356999)(7736002)(42186005)(2906002)(6486002)(50986999)(2351001)(8676002)(2361001)(5660300001)(81166006)(189998001)(305945005)(478600001)(6666003)(6916009)(33656002)(86362001)(25786009)(72206003)(107886003)(6496005)(83506001)(3846002)(6116002)(1076002)(50466002)(110136004)(23726003)(53936002)(38730400002)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:CY4PR07MB3176; H:localhost; FPR:; SPF:None; MLV:sfv; LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; CY4PR07MB3176; 23:zVFeoRkn3z8U4W21N9lfgAeKvUGYnZwNlCMY8GwHA?= =?us-ascii?Q?O17B9QkDWwn5N7z2ztdZ3hWiFJg9QujSyeZnYhbx70O1o+T6+sd2VBWUtJ+a?= =?us-ascii?Q?dTbszHSPhvoXhiAWVDkzqhkKoUgZxqm3qKhOlNbSTUbTEfroBwcdsYELIpGX?= =?us-ascii?Q?Bsvw+LRv4rtEi1q6kGsUIXX/rPE3oOgmzjD515T/jE7KSac0kzqTKkPKjlev?= =?us-ascii?Q?6YpXvO73T+Btv8SF8KBt4+bkM5bHG768VblA27zq/c2t0PnzeoAJ84jWWRqK?= =?us-ascii?Q?A7fFKUP8e28EEMgzv596SJfragWxYw5JIoB3p99vK9Cwu237epfA6EjIA0Ik?= =?us-ascii?Q?P8nJJOU9OpqRQJLr4AVMQ/nr4FJ1Xru7tQzlQYAGoPHjn7ha7sy1WbZgt4Lu?= =?us-ascii?Q?ZntEH+e5NQEzpVLJSR+DnSPQL/larXlW3XljRV9NybGazaY9fSQOT/9ItSHv?= =?us-ascii?Q?Utkf13gzIA0bXUeAfNXYMYW/NuEJNqbmNkp7c2XfJ4pTGjL1XuszZVJ4qJ5J?= =?us-ascii?Q?lGOn8TZ7isrVQsmVFSaTmRZJ72VvnbKxPg4Q+YwdclidW22+xuUz9JzxiZus?= =?us-ascii?Q?yB6Sg4xJN7STO+OdCy1EMdmqxJ8dg5jYYo3uWE8kRN1Z3gWBkvKs5uqvsQEf?= =?us-ascii?Q?qjuuRLa4SzFitYwI+K+3CC1niPhBa4iZPXzT3Px7hayKT3c/r94o8+ls0T6x?= =?us-ascii?Q?VeUhwJHN9SJENCYp61IgtqxPBO8i5z2vt/92MxsIWjaTV+k04TiqytFVGiEC?= =?us-ascii?Q?BLP2fF/glp8QROMHoDJuQbal2hrlvEoj+Dnn6EHIe8EA5YAf/igqBkvlYWiQ?= =?us-ascii?Q?+WrnKXU2Q7KtGxWvVSa+RDUQvQVc4LNLg92v0ifXpXPEmrXYlYPbFITQxwMT?= =?us-ascii?Q?Xi12Kf6xVSMYEXFmUv0+Ct83I+AKNc0kt+RsvngNuU6/IbketMPbT8wIQrhX?= =?us-ascii?Q?yNAaTONurxV6VAV3bwZ7TlIBnBnB9wCXB80JWDG5IRZzhZxL7iVsxlj07g+m?= =?us-ascii?Q?HQASNUA5cpJpP0L7bap+r4YwaKq3R8l66XOl+72p8VRf3JVyPsZ20hqrrCZk?= =?us-ascii?Q?bUjxrPd0OLLh9+o0g6qcobRF00Jw80cLi81pQkDcOnyJzOyLbDfmzA79w6cK?= =?us-ascii?Q?7vMnWwnjek=3D?= X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3176; 6:l9uOtUc2TcH6aUcPJtbAuyCSkxITgaiq45cqcfAKdwjW4f82Y9FqH2rGSf766NJR+HoiNhtoXTugnNHYGlOSItehPFNQNEvxkvJLqApOZUuzo4nuk3Su64ah2uzzozixm4mmbNz/ilVg3rMu1fpFqVz6PCbEjW1BefvFIG0sig4ptKyu5X6MCeO0okafBPg0DeIZ8afwNCMxdRNotXGLsLwP07tUnqcXGltVnDpoXZ/HBYsbvynkDSkJ+BFJ845hzuAUDYHsvayOrZWfZzS26XFCDkfaJB5FIND5xTA6C6lSywGuagG9PyrTO7rTPfInddRrAZhaJ1NMZ3uyfA/JNfRcfg/ttFJmHEHRFDO6zFvhmuTn7mLmRBcgetfBHy0Fnyv1rg+aYPNDWriamCiDUrRgLNUBZkRC35MiZwD7HVYwso8Kd7wYYixAwdxTVMSZ1qWSK22whfQULUrFsb9bjbprQ2JeQgzkA4Mr9spJekG519ur67dC0Xg20HvHnPH/NSiq//owthTY56iyi1qpeA==; 5:Tv6aJ5TOHyO2/1UUIcEVSTLNsqpJTU4+IRZtHRMznyLeNolt5K/7s30HhRKdgSxgITrZ8q7Bh5+05XeH4LLBuVJ5KvHvHqH0WMcoxhHc3tj2z5gYALWNpOJZOAab41gnNPyKuCPprBsUcBeuq8EAyg==; 24:w5fsrWcNOxTKBsVfxV2lpzJZN3xgWh0tRuxtbtEwhsgCKs2e/cYRSOevWxSWW9ogB411SQ+UXT2/jvRkYvJUC2yGBDdmRBo+HeDqPrq8R3w= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3176; 7:63KaNTmJK7DAViTUmgDywKjT4yn7IZWH+ixhuApTMYtd6V/XF93As7uLaQDHnRUoI491VW1rfscVwwAR4LwDbml5WvVsI9se3VUJ+y6rWsEWjez7z3Te8qnAF3vFXsh9c88yuXlx0EV289eTl6Mp8IiIOnX3Glip5xp5UyaQS4Lg3/rBd92en18v0J8bF5F4Kh9z61IamzmuBSpOFkkxvI9iW7S8BWKNRp0c0qo3mvrCFXsIoQpDxMXommpPH4JwtHwXjPiPkcQzbjNuZSLD+q6l3u12A/rn8a70oag7Nt3PH8/OFuS9sdUFnM1CyQXHYhEkiuwo3jZjk87lqrr/gQ== X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 May 2017 18:28:05.0952 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR07MB3176 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: <netdev.vger.kernel.org> X-Mailing-List: netdev@vger.kernel.org |
From: Felix Manlunas <felix.manlunas@cavium.com> Date: Tue, 16 May 2017 11:28:00 -0700 > In the function assigned to .ndo_set_vf_mac, check the validity of the > vfidx argument before proceeding to tell the firmware to set the VF MAC > address. > > Signed-off-by: Felix Manlunas <felix.manlunas@cavium.com> > Signed-off-by: Derek Chickles <derek.chickles@cavium.com> Applied.
diff --git a/drivers/net/ethernet/cavium/liquidio/lio_main.c b/drivers/net/ethernet/cavium/liquidio/lio_main.c index 927617c..eed6ff0 100644 --- a/drivers/net/ethernet/cavium/liquidio/lio_main.c +++ b/drivers/net/ethernet/cavium/liquidio/lio_main.c @@ -3694,6 +3694,9 @@ static int liquidio_set_vf_mac(struct net_device *netdev, int vfidx, u8 *mac) struct octeon_device *oct = lio->oct_dev; int retval; + if (vfidx < 0 || vfidx >= oct->sriov_info.num_vfs_alloced) + return -EINVAL; + retval = __liquidio_set_vf_mac(netdev, vfidx, mac, true); if (!retval) cn23xx_tell_vf_its_macaddr_changed(oct, vfidx, mac);