HI Javier,
>-----Original Message----- >From: linux-integrity-ow...@vger.kernel.org [mailto:linux-integrity- >ow...@vger.kernel.org] On Behalf Of Javier Martinez Canillas >Sent: Thursday, December 21, 2017 4:49 AM >To: Shaikh, Azhar <azhar.sha...@intel.com>; Jason Gunthorpe ><j...@ziepe.ca> >Cc: linux-kernel@vger.kernel.org; James Ettle <ja...@ettle.org.uk>; Hans de >Goede <hdego...@redhat.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 > >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. I think you are asking to keep this patch as a part of this series, itself and not upload separately. Is this correct? Please correct me if I am wrong. > >>> 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 Regards, Azhar Shaikh
