From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ozlabs.org (ozlabs.org [IPv6:2401:3900:2:1::2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id DDC6A1A0316 for ; Fri, 17 Oct 2014 09:43:56 +1100 (AEDT) Message-ID: <1413499435.28062.2.camel@concordia> Subject: Re: [PATCH] drivers/crypto/nx: prevent oops on module exit From: Michael Ellerman To: Dan Streetman Date: Fri, 17 Oct 2014 09:43:55 +1100 In-Reply-To: <1413487819-6119-1-git-send-email-ddstreet@ieee.org> References: <1413487819-6119-1-git-send-email-ddstreet@ieee.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Cc: linuxppc-dev@lists.ozlabs.org, Seth Jennings , Robert Jennings List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 2014-10-16 at 15:30 -0400, Dan Streetman wrote: > Check old_devdata->dev in nx-842 driver before accessing it, as > on systems without any nx-842 hardware, the ->dev will never be > set. Currently, the module will cause an oops during unload > since old_devdata->dev is null. Hi Dan, Patch looks reasonable. But I can't merge it, at least not without an ACK: $ ./scripts/get_maintainer.pl -f drivers/crypto/nx/ Nathan Fontenot (supporter:IBM Power 842 com...) Marcelo Henrique Cerri (supporter:IBM Power in-Nest...) Fionnuala Gunter (supporter:IBM Power in-Nest...) Herbert Xu (maintainer:CRYPTO API) "David S. Miller" (maintainer:CRYPTO API) linux-crypto@vger.kernel.org (open list:IBM Power in-Nest...) linux-kernel@vger.kernel.org (open list) If any of the above is out of date please send me a patch for MAINTAINERS. cheers