From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752805AbcKQWYo (ORCPT ); Thu, 17 Nov 2016 17:24:44 -0500 Received: from mail-oi0-f66.google.com ([209.85.218.66]:35991 "EHLO mail-oi0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752190AbcKQWYm (ORCPT ); Thu, 17 Nov 2016 17:24:42 -0500 MIME-Version: 1.0 In-Reply-To: <20161117200203.GQ4240@leon.nu> References: <58466423-c87e-3921-101e-bffab8989fd8@redhat.com> <20161117184950.GP4240@leon.nu> <582E089A.3040106@redhat.com> <20161117200203.GQ4240@leon.nu> From: Or Gerlitz Date: Fri, 18 Nov 2016 00:24:40 +0200 Message-ID: Subject: Re: [PULL REQUEST] Please pull rdma.git To: Doug Ledford Cc: linux-rdma , Linux Kernel Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Nov 17, 2016 at 9:44 PM, Doug Ledford wrote: > On 11/17/16 1:49 PM, Leon Romanovsky wrote: >> On Thu, Nov 17, 2016 at 07:13:54AM -0500, Doug Ledford wrote: >>> Hi Linus, >>> >>> Due to various issues, I've been away and couldn't send a pull request >>> for about three weeks. There were a number of -rc patches that built up >>> in the meantime (some where there already from the early -rc stages). >>> Obviously, there were way too many to send now, so I tried to pare the >>> list down to the more important patches for the -rc cycle. Hi Doug, Except for the hfi1 patches, all the rest (core, mlx5, mlx4 and rxe) are marked now as only 21 hours old in your 4.9-rc branch and they seems be made from you picking partial subsets of multiple series, with none of them acked by you on the list. If you agree that I am describing things correctly - how are we expected to follow on your patch picking? I find it sort of impossible and error prone. >> Are you adding the rest to your for-next branch? We would like to have >> enough time to check that nothing is lost. > Yes, it's already there in the mlx-next branch on github. Re the patches there, this one IB/mlx4: Set traffic class in AH "Set traffic class within sl_tclass_flowlabel when create iboe AH. Without this the TOS value will be empty when running VLAN tagged traffic, because the TOS value is taken from the traffic class in the address handle attributes. Fixes: 9106c41 ('IB/mlx4: Fix SL to 802.1Q priority-bits mapping for IBoE')" claims to fix my commit, I have approached Leon and Co for clarifications/questions over the list on the patch and nothing was answered. Please do not send it to Linus and wait for them to respond. I disagree that it fixes my commit b/c my commit was prior to when route-able RoCE was introduced and on that time TOS had no relation. and this one "IB/mlx4: Put non zero value in max_ah device attribute Use INT_MAX since this is the max value the attribute can hold, though hardware capability is unlimited. Fixes: 225c7b1 ('IB/mlx4: Add a driver Mellanox ConnectX InfiniBand adapters')" does a tiny enhancement for a 10y old commit of Roland, why you think we need it in 4.9-rc6 or 7?? Or.