Hello Azhar, On 12/20/2017 08:15 PM, Shaikh, Azhar wrote: > > >> -----Original Message----- >> From: Jason Gunthorpe [mailto:j...@ziepe.ca] >> Sent: Wednesday, December 20, 2017 10:55 AM >> To: Javier Martinez Canillas <javi...@redhat.com> >> Cc: linux-kernel@vger.kernel.org; James Ettle <ja...@ettle.org.uk>; Hans de >> Goede <hdego...@redhat.com>; Shaikh, Azhar <azhar.sha...@intel.com>; >> Arnd Bergmann <a...@arndb.de>; Jarkko Sakkinen >> <jarkko.sakki...@linux.intel.com>; Peter Huewe <peterhu...@gmx.de>; >> Greg Kroah-Hartman <gre...@linuxfoundation.org>; linux- >> integr...@vger.kernel.org >> Subject: Re: [PATCH 1/4] tpm: fix access attempt to an already unmapped I/O >> memory region >> >> On Wed, Dec 20, 2017 at 07:21:31PM +0100, Javier Martinez Canillas wrote: >> >>>> The below draft fixes everything except #1. That needs a more >>>> thoughtful idea.. >>>> >>> >>> I'll just drop this patch from the series and you can fix all the >>> issues in the error / driver removal paths. It's not a dependency >>> anyways, I included it just because noticed the issue while reading the >>> code. >> >> Azhar, this means it becomes your problem :) >> > > Sure, I will fix this. > Javier you can drop this patch. >
On a second though, we should just merge $SUBJECT since is an obvious fix for one of the issues. Then you could fix the remaining bugs in error and driver removal paths. >> Since the patches Jarkko has queued from you introduce oops's you need to >> fix them.. >> >> Jason > > Regards, > Azhar Shaikh > Best regards, -- Javier Martinez Canillas Software Engineer - Desktop Hardware Enablement Red Hat
