Re: [GIT PULL] remaining tpmdd fixes for Linux 4.5

[Jarkko Sakkinen <jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>]

On Mon, Feb 22, 2016 at 12:56:53PM +1100, James Morris wrote:
> On Sat, 20 Feb 2016, Jarkko Sakkinen wrote:
> 
> > Hi James,
> > 
> > I'm sorry for the late pull request for 4.5. The reason for this was
> > the latency in my previous one. I picked with care the absolutely
> > critical fixes so that we can make a sound tpmdd release.
> > 
> > I really hope you can still pick these as one of them is absolutely
> > critical to get authorization policy sealing API right (kernel keeps
> > it finger out of user space created objects).
> 
> Pushed to next for more testing and review.
> 
> This really is getting too late in the development cycle for so many 
> fixes.  It means the code was not ready to be merged in the first place.

I fully agree what you're saying. I'll learn the lesson here and take
factors more conservative attitude from now on. No excuses. I'm sorry
about this.

Partly the reason for recent increase in regressions has been
increased real-world use of TPM2 and thus issues have started to pop
up that's a lame excuse anyway.

> Also, any idea why I'm seeing this:
> 
> drivers/char/tpm/tpm_tis.c:838: warning: ‘tpm_tis_resume’ defined but not 
> used

Bisected the patch: 00194826e6be

Do you want me to send a pull request containing a fix for the build
warning or reverting the whole commit? My call would be to apply the
fix because this commit has been tested both TPM 1.2 by Martin and
with TPM 2.0 by me and things have worked well.

I can live with either option.

I already pushed a fix to my master for this issue:

https://github.com/jsakkine/linux-tpmdd/commit/6386544ad7bceb3d0248b85da29d4d99eebe9161

> -- 
> James Morris
> <jmorris@namei.org>

I've been recently working on a custom BR environment that bundles my
latest master with initramfs user space [1]. At minimum I'll start
using this environment to create builds of this env with and without
PM for release testing and run the images both 1.2 and 2.0 HW.

This should prevent the warning you experienced never happening again.

[1]  http://git.infradead.org/users/jjs/buildroot-tpmdd.git

/Jarkko

------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
tpmdd-devel mailing list
tpmdd-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tpmdd-devel