Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751648AbcKGFvP (ORCPT ); Mon, 7 Nov 2016 00:51:15 -0500 Received: from mail-ua0-f175.google.com ([209.85.217.175]:33195 "EHLO mail-ua0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750922AbcKGFvN (ORCPT ); Mon, 7 Nov 2016 00:51:13 -0500 MIME-Version: 1.0 In-Reply-To: References: <1477551554-30349-1-git-send-email-binoy.jayan@linaro.org> <1477551554-30349-11-git-send-email-binoy.jayan@linaro.org> From: Binoy Jayan Date: Mon, 7 Nov 2016 11:21:11 +0530 Message-ID: Subject: Re: [PATCH v4 10/10] IB/mlx5: Simplify completion into a wait_event To: Linus Torvalds Cc: Doug Ledford , Sean Hefty , Hal Rosenstock , Arnd Bergmann , "linux-rdma@vger.kernel.org" , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 344 Lines: 14 Hi Linus, On 3 November 2016 at 21:07, Linus Torvalds wrote: > This is wrong. Will change it back. > Since that "umr_context" variable is on the stack, and you are waiting > for it to be fully done, it really should be a completion. Thank you for sharing your insight with wait_event and completion. -Binoy