[SLOF] [PATCH v4 00/33] Add vTPM support to SLOF

[Alexey Kardashevskiy]

On 24/12/2019 08:40, Stefan Berger wrote:
> On 12/18/19 6:05 PM, Alexey Kardashevskiy wrote:
>> Hey,
>>
>> It's been a while since the last attempt :) 33 is a lot! Comments
>> below...
> 
> 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.
> 
> 
> https://github.com/stefanberger/SLOF-tpm/commits/SLOF-tpm.Dec2019.23

imho you are overthinking this. This needs to be several prerequisite
patches (one per lib/directory/driver/whatever), one patch for vtpm v1
(and I still eagerly want to hear why we want v1 at all), one patch for
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,


-- 
Alexey