From patchwork Tue Dec 1 05:41:10 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Brent Taylor X-Patchwork-Id: 7732881 X-Patchwork-Delegate: kvalo@adurom.com Return-Path: X-Original-To: patchwork-linux-wireless@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id D941D9F39D for ; Tue, 1 Dec 2015 05:41:32 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 1451F20698 for ; Tue, 1 Dec 2015 05:41:32 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 4473C20684 for ; Tue, 1 Dec 2015 05:41:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755094AbbLAFlT (ORCPT ); Tue, 1 Dec 2015 00:41:19 -0500 Received: from mail-ig0-f172.google.com ([209.85.213.172]:37596 "EHLO mail-ig0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755028AbbLAFlR (ORCPT ); Tue, 1 Dec 2015 00:41:17 -0500 Received: by igcto18 with SMTP id to18so88599812igc.0; Mon, 30 Nov 2015 21:41:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id; bh=lQYG8Xt7NiWk1zVOFGl9vy8zNX4Pl3jXQdXDHN8cTnE=; b=xmDoyCf7CvDLIZi+ZBRGHLytfLiq1xU8G5KytiPM9QBcmNfjE1KqWeEI32n2r/GI7i jpaybPRonRmJK37jdbtDEe6d8nuJUgYGfdU5GWTN/tlgT7yY/HMHv2F0/C+181qqZjXB 1R9ei04hQUhX4cq8Qv81GO2MxDmJbyPtMki7wYjAjinojHdhEDozrGf2TsXKf4EEKHvP p4vBs1lIw9F7zBI6DOBIXsQ9bE24d23lE9ZqtBSXezYMhDS+jrsvmsngoRNo1Kn2P2lb zm3WDmV+09FF3Wl4b/9n3v8nMfpRp/YJlfwVVmp3OpHixGLwnX2EmzqGGA/67FDFkm9c /1TQ== X-Received: by 10.50.132.102 with SMTP id ot6mr1214324igb.18.1448948477343; Mon, 30 Nov 2015 21:41:17 -0800 (PST) Received: from cartman.taylor.net (c-73-168-44-213.hsd1.il.comcast.net. [73.168.44.213]) by smtp.gmail.com with ESMTPSA id 75sm18875555ios.35.2015.11.30.21.41.16 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 30 Nov 2015 21:41:16 -0800 (PST) From: Brent Taylor To: kvalo@qca.qualcomm.com Cc: linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, ath6kl@lists.infradead.org, Brent Taylor Subject: [PATCH] ath6kl: Use vmalloc to allocate ar->fw for api1 method Date: Mon, 30 Nov 2015 23:41:10 -0600 Message-Id: <1448948470-31111-1-git-send-email-motobud@gmail.com> X-Mailer: git-send-email 2.6.3 Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org X-Spam-Status: No, score=-6.8 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, RCVD_IN_DNSWL_HI, T_DKIM_INVALID, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Since commit 8437754c83351d6213c1a47ff029c1126d6042a7, ar->fw is expected to be pointing to memory allocated by vmalloc. If the api1 method (via ath6kl_fetch_fw_api1) is used to allocate memory for ar->fw, then kmemdup is used. This patch checks if the firmware being loaded is the 'fw' image, then use vmalloc, otherwise use kmalloc. Signed-off-by: Brent Taylor --- drivers/net/wireless/ath/ath6kl/init.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath6kl/init.c b/drivers/net/wireless/ath/ath6kl/init.c index 6ae0734..4f2b124d 100644 --- a/drivers/net/wireless/ath/ath6kl/init.c +++ b/drivers/net/wireless/ath/ath6kl/init.c @@ -673,10 +673,15 @@ static int ath6kl_get_fw(struct ath6kl *ar, const char *filename, return ret; *fw_len = fw_entry->size; - *fw = kmemdup(fw_entry->data, fw_entry->size, GFP_KERNEL); + if (&ar->fw == fw) + *fw = vmalloc(fw_entry->size); + else + *fw = kmalloc(fw_entry->size, GFP_KERNEL); if (*fw == NULL) ret = -ENOMEM; + else + memcpy(*fw, fw_entry->data, fw_entry->size); release_firmware(fw_entry);