On Tue, Jan 10, 2017 at 03:42:25PM -0700, Jason Gunthorpe wrote: > On Tue, Jan 10, 2017 at 05:31:45PM -0500, Ken Goldman wrote: > > On 1/10/2017 3:08 PM, Jason Gunthorpe wrote: > > >> 4 - Is a write() error desirable? I think the application would prefer > > >> a TPM formatted response like TPM_RC_VALUE. > > > > > > > > > IMHO, I prefer the write errno, but we need to clearly define what our > > > errnos means. Errnos used by RM should not overlap with errnos from > > > other parts of our kernel stack. > > > > > > This makes it clear the kernel is source of the error, not the physical > > > TPM. > > > > Except that the kernel is clearly not the source of the error. The user > > application tried to flush a handle and specified the wrong handle number. > > > > "write error" sounds like a write error, but the TPMDD didn't actually > > write anything. > > We are probably going to be going to ioctl, so it would be an ioctl > error. > > > "bad address" sounds like the kernel tried to access a bad address. But > > it didn't access any address. > > .. and we have to define what all the possible errnos mean. Defining > EBADF to mean 'RM found invalid handle in message' is probably sane.
[x] Works for me. /Jarkko ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today. http://sdm.link/xeonphi _______________________________________________ tpmdd-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/tpmdd-devel
