lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <52028f1a-e7f4-0ffe-e359-d95fbbc86cf0@quicinc.com>
Date:   Thu, 20 Apr 2023 12:42:24 +0530
From:   Mukesh Ojha <quic_mojha@...cinc.com>
To:     Sarannya S <quic_sarannya@...cinc.com>,
        <quic_bjorande@...cinc.com>, <arnaud.pouliquen@...s.st.com>,
        <swboyd@...omium.org>, <quic_clew@...cinc.com>,
        <mathieu.poirier@...aro.org>
CC:     <linux-kernel@...r.kernel.org>, <linux-arm-msm@...r.kernel.org>,
        <linux-remoteproc@...r.kernel.org>
Subject: Re: [PATCH V6 0/3] rpmsg signaling/flowcontrol patches



On 4/20/2023 11:51 AM, Sarannya S wrote:
> Added new IOCTLS- RPMSG_GET_OUTGOING_FLOWCONTROL and RPMSG_SET_INCOMING_FLOWCONTROL
> to get/set the rpmsg char device's flow control signal.
> Updated the cmd name 'RPM_CMD_SIGNALS' to 'GLINK_CMD_SIGNALS'.
> Changed 'remote_flow' to bool type, and updated it's evaluation.
> Updated evaluation of 'set' in rpmsg_char as 'set = !!arg'.
> Add destination address of endpoint as parameter in rpmsg_set_flow_control.
> Addressed review comments to change variable names/descriptions.

Something to notice, you are not changing the version number as you have 
sent v6 here again.

Also maintain the changes happened from the last patch series properly.
Like

Changes in v6:
  - ..
  - ..

Change in v5: (a lore kernel link will be helpful here for reviewer to 
remind the conversation happened during last review)
  - ..
  - ..



--Mukesh

> 
> Chris Lew (2):
>    rpmsg: glink: Add support to handle signals command
>    rpmsg: char: Add RPMSG GET/SET FLOWCONTROL IOCTL support
> 
> Deepak Kumar Singh (1):
>    rpmsg: core: Add signal API support
> 
>   drivers/rpmsg/qcom_glink_native.c | 64 +++++++++++++++++++++++++++++++++++++++
>   drivers/rpmsg/rpmsg_char.c        | 49 ++++++++++++++++++++++++++----
>   drivers/rpmsg/rpmsg_core.c        | 21 +++++++++++++
>   drivers/rpmsg/rpmsg_internal.h    |  2 ++
>   include/linux/rpmsg.h             | 15 +++++++++
>   include/uapi/linux/rpmsg.h        | 11 ++++++-
>   6 files changed, 155 insertions(+), 7 deletions(-)
> 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ