From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f47.google.com (mail-lf1-f47.google.com [209.85.167.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EB44436F for ; Sat, 13 Jan 2024 08:31:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ragnatech.se Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ragnatech.se Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=ragnatech.se header.i=@ragnatech.se header.b="Acm24Q4k" Received: by mail-lf1-f47.google.com with SMTP id 2adb3069b0e04-50e5a9bcec9so8833865e87.3 for ; Sat, 13 Jan 2024 00:31:21 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ragnatech.se; s=google; t=1705134680; x=1705739480; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=0gF3sw/Ea39zWxuD0gLzKFUce26V5brwlX/JeoC/CYQ=; b=Acm24Q4k027vz2xM+HhYdbmskOzawnZ2RHCM/5Yr7kdOSHZkEthTSDtiMl3IvuqpSI S+7b5YJZ6FsrBQNvQLCuwxjDBOAJsidCbMWm9hgFVl9cEHKCT/1uSesVOWuAcKTt9UhV wDHLbzX94KH2Tolvn6Lp87OHLrOCx+K6XCF3xRFnpZfUPKI+H+6R/BTG1U57b0rMBxCz FzW9ULLg7YVGAJVJzCde6c/Pjd4luIhmNC52QgXyA02hYxJVWP7oxHd/y1vO2aQxG1qx +tEogBfsWosntcMyCDiE+NmUjhCmZcwrWGiOQdjIaWhkvSXx7w2bIn/eME4x3qJ78o5I CwWg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1705134680; x=1705739480; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=0gF3sw/Ea39zWxuD0gLzKFUce26V5brwlX/JeoC/CYQ=; b=lALO7uCfzBn2P04VIeCTN4Qv6eVzGbDkxFsl+YVbFlaJPlU7BiEnAGCibVpG/gpTk3 F7j3FV3s5R20jbY0oOUUiVlOSMzDv4cF/rfS3W38moxVa1zdN2rZLwu3MHjQqsFtPPW4 k4PUvMWRjWhWEAfcWEs5jbpe4SpO/3bkxTPloFWjQpZyW2fBE8sqE3Ywyv86Z00Xs9kg R4yxm2/5L2LRuHVA4TcOUY4L1C0msII4LmSBF3NjBMyZklT2m7vejFmclI1LK91T77zW +o+vjfyJ2Bqj3XlT9vBg/4nl06sg3d6Wac/EATxslRZglIfTpZg81FbMksy8u3A6lwgN Z7XQ== X-Gm-Message-State: AOJu0Yz/lViJXAfJQJlslHbXfJQr8y9V3uib0m4I+LN6pSAEa9/z4BBw X500GfXR43WhkQUXW6qiZjYV/ehnizH2pQ== X-Google-Smtp-Source: AGHT+IFclI/5HicsKwUTEr7skf591nEgQRXaAa9kiQQu/l4k9zf0rvOYj4zDXmS1hH+xO3HBGCxOkw== X-Received: by 2002:a05:6512:390e:b0:50e:74ec:75f6 with SMTP id a14-20020a056512390e00b0050e74ec75f6mr1161110lfu.136.1705134679578; Sat, 13 Jan 2024 00:31:19 -0800 (PST) Received: from localhost (h-46-59-36-113.A463.priv.bahnhof.se. [46.59.36.113]) by smtp.gmail.com with ESMTPSA id x26-20020a19f61a000000b0050e7e21b338sm773810lfe.14.2024.01.13.00.31.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 Jan 2024 00:31:18 -0800 (PST) Date: Sat, 13 Jan 2024 09:31:17 +0100 From: Niklas =?utf-8?Q?S=C3=B6derlund?= To: Nikita Yushchenko Cc: "David S. Miller" , Jakub Kicinski , Paolo Abeni , Sergey Shtylyov , Claudiu Beznea , Yoshihiro Shimoda , Wolfram Sang , Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= , netdev@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] net: ravb: Fix dma_addr_t truncation in error case Message-ID: <20240113083117.GI1643864@ragnatech.se> References: <20240113042221.480650-1-nikita.yoush@cogentembedded.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20240113042221.480650-1-nikita.yoush@cogentembedded.com> Hello Nikita, Thanks for your patch. On 2024-01-13 10:22:21 +0600, Nikita Yushchenko wrote: > In ravb_start_xmit(), ravb driver uses u32 variable to store result of > dma_map_single() call. Since ravb hardware has 32-bit address fields in > descriptors, this works properly when mapping is successful - it is > platform's job to provide mapping addresses that fit into hardware > limitations. > > However, in failure case dma_map_single() returns DMA_MAPPING_ERROR > constant that is 64-bit when dma_addr_t is 64-bit. Storing this constant > in u32 leads to truncation, and further call to dma_mapping_error() > fails to notice the error. > > Fix that by storing result of dma_map_single() in a dma_addr_t > variable. > > Fixes: c156633f1353 ("Renesas Ethernet AVB driver proper") > Signed-off-by: Nikita Yushchenko Reviewed-by: Niklas Söderlund > --- > drivers/net/ethernet/renesas/ravb_main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/renesas/ravb_main.c b/drivers/net/ethernet/renesas/ravb_main.c > index 8649b3e90edb..0e3731f50fc2 100644 > --- a/drivers/net/ethernet/renesas/ravb_main.c > +++ b/drivers/net/ethernet/renesas/ravb_main.c > @@ -1949,7 +1949,7 @@ static netdev_tx_t ravb_start_xmit(struct sk_buff *skb, struct net_device *ndev) > struct ravb_tstamp_skb *ts_skb; > struct ravb_tx_desc *desc; > unsigned long flags; > - u32 dma_addr; > + dma_addr_t dma_addr; > void *buffer; > u32 entry; > u32 len; > -- > 2.39.2 > > -- Kind Regards, Niklas Söderlund