Re: [PATCH] Net: ceph: osd_client: Remove con argument in handle_reply

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Oct 18, 2015 at 10:25 AM, Shraddha Barke
<shraddha.6596@xxxxxxxxx> wrote:
> Since the function handle_reply does not use it's con argument,
> remove it.
>
> Signed-off-by: Shraddha Barke <shraddha.6596@xxxxxxxxx>
> ---
>  net/ceph/osd_client.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c
> index 80b94e3..e281f60 100644
> --- a/net/ceph/osd_client.c
> +++ b/net/ceph/osd_client.c
> @@ -1745,8 +1745,7 @@ static void complete_request(struct ceph_osd_request *req)
>   * handle osd op reply.  either call the callback if it is specified,
>   * or do the completion to wake up the waiting thread.
>   */
> -static void handle_reply(struct ceph_osd_client *osdc, struct ceph_msg *msg,
> -                        struct ceph_connection *con)
> +static void handle_reply(struct ceph_osd_client *osdc, struct ceph_msg *msg)
>  {
>         void *p, *end;
>         struct ceph_osd_request *req;
> @@ -2802,7 +2801,7 @@ static void dispatch(struct ceph_connection *con, struct ceph_msg *msg)
>                 ceph_osdc_handle_map(osdc, msg);
>                 break;
>         case CEPH_MSG_OSD_OPREPLY:
> -               handle_reply(osdc, msg, con);
> +               handle_reply(osdc, msg);
>                 break;
>         case CEPH_MSG_WATCH_NOTIFY:
>                 handle_watch_notify(osdc, msg);

Applied, with a slightly tweaked change log - see
https://github.com/ceph/ceph-client/commit/81c842af2b39009ec1f8272a33b149ad0e845aaf.

Thanks,

                Ilya
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux