From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1a9IiE-0005yG-RI for ath10k@lists.infradead.org; Wed, 16 Dec 2015 20:30:15 +0000 Message-ID: <5671C99A.703@codeaurora.org> Date: Wed, 16 Dec 2015 12:29:14 -0800 From: Peter Oh MIME-Version: 1.0 Subject: Re: [PATCH] ath10k: add modparam 'hw_csum' to make HW checksum configurable References: <1450290051-15593-1-git-send-email-poh@qca.qualcomm.com> <5671AD10.70004@openwrt.org> In-Reply-To: <5671AD10.70004@openwrt.org> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Felix Fietkau , Peter Oh , ath10k@lists.infradead.org Cc: linux-wireless@vger.kernel.org On 12/16/2015 10:27 AM, Felix Fietkau wrote: > On 2015-12-16 19:20, Peter Oh wrote: >> Some hardwares such as QCA988X and QCA99X0 doesn't have >> capability of checksum offload when frame formats are not >> suitable for it such as Mesh frame. >> Hence add a module parameter, hw_csum, to make checksum offload >> configurable during module registration time. >> >> Signed-off-by: Peter Oh > How about instead of inventing yet another crappy module parameter, you > call skb_checksum_help() in the driver in cases where the hardware is > unable to offload the checksum calculation. > > That way the user has to worry about less driver specific hackery ;) That will be good option for hardware not supporting HW checksum, but I mind that using the function will add more workload per every packet on critical data path when HW supports checksum resulting in throughput down. > - Felix > -- > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html Thanks, Peter _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k