All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yuval Shaia <yuval.shaia-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
To: Adit Ranadive <aditr-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org,
	linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	pv-drivers-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org
Subject: Re: [PATCH 3/3] IB/vmw_pvrdma: Dont hardcode QP header page
Date: Thu, 12 Jan 2017 08:44:01 +0200	[thread overview]
Message-ID: <20170112064400.GA3552@yuval-lap> (raw)
In-Reply-To: <258eb6bc-1322-e30e-7571-bd214975a08d-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>

On Wed, Jan 11, 2017 at 02:27:31PM -0800, Adit Ranadive wrote:
> On Wed, Jan 11, 2017 at 09:09:53AM +0200, Yuval Shaia wrote:
> > On Tue, Jan 10, 2017 at 11:15:41AM -0800, Adit Ranadive wrote:
> > > Moved the header page count to a macro [1]. In a future patch we will
> > > separate out the allocation for the header page. Thanks Yuval.
> > > Also, clear out the alloc_ucontext user response [2]. Thanks Dan.
> > 
> > Though i appreciate the above acknowledgment i'm not sure it can be part of
> > the commit message.
> 
> Okay.
> 
> Doug, if possible please update the commit message as follows:
> --
> Moved the header page count to a macro. Also, clear out the alloc_ucontext
> user response.
> --

With this commit message fix:

Reviewed-by: Yuval Shaia <yuval.shaia-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>

> 
> If you prefer I can send a new series with the updated message.
> 
> Thanks,
> Adit
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

      parent reply	other threads:[~2017-01-12  6:44 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-10 19:15 [PATCH 0/3] IB/vmw_pvrdma patches for 4.10 Adit Ranadive
     [not found] ` <cover.1484075557.git.aditr-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-10 19:15   ` [PATCH 1/3] IB/vmw_pvrdma: Defer activating device until vmxnet3 link is up Adit Ranadive
     [not found]     ` <4b515ded56300f12cedc67253d42ab8fbc52134e.1484075557.git.aditr-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-11  8:31       ` Yuval Shaia
2017-01-11 22:19         ` Adit Ranadive
2017-01-16  7:38       ` Leon Romanovsky
     [not found]         ` <20170116073858.GA25853-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org>
2017-01-18  0:31           ` Adit Ranadive
     [not found]             ` <35f6cce2-00c8-16b6-e3e0-481f234bd9c9-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-18  4:21               ` Parav Pandit
     [not found]                 ` <VI1PR0502MB30081629AE674E7F24337835D17F0-o1MPJYiShExKsLr+rGaxW8DSnupUy6xnnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
2017-01-18 18:30                   ` Adit Ranadive
     [not found]                     ` <ad2f92cd-f120-b04a-0e94-b49915cadf02-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-18 18:42                       ` Jason Gunthorpe
     [not found]                         ` <20170118184236.GA4864-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2017-01-18 20:25                           ` Doug Ledford
2017-01-18 20:30                           ` Adit Ranadive
     [not found]                             ` <c44cf5cf-6096-f2df-df79-e1c8da924e84-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-18 20:41                               ` Jason Gunthorpe
     [not found]                                 ` <20170118204130.GA6422-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2017-01-18 20:52                                   ` Adit Ranadive
     [not found]                                     ` <caee0761-e791-a56b-e8ed-609c15bdc37a-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-18 22:50                                       ` Parav Pandit
     [not found]                                         ` <VI1PR0502MB3008DAF4097F78DB313AE73CD17F0-o1MPJYiShExKsLr+rGaxW8DSnupUy6xnnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
2017-01-19  0:31                                           ` Adit Ranadive
     [not found]                                             ` <7b3e522d-b70c-b09b-19c7-358bb5efcc91-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-19  6:45                                               ` Doug Ledford
     [not found]                                                 ` <1484808317.2406.89.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-01-19 20:09                                                   ` Adit Ranadive
2017-01-18 20:23               ` Doug Ledford
     [not found]                 ` <1484770990.2406.46.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-01-18 20:41                   ` Adit Ranadive
2017-01-10 19:15   ` [PATCH 2/3] IB/vmw_pvrdma: Cleanup unused variables Adit Ranadive
     [not found]     ` <8631bae587fe5c6b63f8c1f486d79f564041ddae.1484075557.git.aditr-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-11  7:29       ` Yuval Shaia
2017-01-10 19:15   ` [PATCH 3/3] IB/vmw_pvrdma: Dont hardcode QP header page Adit Ranadive
     [not found]     ` <2035e9eca59810687d9b53d7d6e603b765729b1f.1484075557.git.aditr-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-11  7:09       ` Yuval Shaia
     [not found]         ` <20170111070952.GB5620-Hxa29pjIrETlQW142y8m19+IiqhCXseY@public.gmane.org>
2017-01-11 22:27           ` Adit Ranadive
     [not found]             ` <258eb6bc-1322-e30e-7571-bd214975a08d-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
2017-01-12  6:44               ` Yuval Shaia [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170112064400.GA3552@yuval-lap \
    --to=yuval.shaia-qhclzuegtsvqt0dzr+alfa@public.gmane.org \
    --cc=aditr-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org \
    --cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=pv-drivers-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.