Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.36 as permitted sender) client-ip=23.128.96.36;
Message-ID: <230ca044-99c7-45df-9947-4605b8c0d295@linaro.org>
Date:   Sun, 1 Oct 2023 15:40:30 +0100
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v1 0/3] Add fixes for FastRPC driver
Content-Language: en-US
To:     Ekansh Gupta <quic_ekangupt@quicinc.com>,
        srinivas.kandagatla@linaro.org, linux-arm-msm@vger.kernel.org
Cc:     ekangupt@qti.qualcomm.com, gregkh@linuxfoundation.org,
        linux-kernel@vger.kernel.org, fastrpc.upstream@qti.qualcomm.com
References: <1695973360-14369-1-git-send-email-quic_ekangupt@quicinc.com>
From:   Caleb Connolly <caleb.connolly@linaro.org>
In-Reply-To: <1695973360-14369-1-git-send-email-quic_ekangupt@quicinc.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 29/09/2023 08:42, Ekansh Gupta wrote:
> This patchset carries the following fixes.
> - Reset metadata buffer to avoid incorrect fd getting freed
> - Free DMA handles for remote calls with no arguments
> - Clean buffers on remote call failures

Bit of a nit pick here, but you should try and use the cover letter to 
explain the necessary context and justification for your series. An 
overview of the commits is already included below by b4 or git-send-email.

It would be good to establish if these fixes are all related in some way 
(ie to enable a specific usecase), or if it's just a few random things 
that folks have discovered. Although it's almost always useful for 
maintainers to know how these issues where discovered as it could expose 
a gap in testing.

Regards,

> 
> Ekansh Gupta (3):
>    misc: fastrpc: Reset metadata buffer to avoid incorrect free
>    misc: fastrpc: Free DMA handles for RPC calls with no arguments
>    misc: fastrpc: Clean buffers on remote invocation failures
> 
>   drivers/misc/fastrpc.c | 26 ++++++++++++--------------
>   1 file changed, 12 insertions(+), 14 deletions(-)
> 

-- 
// Caleb (they/them)