[SLOF] [PATCH v4 00/33] Add vTPM support to SLOF
aik at ozlabs.ru
Sat Dec 28 15:11:25 AEDT 2019
On 28/12/2019 08:13, Stefan Berger wrote:
> 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:
Ok. But what about v2.0? "trousers" does not support it?
>> 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...
Not seeing how helpers are actually used does not help much and bisectability suffers as well. Thanks,
More information about the SLOF