From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 52BB3C433DF for ; Tue, 18 Aug 2020 20:15:05 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 367D020789 for ; Tue, 18 Aug 2020 20:15:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726560AbgHRUPD (ORCPT ); Tue, 18 Aug 2020 16:15:03 -0400 Received: from vps0.lunn.ch ([185.16.172.187]:59890 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725554AbgHRUPD (ORCPT ); Tue, 18 Aug 2020 16:15:03 -0400 Received: from andrew by vps0.lunn.ch with local (Exim 4.94) (envelope-from ) id 1k880b-009ydn-Lg; Tue, 18 Aug 2020 22:15:01 +0200 Date: Tue, 18 Aug 2020 22:15:01 +0200 From: Andrew Lunn To: David Awogbemila Cc: netdev@vger.kernel.org, Catherine Sullivan , Yangchun Fu Subject: Re: [PATCH net-next 04/18] gve: Add support for dma_mask register Message-ID: <20200818201501.GL2330298@lunn.ch> References: <20200818194417.2003932-1-awogbemila@google.com> <20200818194417.2003932-5-awogbemila@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200818194417.2003932-5-awogbemila@google.com> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Tue, Aug 18, 2020 at 12:44:03PM -0700, David Awogbemila wrote: > From: Catherine Sullivan > > Add the dma_mask register and read it to set the dma_masks. > gve_alloc_page will alloc_page with: > GFP_DMA if priv->dma_mask is 24, > GFP_DMA32 if priv->dma_mask is 32. This needs reviewing by somebody who knows this stuff. My limited understanding is the core should take care of most of this, so long as you tell it of any device restrictions. Andrew