lkml.org 
[lkml]   [2023]   [Oct]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v1 0/3] Add fixes for FastRPC driver
From


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)

\
 
 \ /
  Last update: 2023-10-01 16:41    [W:0.766 / U:0.080 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site