From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f52.google.com (mail-wr1-f52.google.com [209.85.221.52]) (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 360E218784A for ; Mon, 30 Dec 2024 20:32:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1735590754; cv=none; b=uMQnJQWhxiQdwTP+dWH0K9aTI+gL+4jqMK0/NyrNNJwznIq+3lAVVHzuo0kie3Q0eazJCI5GgvtB6iQUHNc66Tb9PeYCMfuLWc9LTdOiFvhkXoSx80ddOyMbHYqTupnzw1lW6VNmRPChzNIEseFxxboj3UuhuvXa8zo/j0g9A5s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1735590754; c=relaxed/simple; bh=bZYw2r5oOyrmTwDfLz6CNZZIKHc7PL13H43dAGE315E=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=jIQFoKVTev2dPoCsgzMP2ethieEEeLsr0nk54mmKTOcp6z9XTE5uFxdtX65q5oKJHDA4FAbv2j5AB+sGuZ2DDbXplC3E1TTB+J8mcwqcOB929rA9fNjaEffUK2h/GRCa3He6229g9RayaL8MHEsZMHVBD0/FUnmrb0dDHso0qWo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=bshTT+hf; arc=none smtp.client-ip=209.85.221.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="bshTT+hf" Received: by mail-wr1-f52.google.com with SMTP id ffacd0b85a97d-385e87b25f0so6642430f8f.0 for ; Mon, 30 Dec 2024 12:32:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1735590751; x=1736195551; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=3pq0s3ckIiTlqXik31Tw6lXUUphfCKc9DjHpksYo6NA=; b=bshTT+hfGWwLSlYr1AS75D2MVfndFzPOTEvdoQG432ny9BIFc5l6wgP4T18DDF8NZl tYn/dSnBJvYHwRjK0n9d+fv+/PaLwLGDi4RdDxxVHhsBKmsDSX6INWLez8/4mlNTt8nm llNyLDWw1ERViMGRK++hC1hk+Iw+PNlYDhQgGrqfGkwZ375cMOl5bGMCG94UCGyTAoq5 GXMF7LhCgfAGwGiNOKtv/F6oq+nyRGcaO0I/Dx+3K6c/PS/s/Pp0hr3a66Y7H3MRQLH6 IbiQkhqG0f2EA5xhcINXenHmSiesE7cB/uFT8ri5kz2qG8PcgZNTmIeaeHZ+y/XmM+3d 4D/Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1735590751; x=1736195551; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=3pq0s3ckIiTlqXik31Tw6lXUUphfCKc9DjHpksYo6NA=; b=BemiXjRD//UcUDGptAmw4J3c7Pqrxxmm01VfdQWrZDv2bjWtQhIbWOP1qpmZyVvtid jeuVnAHMwbu+X1vVtsclzEiZNRr8OjBlxDYWZZQQ5LyzJ5CIbEifbwgAlfZYloevwaWV 5nXxEAOpZbLqmeGrZp+VdgYQd3o4qy6TdD3nj92XjRDqwCsXr4cBcQf9/fjjliInkaQt kYevkBOeXkaX4jN7TDfp48uq+hizuPC5Rvn7HL/VpqXkKJl7+1itCY4QY58xwo+/f0ZF cWmcjGcmDJp/7ghJXZTDi65AeYWcHMXKG+XvkIUP1Ic2zF8lc7BZE5KOQSw+9iePz6OG d3cw== X-Forwarded-Encrypted: i=1; AJvYcCUtVjSrxk80otOFcoNipE6muIveqMOCdSTJCFH6rqL1Ujfpc5wbpWI4+RsWSX4P5b/hDbxSd1aYenQd3Wg=@vger.kernel.org X-Gm-Message-State: AOJu0YwKgsdrN3MgaT8ZG0DhRlty3l9/VfyjMhpf7AUnSFzT5xLG9MSN r2Y51d/SRumd61v4nNPB5KhFOANvL98MW0ey/L8jF/CxMS6HTBfGERiv+8twf6o= X-Gm-Gg: ASbGncuQ4+Z78kKrciNCMeL+rsgq8xy91O1ZLbuPBy4tp0O9gFSSBldvlNkZIjtB42g A2k99VgR2/pjr844UC0OiFm4JfrjGw6MKwW07GqQ21dTgIup1Ev1o3Z1zp7axmdHyw4wP20Wsr5 W5en0o3oFpT4BXou/0YCAf3QVc/m20YlkIXWAXOmK1MeWvDHQ5iiPouJ/lw4GyXls2EQiwW15kw gyCDfNi3v5ZYikxWTunOwTuZUEMZ3ZuBrApcRz/up9BjQKbdnp7l8idqUv3W+gU+fSO+VypkVk= X-Google-Smtp-Source: AGHT+IGnMunM/UAWS3Rwrx3prMio8qRf5DD/jYD0DbMIHgWiFAB26vYnuoJYyxLlaG3R4f9Ruoy8wQ== X-Received: by 2002:a05:6000:703:b0:385:f7e5:de88 with SMTP id ffacd0b85a97d-38a229ec5b0mr27423543f8f.3.1735590751612; Mon, 30 Dec 2024 12:32:31 -0800 (PST) Received: from [192.168.68.114] ([5.133.47.210]) by smtp.googlemail.com with ESMTPSA id ffacd0b85a97d-38a1c84840asm30889572f8f.61.2024.12.30.12.32.30 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 30 Dec 2024 12:32:31 -0800 (PST) Message-ID: <8dee006c-cc1a-4274-8691-2d58372bc022@linaro.org> Date: Mon, 30 Dec 2024 20:32:30 +0000 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v1 0/2] Add missing fixes in fastrpc_get_args To: Dmitry Baryshkov Cc: linux-arm-msm@vger.kernel.org, Ekansh Gupta , gregkh@linuxfoundation.org, quic_bkumar@quicinc.com, linux-kernel@vger.kernel.org, quic_chennak@quicinc.com, dri-devel@lists.freedesktop.org, arnd@arndb.de References: <20241218102429.2026460-1-quic_ekangupt@quicinc.com> <173557534277.273714.16861047953843054499.b4-ty@linaro.org> Content-Language: en-US From: Srinivas Kandagatla In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 30/12/2024 18:22, Dmitry Baryshkov wrote: > On Mon, Dec 30, 2024 at 04:15:42PM +0000, Srinivas Kandagatla wrote: >> >> On Wed, 18 Dec 2024 15:54:27 +0530, Ekansh Gupta wrote: >>> This patch series adds the listed bug fixes that have been missing >>> in upstream fastRPC driver: >>> - Page address for registered buffer(with fd) is not calculated >>> properly. >>> - Page size calculation for non-registered buffer(copy buffer) is >>> incorrect. >>> >>> [...] >> >> Applied, thanks! > > May I ask, why they are being accepted with the obvious checkpatch > warnings? If you are referring to this warning. WARNING: Invalid email format for stable: 'stable ', prefer 'stable@kernel.org' I tend to fix such small warnings before applying. These are fixed now. > > What kind of process is being followed, as those patches had review > comments to be implemented in the next iteration. I apply these patches if it looks good to me. This also helps with getting it tested from wider audience via linux-next. I do run TFLite workloads before it ends up in char-misc, but not for every patch. sorry If I missed any blocker comments, but your comments were more on the cover letter content and asking about the work loads which triggers these bugs. Are these patches breaking any of your test-cases? --srini