From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Steve Wise" Subject: RE: [PATCH 1/2] cm: Don't allocate ib_cm workqueue with WQ_MEM_RECLAIM Date: Tue, 15 Aug 2017 12:30:12 -0500 Message-ID: <009901d315ec$263eee80$72bccb80$@opengridcomputing.com> References: <1502813574-6040-1-git-send-email-sagi@grimberg.me> <1502813574-6040-2-git-send-email-sagi@grimberg.me> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1502813574-6040-2-git-send-email-sagi-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org> Content-Language: en-us Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: 'Sagi Grimberg' , 'Doug Ledford' , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org > create_workqueue always creates the worqeueu with WQ_MEM_RECLAIM Typo "worqeueu" -> "workqueue" > and silences a flush dependancy warn for WQ_LEGACY. Instead, we > want to keep the warn in case the allocator tries to flush the > cm workqueue because its very likely that cm work execution will > yield memory allocations (for example cm connection requests) . > > Reported-by: Steve Wise > Signed-off-by: Sagi Grimberg The change looks good. Reviewed-by: Steve Wise -- 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