Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754807AbbLGIsg (ORCPT ); Mon, 7 Dec 2015 03:48:36 -0500 Received: from mail-wm0-f43.google.com ([74.125.82.43]:35100 "EHLO mail-wm0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753792AbbLGIse (ORCPT ); Mon, 7 Dec 2015 03:48:34 -0500 Subject: Re: [PATCH] infiniband:core:Add needed error path in cm_init_av_by_path To: Haggai Eran , Nicholas Krause , "dledford@redhat.com" References: <1449252173-12131-1-git-send-email-xerofoify@gmail.com> Cc: "sean.hefty@intel.com" , "hal.rosenstock@gmail.com" , "jgunthorpe@obsidianresearch.com" , Matan Barak , "ted.h.kim@oracle.com" , Doron Tsur , Erez Shitrit , "david.ahern@oracle.com" , "linux-rdma@vger.kernel.org" , "linux-kernel@vger.kernel.org" From: Michael Wang Message-ID: <566547DF.9040509@profitbricks.com> Date: Mon, 7 Dec 2015 09:48:31 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1707 Lines: 38 On 12/07/2015 08:57 AM, Haggai Eran wrote: > On Friday, December 4, 2015 8:02 PM, Nicholas Krause wrote: >> To: dledford@redhat.com >> Cc: sean.hefty@intel.com; hal.rosenstock@gmail.com; Haggai Eran; jgunthorpe@obsidianresearch.com; Matan Barak; yun.wang@profitbricks.com; ted.h.kim@oracle.com; Doron Tsur; Erez Shitrit; david.ahern@oracle.com; linux-rdma@vger.kernel.org; linux-kernel@vger.kernel.org >> Subject: [PATCH] infiniband:core:Add needed error path in cm_init_av_by_path >> >> This adds a needed error path in the function, cm_init_av_by_path >> after the call to ib_init_ah_from_path in order to avoid incorrectly >> accessing the path pointer of structure type ib_sa_path_rec if this >> function call fails to complete its intended work successfully by >> returning a error code. >> >> Signed-off-by: Nicholas Krause > > The subject doesn't seem to match the convention but apart from that, > > Reviewed-by: Haggai Eran > > I wonder if this should go to stable. If I understand correctly, this will fail only when the SGID isn't found in the GID table, but such connections would fail later on when creating a QP, right? Me too think this need a reconsider, to me the current logical don't really care the missing gid in cache when initializing AV, I'm not sure if it's necessary to fail all the following path for such cache missing... Regards, Michael Wang > > Haggai > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/