From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1752236AbeDDUaS (ORCPT <rfc822;w@1wt.eu>);
        Wed, 4 Apr 2018 16:30:18 -0400
Received: from mx2.suse.de ([195.135.220.15]:57199 "EHLO mx2.suse.de"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1751896AbeDDUaR (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Wed, 4 Apr 2018 16:30:17 -0400
Date: Wed, 04 Apr 2018 22:30:13 +0200
Message-ID: <s5hsh8ayasq.wl-tiwai@suse.de>
From: Takashi Iwai <tiwai@suse.de>
To: Stuart Hayes <stuart.w.hayes@gmail.com>
Cc: linux-kernel@vger.kernel.org, Douglas_Warzecha@Dell.com
Subject: Re: [PATCH v3] dell_rbu: make firmware payload memory uncachable
In-Reply-To: <26829bed-39e5-c35d-8800-7116adaea603@gmail.com>
References: <26829bed-39e5-c35d-8800-7116adaea603@gmail.com>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka)
 FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/25.3
 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO)
MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, 28 Mar 2018 17:07:47 +0200,
Stuart Hayes wrote:
> 
> @@ -180,6 +181,12 @@ static int create_packet(void *data, size_t length)
>  			invalid_addr_packet_array[idx++] = packet_data_temp_buf;
>  			packet_data_temp_buf = NULL;
>  		}
> +		/*
> +		 * set to uncachable or it may never get written back before
> +		 * reboot
> +		 */
> +		set_memory_uc((unsigned long)packet_data_temp_buf,
> +			1 << ordernum);

Won't this cause Oops when the if-condition above meets?
Namely packet_data_temp_buf is set to NULL there.

Maybe better to try a fault injection to check the error handling.


thanks,

Takashi