[SLOF] [PATCH v4 00/33] Add vTPM support to SLOF
stefanb at linux.ibm.com
Sat Dec 28 08:13:47 AEDT 2019
On 12/23/19 6:41 PM, Alexey Kardashevskiy wrote:
> On 24/12/2019 08:40, Stefan Berger wrote:
>> On 12/18/19 6:05 PM, Alexey Kardashevskiy wrote:
>>> It's been a while since the last attempt :) 33 is a lot! Comments
>> I have a series of patches now here with some of the concerns addressed.
>> I would say it's bisectable but my concern is that reshuffling more code
>> will cause issues that aren't there right now. So I would appreciate
>> some tolerance for refactoring while building up TPM 2 support.
> imho you are overthinking this. This needs to be several prerequisite
> patches (one per lib/directory/driver/whatever), one patch for vtpm v1
What are prerequisites? Are these functions needed by lib/libtpm but are
located in other directories? I have certainly not put the patches in
random order, so they do compile at every step and functions needed in
one step are introduced before it.
> (and I still eagerly want to hear why we want v1 at all), one patch for
The application stack for TSS 1.2 is (still) being packaged for ppc64le:
> vtpm v2, one patch for the menu. "One patch" could be more if patch 1
> adds some function on its own and consequent patches enhance patch 1
> (such as add vtpm with 1 algo enabled in patch 1 and then add other
> hashing algorithms later) but I do not see this happening or very
> useful. Thanks,
I tried to make it small patches for easier digestion...
More information about the SLOF