Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Thu, 17 May 2018 10:26:05 +0200
Message-ID: <s5hsh6q7ko2.wl-tiwai@suse.de>
From:   Takashi Iwai <tiwai@suse.de>
To:     Stuart Hayes <stuart.w.hayes@gmail.com>
Cc:     Andrew Morton <akpm@linux-foundation.org>,
        linux-kernel@vger.kernel.org, charles_rose@dell.com
Subject: Re: [PATCH RESENT v4] dell_rbu: make firmware payload memory uncachable
In-Reply-To: <848a3ac6-4781-dd54-0b4a-d4839d902284@gmail.com>
References: <20180418054634.10289-1-tiwai@suse.de>
        <848a3ac6-4781-dd54-0b4a-d4839d902284@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
Precedence: bulk

On Wed, 09 May 2018 17:17:45 +0200,
Stuart Hayes wrote:
> 
> 
> On 4/18/2018 12:46 AM, Takashi Iwai wrote:
> > From: Stuart Hayes <stuart.w.hayes@gmail.com>
> > 
> > The dell_rbu driver takes firmware update payloads and puts them in memory so
> > the system BIOS can find them after a reboot.  This sometimes fails (though
> > rarely), because the memory containing the payload is in the CPU cache but
> > never gets written back to main memory before the system is rebooted (CPU
> > cache contents are lost on reboot).
> > 
> > With this patch, the payload memory will be changed to uncachable to ensure
> > that the payload is actually in main memory before the system is rebooted.
> > 
> > Signed-off-by: Stuart Hayes <stuart.w.hayes@gmail.com>
> > Reviewed-by: Takashi Iwai <tiwai@suse.de>
> > Signed-off-by: Takashi Iwai <tiwai@suse.de>
> > ---
> > v2 Added include, removed extra parentheses
> > v3 Corrected formatting and include line
> > v4 Moved set_memory_uc() outside the while loop so that the memory is
> >    definitely allocated before it is set to uncachable
> > 
> > Andrew, could you pick up this orphan one?  Thanks!
> > 
> > diff --git a/drivers/firmware/dell_rbu.c b/drivers/firmware/dell_rbu.c
> > index 2f452f1f7c8a..53f27a6e2d76 100644
> > --- a/drivers/firmware/dell_rbu.c
> > +++ b/drivers/firmware/dell_rbu.c
> > @@ -45,6 +45,7 @@
> >  #include <linux/moduleparam.h>
> >  #include <linux/firmware.h>
> >  #include <linux/dma-mapping.h>
> > +#include <asm/set_memory.h>
> >  
> >  MODULE_AUTHOR("Abhay Salunke <abhay_salunke@dell.com>");
> >  MODULE_DESCRIPTION("Driver for updating BIOS image on DELL systems");
> > @@ -181,6 +182,11 @@ static int create_packet(void *data, size_t length)
> >  			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);
> > +
> >  	spin_lock(&rbu_data.lock);
> >  
> >  	newpacket->data = packet_data_temp_buf;
> > @@ -349,6 +355,8 @@ static void packet_empty_list(void)
> >  		 * to make sure there are no stale RBU packets left in memory
> >  		 */
> >  		memset(newpacket->data, 0, rbu_data.packetsize);
> > +		set_memory_wb((unsigned long)newpacket->data,
> > +			1 << newpacket->ordernum);
> >  		free_pages((unsigned long) newpacket->data,
> >  			newpacket->ordernum);
> >  		kfree(newpacket);
> > 
> 
> Just a reminder... I don't think this has been picked up yet.  Please let me know if there's anything I could do.

Another gentle reminder ping.

Andrew, if it's not suitable to your tree, just let me know.
If so, I'll either submit directly to Linus or create a new  tree for
the misc stuff.


thanks,

Takashi