On Fri, Oct 28, 2016 at 03:54:25PM +0300, Or Gerlitz wrote: > On Thu, Oct 27, 2016 at 4:36 PM, Leon Romanovsky <leon@xxxxxxxxxx> wrote: > > From: Max Gurtovoy <maxg@xxxxxxxxxxxx> > > > > Replace the pre-defined macro signifying inline umr instead > > of the numerical constant. > > Leon, > > By all means (or no means, choose), this is a fix. If you are telling > vmware and Broadcom people here for months how to write their code, > let's stop for a minute and look in the mirror, drop this patch. Thanks Or, I appreciate your time invested in the review and providing so much valuable feedback. Since change of numeric value to the same value defined by macro is not a fix but improvement and provided here to simplify maintainer's work (Doug), I have no plans to change the patch or/and drop it. > > Or. > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html
Attachment:
signature.asc
Description: PGP signature