* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
[not found] ` <5513E7B3.6030005@linux.vnet.ibm.com>
@ 2015-03-27 7:58 ` Xu, Quan
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8CA80B@SHSMSX101.ccr.corp.intel.com>
1 sibling, 0 replies; 10+ messages in thread
From: Xu, Quan @ 2015-03-27 7:58 UTC (permalink / raw)
To: Stefan Berger, Kevin O'Connor
Cc: wei.liu2, Daniel De Graaf, seabios, xen-devel, stefano.stabellini
> -----Original Message-----
> From: Xu, Quan
> Sent: Friday, March 27, 2015 10:01 AM
> To: 'Stefan Berger'; Kevin O'Connor
> Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com
> Subject: RE: [PATCH v10 3/6] Support for BIOS interrupt handler
>
>
>
> > -----Original Message-----
> > From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
> > Sent: Thursday, March 26, 2015 7:04 PM
> > To: Xu, Quan; Kevin O'Connor
> > Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com
> > Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
> >
> > On 03/26/2015 07:01 AM, Xu, Quan wrote:
> > >
> > >> -----Original Message-----
> > >> From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
> > >> Sent: Thursday, March 26, 2015 6:18 PM
> > >> To: Kevin O'Connor; Xu, Quan
> > >> Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com
> > >> Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
> > >>
> > >> On 03/25/2015 06:42 PM, Kevin O'Connor wrote:
> > >>> On Tue, Mar 24, 2015 at 11:10:03AM -0400, Stefan Berger wrote:
> > >>>> On 03/23/2015 08:13 PM, Kevin O'Connor wrote:
> > >>>>> Because of the mixed 16bit/32bit code in SeaBIOS, all assembler
> > >>>>> must use size suffixes - so the above should be "roll" instead of "rol".
> > >>>> Ok, fixed.
> > >>>>
> > >>>>> As before - both issues are minor and can be addressed after
> > >>>>> merge (as long as there is agreement that the sha1.c file can be
> > >>>>> licensed as LGPLv3).
> > >>>> It can have that license. I can post v11 or you can modify it,
> > >>>> either way is fine.
> > >>> Thanks. I pushed the first three patches into a test branch at:
> > >>>
> > >>> https://github.com/KevinOConnor/seabios/tree/tcg-testing
> > >>>
> > >>> I'd like to get confirmation that this works for the Xen
> > >>> requirements before merging.
> > >> I don't use Xen. I hope that Quan will provide feedback.
> > >>
> > >> Stefan
> > > Sure, I am glad to help you test it :):) Try to
> > > https://github.com/KevinOConnor/seabios/tree/tcg-testing ??
> >
> > Yes.
> >
> > Stefan
>
> Just for check,
> I can NOT git clone https://github.com/KevinOConnor/seabios/tree/tcg-testing
> I can clone https://github.com/KevinOConnor/seabios and checkout * tcg-testing
> branch.
> Correct?
>
> Quan
Share the test result first. It is not working with Xen vTPM.
The log from vtpm mini-os:
[.. ]
tpm_testing.c:229: Debug: verify plain text
tpm_testing.c:261: Info: Self-Test succeeded
tpm_startup.c:43: Info: TPM_Startup(1)
Tpmback:Info Frontend 0/0 connected
tpm_cmd_handler.c:4217: Debug: tpm_handle_command(0)
tpm_cmd_handler.c:3514: Debug: [TPM_TAG_RQU_COMMAND]
tpm_cmd_handler.c:3537: Debug: [TPM_ORD_Startup]
tpm_startup.c:43: Info: TPM_Startup(1)
tpm_cmd_handler.c:4151: Info: TPM command failed: (0x26) The command was received in the wrong sequence
relative to TPM_Init and a subsequent TPM_Startup.
[..]
In my opinion, we need this patch for Xen vTPM too.
now Xen vTPM is working with two basic functions: TPM TCPA / SSDT and registers reset
in my previous seabios patch.
Quan
>
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8CA80B@SHSMSX101.ccr.corp.intel.com>
@ 2015-03-27 13:45 ` Stefan Berger
[not found] ` <55155EE8.4020203@linux.vnet.ibm.com>
1 sibling, 0 replies; 10+ messages in thread
From: Stefan Berger @ 2015-03-27 13:45 UTC (permalink / raw)
To: Xu, Quan, Kevin O'Connor
Cc: wei.liu2, Daniel De Graaf, seabios, xen-devel, stefano.stabellini
On 03/27/2015 03:58 AM, Xu, Quan wrote:
>
>> -----Original Message-----
>> From: Xu, Quan
>> Sent: Friday, March 27, 2015 10:01 AM
>> To: 'Stefan Berger'; Kevin O'Connor
>> Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com
>> Subject: RE: [PATCH v10 3/6] Support for BIOS interrupt handler
>>
>>
>>
>>> -----Original Message-----
>>> From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
>>> Sent: Thursday, March 26, 2015 7:04 PM
>>> To: Xu, Quan; Kevin O'Connor
>>> Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com
>>> Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
>>>
>>> On 03/26/2015 07:01 AM, Xu, Quan wrote:
>>>>> -----Original Message-----
>>>>> From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
>>>>> Sent: Thursday, March 26, 2015 6:18 PM
>>>>> To: Kevin O'Connor; Xu, Quan
>>>>> Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com
>>>>> Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
>>>>>
>>>>> On 03/25/2015 06:42 PM, Kevin O'Connor wrote:
>>>>>> On Tue, Mar 24, 2015 at 11:10:03AM -0400, Stefan Berger wrote:
>>>>>>> On 03/23/2015 08:13 PM, Kevin O'Connor wrote:
>>>>>>>> Because of the mixed 16bit/32bit code in SeaBIOS, all assembler
>>>>>>>> must use size suffixes - so the above should be "roll" instead of "rol".
>>>>>>> Ok, fixed.
>>>>>>>
>>>>>>>> As before - both issues are minor and can be addressed after
>>>>>>>> merge (as long as there is agreement that the sha1.c file can be
>>>>>>>> licensed as LGPLv3).
>>>>>>> It can have that license. I can post v11 or you can modify it,
>>>>>>> either way is fine.
>>>>>> Thanks. I pushed the first three patches into a test branch at:
>>>>>>
>>>>>> https://github.com/KevinOConnor/seabios/tree/tcg-testing
>>>>>>
>>>>>> I'd like to get confirmation that this works for the Xen
>>>>>> requirements before merging.
>>>>> I don't use Xen. I hope that Quan will provide feedback.
>>>>>
>>>>> Stefan
>>>> Sure, I am glad to help you test it :):) Try to
>>>> https://github.com/KevinOConnor/seabios/tree/tcg-testing ??
>>> Yes.
>>>
>>> Stefan
>> Just for check,
>> I can NOT git clone https://github.com/KevinOConnor/seabios/tree/tcg-testing
>> I can clone https://github.com/KevinOConnor/seabios and checkout * tcg-testing
>> branch.
>> Correct?
>>
>> Quan
> Share the test result first. It is not working with Xen vTPM.
> The log from vtpm mini-os:
>
> [.. ]
> tpm_testing.c:229: Debug: verify plain text
> tpm_testing.c:261: Info: Self-Test succeeded
> tpm_startup.c:43: Info: TPM_Startup(1)
> Tpmback:Info Frontend 0/0 connected
> tpm_cmd_handler.c:4217: Debug: tpm_handle_command(0)
> tpm_cmd_handler.c:3514: Debug: [TPM_TAG_RQU_COMMAND]
> tpm_cmd_handler.c:3537: Debug: [TPM_ORD_Startup]
> tpm_startup.c:43: Info: TPM_Startup(1)
> tpm_cmd_handler.c:4151: Info: TPM command failed: (0x26) The command was received in the wrong sequence
> relative to TPM_Init and a subsequent TPM_Startup.
> [..]
>
>
> In my opinion, we need this patch for Xen vTPM too.
Which patch is 'this patch'?
Does the above indicate that TPM_Startup(1) is sent twice to the Xen
vTPM? Besides SeaBIOS, what else is sending a TPM_Startup()? If there is
something else in Xen that sends a TPM_Startup() to the TPM, before
SeaBIOS does, then the following code may apply as well:
This part is from src/tcgbios.c.
+ rc = build_and_send_cmd(TPM_ORD_Startup,
+ Startup_ST_CLEAR, sizeof(Startup_ST_CLEAR),
+ NULL, 10, &returnCode,
TPM_DURATION_TYPE_SHORT);
+
+ dprintf(DEBUG_tcg, "Return code from TPM_Startup = 0x%08x\n",
+ returnCode);
+
+ if (CONFIG_COREBOOT) {
Here you could add a check whether you are running on Xen and ignore the
error returned from TPM_Startup().
+ /* with other firmware on the system the TPM may already have been
+ * initialized
+ */
+ if (returnCode == TPM_INVALID_POSTINIT)
+ returnCode = 0;
+ }
+
+ if (rc || returnCode)
+ goto err_exit;
Regards,
Stefan
> now Xen vTPM is working with two basic functions: TPM TCPA / SSDT and registers reset
> in my previous seabios patch.
>
>
> Quan
>
>
>>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
[not found] ` <55155EE8.4020203@linux.vnet.ibm.com>
@ 2015-03-30 5:09 ` Xu, Quan
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8D6B72@SHSMSX101.ccr.corp.intel.com>
1 sibling, 0 replies; 10+ messages in thread
From: Xu, Quan @ 2015-03-30 5:09 UTC (permalink / raw)
To: Stefan Berger, Kevin O'Connor
Cc: wei.liu2, Daniel De Graaf, seabios, xen-devel, stefano.stabellini
> -----Original Message-----
> From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
> Sent: Friday, March 27, 2015 9:45 PM
> To: Xu, Quan; Kevin O'Connor
> Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com;
> xen-devel@lists.xen.org; Daniel De Graaf; wei.liu2@citrix.com
> Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
>
> On 03/27/2015 03:58 AM, Xu, Quan wrote:
> >
> >> -----Original Message-----
> >> From: Xu, Quan
> >> Sent: Friday, March 27, 2015 10:01 AM
> >> To: 'Stefan Berger'; Kevin O'Connor
> >> Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com
> >> Subject: RE: [PATCH v10 3/6] Support for BIOS interrupt handler
> >>
> >>
> >>
> >>> -----Original Message-----
> >>> From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
> >>> Sent: Thursday, March 26, 2015 7:04 PM
> >>> To: Xu, Quan; Kevin O'Connor
> >>> Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com
> >>> Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
> >>>
> >>> On 03/26/2015 07:01 AM, Xu, Quan wrote:
> >>>>> -----Original Message-----
> >>>>> From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
> >>>>> Sent: Thursday, March 26, 2015 6:18 PM
> >>>>> To: Kevin O'Connor; Xu, Quan
> >>>>> Cc: seabios@seabios.org; stefano.stabellini@eu.citrix.com
> >>>>> Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
> >>>>>
> >>>>> On 03/25/2015 06:42 PM, Kevin O'Connor wrote:
> >>>>>> On Tue, Mar 24, 2015 at 11:10:03AM -0400, Stefan Berger wrote:
> >>>>>>> On 03/23/2015 08:13 PM, Kevin O'Connor wrote:
> >>>>>>>> Because of the mixed 16bit/32bit code in SeaBIOS, all assembler
> >>>>>>>> must use size suffixes - so the above should be "roll" instead of "rol".
> >>>>>>> Ok, fixed.
> >>>>>>>
> >>>>>>>> As before - both issues are minor and can be addressed after
> >>>>>>>> merge (as long as there is agreement that the sha1.c file can
> >>>>>>>> be licensed as LGPLv3).
> >>>>>>> It can have that license. I can post v11 or you can modify it,
> >>>>>>> either way is fine.
> >>>>>> Thanks. I pushed the first three patches into a test branch at:
> >>>>>>
> >>>>>> https://github.com/KevinOConnor/seabios/tree/tcg-testing
> >>>>>>
> >>>>>> I'd like to get confirmation that this works for the Xen
> >>>>>> requirements before merging.
> >>>>> I don't use Xen. I hope that Quan will provide feedback.
> >>>>>
> >>>>> Stefan
> >>>> Sure, I am glad to help you test it :):) Try to
> >>>> https://github.com/KevinOConnor/seabios/tree/tcg-testing ??
> >>> Yes.
> >>>
> >>> Stefan
> >> Just for check,
> >> I can NOT git clone
> >> https://github.com/KevinOConnor/seabios/tree/tcg-testing
> >> I can clone https://github.com/KevinOConnor/seabios and checkout *
> >> tcg-testing branch.
> >> Correct?
> >>
> >> Quan
> > Share the test result first. It is not working with Xen vTPM.
> > The log from vtpm mini-os:
> >
> > [.. ]
> > tpm_testing.c:229: Debug: verify plain text
> > tpm_testing.c:261: Info: Self-Test succeeded
> > tpm_startup.c:43: Info: TPM_Startup(1) Tpmback:Info Frontend 0/0
> > connected
> > tpm_cmd_handler.c:4217: Debug: tpm_handle_command(0)
> > tpm_cmd_handler.c:3514: Debug: [TPM_TAG_RQU_COMMAND]
> > tpm_cmd_handler.c:3537: Debug: [TPM_ORD_Startup]
> > tpm_startup.c:43: Info: TPM_Startup(1)
> > tpm_cmd_handler.c:4151: Info: TPM command failed: (0x26) The command
> > was received in the wrong sequence relative to TPM_Init and a subsequent
> TPM_Startup.
> > [..]
> >
> >
> > In my opinion, we need this patch for Xen vTPM too.
>
> Which patch is 'this patch'?
This patch is '[PATCH v10 3/6] Support for BIOS interrupt handler'.
Is interrupt for tGrub? Correct me if I am wrong..
> Does the above indicate that TPM_Startup(1) is sent twice to the Xen vTPM?
> Besides SeaBIOS, what else is sending a TPM_Startup()? If there is something
> else in Xen that sends a TPM_Startup() to the TPM, before SeaBIOS does, then
> the following code may apply as well:
>
>
> This part is from src/tcgbios.c.
>
> + rc = build_and_send_cmd(TPM_ORD_Startup,
> + Startup_ST_CLEAR, sizeof(Startup_ST_CLEAR),
> + NULL, 10, &returnCode,
> TPM_DURATION_TYPE_SHORT);
> +
> + dprintf(DEBUG_tcg, "Return code from TPM_Startup = 0x%08x\n",
> + returnCode);
> +
> + if (CONFIG_COREBOOT) {
>
> Here you could add a check whether you are running on Xen and ignore the error
> returned from TPM_Startup().
>
> + /* with other firmware on the system the TPM may already have been
> + * initialized
> + */
> + if (returnCode == TPM_INVALID_POSTINIT)
> + returnCode = 0;
> + }
> +
> + if (rc || returnCode)
> + goto err_exit;
>
>
>
> Regards,
> Stefan
>
I will go through all of these seabios patch, and try to make it compatible for Xen vTPM.
-Quan
> > now Xen vTPM is working with two basic functions: TPM TCPA / SSDT and
> > registers reset in my previous seabios patch.
> >
> >
> > Quan
> >
> >
> >>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8D6B72@SHSMSX101.ccr.corp.intel.com>
@ 2015-04-21 15:22 ` Kevin O'Connor
[not found] ` <20150421152233.GA13648@morn.localdomain>
1 sibling, 0 replies; 10+ messages in thread
From: Kevin O'Connor @ 2015-04-21 15:22 UTC (permalink / raw)
To: Xu, Quan
Cc: wei.liu2, stefano.stabellini, seabios, xen-devel, Stefan Berger,
Daniel De Graaf
On Mon, Mar 30, 2015 at 05:09:47AM +0000, Xu, Quan wrote:
> > From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
> > On 03/27/2015 03:58 AM, Xu, Quan wrote:
> > >> From: Xu, Quan
> > >>> From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
> > >>> On 03/26/2015 07:01 AM, Xu, Quan wrote:
> > >>>>> From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
> > >>>>> On 03/25/2015 06:42 PM, Kevin O'Connor wrote:
> > >>>>>> On Tue, Mar 24, 2015 at 11:10:03AM -0400, Stefan Berger wrote:
> > >>>>>>> On 03/23/2015 08:13 PM, Kevin O'Connor wrote:
> > >>>>>>>> Because of the mixed 16bit/32bit code in SeaBIOS, all assembler
> > >>>>>>>> must use size suffixes - so the above should be "roll" instead of "rol".
> > >>>>>>> Ok, fixed.
> > >>>>>>>
> > >>>>>>>> As before - both issues are minor and can be addressed after
> > >>>>>>>> merge (as long as there is agreement that the sha1.c file can
> > >>>>>>>> be licensed as LGPLv3).
> > >>>>>>> It can have that license. I can post v11 or you can modify it,
> > >>>>>>> either way is fine.
> > >>>>>> Thanks. I pushed the first three patches into a test branch at:
> > >>>>>>
> > >>>>>> https://github.com/KevinOConnor/seabios/tree/tcg-testing
> > >>>>>>
> > >>>>>> I'd like to get confirmation that this works for the Xen
> > >>>>>> requirements before merging.
> > >>>>> I don't use Xen. I hope that Quan will provide feedback.
> > >>>>>
> > >>>>> Stefan
> > >>>> Sure, I am glad to help you test it :):) Try to
> > >>>> https://github.com/KevinOConnor/seabios/tree/tcg-testing ??
> > >>> Yes.
[...]
> I will go through all of these seabios patch, and try to make it compatible for Xen vTPM.
>
What's the status of this? Is it safe to push forward parts of
Stefan's patches, or does that represent a regression for Xen?
-Kevin
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
[not found] ` <20150421152233.GA13648@morn.localdomain>
@ 2015-04-29 11:18 ` Stefan Berger
2015-04-30 13:21 ` Xu, Quan
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8F394A@SHSMSX101.ccr.corp.intel.com>
0 siblings, 2 replies; 10+ messages in thread
From: Stefan Berger @ 2015-04-29 11:18 UTC (permalink / raw)
To: Xu, Quan
Cc: wei.liu2, stefano.stabellini, seabios, xen-devel,
Kevin O'Connor, Daniel De Graaf
On 04/21/2015 11:22 AM, Kevin O'Connor wrote:
> On Mon, Mar 30, 2015 at 05:09:47AM +0000, Xu, Quan wrote:
>>>
> [...]
>> I will go through all of these seabios patch, and try to make it compatible for Xen vTPM.
>>
> What's the status of this? Is it safe to push forward parts of
> Stefan's patches, or does that represent a regression for Xen?
Xu, any news? I would like to see these patches making it into the
repository.
Regards,
Stefan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
2015-04-29 11:18 ` Stefan Berger
@ 2015-04-30 13:21 ` Xu, Quan
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8F394A@SHSMSX101.ccr.corp.intel.com>
1 sibling, 0 replies; 10+ messages in thread
From: Xu, Quan @ 2015-04-30 13:21 UTC (permalink / raw)
To: Stefan Berger
Cc: wei.liu2, stefano.stabellini, seabios, xen-devel,
Kevin O'Connor, Daniel De Graaf
> -----Original Message-----
> From: Stefan Berger [mailto:stefanb@linux.vnet.ibm.com]
> Sent: Wednesday, April 29, 2015 7:18 PM
> To: Xu, Quan
> Cc: Kevin O'Connor; seabios@seabios.org; stefano.stabellini@eu.citrix.com;
> xen-devel@lists.xen.org; Daniel De Graaf; wei.liu2@citrix.com
> Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
>
> On 04/21/2015 11:22 AM, Kevin O'Connor wrote:
> > On Mon, Mar 30, 2015 at 05:09:47AM +0000, Xu, Quan wrote:
> >>>
> > [...]
> >> I will go through all of these seabios patch, and try to make it compatible for
> Xen vTPM.
> >>
> > What's the status of this? Is it safe to push forward parts of
> > Stefan's patches, or does that represent a regression for Xen?
>
> Xu, any news? I would like to see these patches making it into the repository.
>
> Regards,
> Stefan
Stefan,
Sorry to reply so late to you. I try to make it compatible with Xen vTPM by below patch
Against your serious of patch(https://github.com/KevinOConnor/seabios/tree/tcg-testing ).
Also I have tested this patch, which is working.
### >>> patch
commit 2c556064695b0a979adb4039f13db1b29a19d3cf
Author: Quan Xu <quan.xu@intel.com>
Date: Thu Apr 30 19:43:04 2015 -0400
make SeaBios compatible with Xen vTPM.
Signed-off-by: Quan Xu <quan.xu@intel.com>
diff --git a/src/tcgbios.c b/src/tcgbios.c
index c837ab6..4cda800 100644
--- a/src/tcgbios.c
+++ b/src/tcgbios.c
@@ -23,7 +23,7 @@
#include "std/acpi.h" // RSDP_SIGNATURE, rsdt_descriptor
#include "bregs.h" // struct bregs
#include "sha1.h" // sha1
-
+#include "fw/paravirt.h" // runningOnXen
static const u8 Startup_ST_CLEAR[2] = { 0x00, TPM_ST_CLEAR };
static const u8 Startup_ST_STATE[2] = { 0x00, TPM_ST_STATE };
@@ -494,6 +494,8 @@ tpm_start(void)
return 0;
tpm_acpi_init();
+ if (runningOnXen())
+ return 0;
return tpm_startup();
}
### <<< patch
Thanks
Quan Xu
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8F394A@SHSMSX101.ccr.corp.intel.com>
@ 2015-04-30 15:34 ` Stefan Berger
[not found] ` <55424B7C.1090406@linux.vnet.ibm.com>
1 sibling, 0 replies; 10+ messages in thread
From: Stefan Berger @ 2015-04-30 15:34 UTC (permalink / raw)
To: Xu, Quan, Kevin O'Connor
Cc: wei.liu2, Daniel De Graaf, seabios, xen-devel, stefano.stabellini
On 04/30/2015 09:21 AM, Xu, Quan wrote:
>
> Stefan,
> Sorry to reply so late to you. I try to make it compatible with Xen vTPM by below patch
> Against your serious of patch(https://github.com/KevinOConnor/seabios/tree/tcg-testing ).
> Also I have tested this patch, which is working.
Thanks.
>
>
> ### >>> patch
> commit 2c556064695b0a979adb4039f13db1b29a19d3cf
> Author: Quan Xu <quan.xu@intel.com>
> Date: Thu Apr 30 19:43:04 2015 -0400
>
> make SeaBios compatible with Xen vTPM.
>
> Signed-off-by: Quan Xu <quan.xu@intel.com>
>
> diff --git a/src/tcgbios.c b/src/tcgbios.c
> index c837ab6..4cda800 100644
> --- a/src/tcgbios.c
> +++ b/src/tcgbios.c
> @@ -23,7 +23,7 @@
> #include "std/acpi.h" // RSDP_SIGNATURE, rsdt_descriptor
> #include "bregs.h" // struct bregs
> #include "sha1.h" // sha1
> -
> +#include "fw/paravirt.h" // runningOnXen
>
> static const u8 Startup_ST_CLEAR[2] = { 0x00, TPM_ST_CLEAR };
> static const u8 Startup_ST_STATE[2] = { 0x00, TPM_ST_STATE };
> @@ -494,6 +494,8 @@ tpm_start(void)
> return 0;
>
> tpm_acpi_init();
> + if (runningOnXen())
> + return 0;
>
> return tpm_startup();
> }
>
> ### <<< patch
>
Kevin, I think this should be good to add after 3/6 ?
Stefan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
[not found] ` <55424B7C.1090406@linux.vnet.ibm.com>
@ 2015-05-21 17:20 ` Kevin O'Connor
[not found] ` <20150521172025.GE13894@morn.localdomain>
1 sibling, 0 replies; 10+ messages in thread
From: Kevin O'Connor @ 2015-05-21 17:20 UTC (permalink / raw)
To: Stefan Berger
Cc: wei.liu2, stefano.stabellini, seabios, xen-devel, Xu, Quan,
Daniel De Graaf
On Thu, Apr 30, 2015 at 11:34:20AM -0400, Stefan Berger wrote:
> On 04/30/2015 09:21 AM, Xu, Quan wrote:
> >
> >Stefan,
> > Sorry to reply so late to you. I try to make it compatible with Xen vTPM by below patch
> >Against your serious of patch(https://github.com/KevinOConnor/seabios/tree/tcg-testing ).
> >Also I have tested this patch, which is working.
[...]
>
> Kevin, I think this should be good to add after 3/6 ?
Thanks - I applied the first three tpm patches along with this patch.
-Kevin
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
[not found] ` <20150521172025.GE13894@morn.localdomain>
@ 2015-05-22 1:18 ` Xu, Quan
2015-05-22 14:27 ` Stefan Berger
0 siblings, 1 reply; 10+ messages in thread
From: Xu, Quan @ 2015-05-22 1:18 UTC (permalink / raw)
To: Kevin O'Connor, Stefan Berger
Cc: wei.liu2, Daniel De Graaf, seabios, xen-devel, stefano.stabellini
> -----Original Message-----
> From: Kevin O'Connor [mailto:kevin@koconnor.net]
> Sent: Friday, May 22, 2015 1:20 AM
> To: Stefan Berger
> Cc: Xu, Quan; seabios@seabios.org; stefano.stabellini@eu.citrix.com;
> xen-devel@lists.xen.org; Daniel De Graaf; wei.liu2@citrix.com
> Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
>
> On Thu, Apr 30, 2015 at 11:34:20AM -0400, Stefan Berger wrote:
> > On 04/30/2015 09:21 AM, Xu, Quan wrote:
> > >
> > >Stefan,
> > > Sorry to reply so late to you. I try to make it compatible with Xen
> > >vTPM by below patch Against your serious of
> patch(https://github.com/KevinOConnor/seabios/tree/tcg-testing ).
> > >Also I have tested this patch, which is working.
> [...]
> >
> > Kevin, I think this should be good to add after 3/6 ?
>
> Thanks - I applied the first three tpm patches along with this patch.
>
> -Kevin
Thanks :))
Quan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v10 3/6] Support for BIOS interrupt handler
2015-05-22 1:18 ` Xu, Quan
@ 2015-05-22 14:27 ` Stefan Berger
0 siblings, 0 replies; 10+ messages in thread
From: Stefan Berger @ 2015-05-22 14:27 UTC (permalink / raw)
To: Xu, Quan, Kevin O'Connor
Cc: wei.liu2, Daniel De Graaf, seabios, xen-devel, stefano.stabellini
On 05/21/2015 09:18 PM, Xu, Quan wrote:
>
>> -----Original Message-----
>> From: Kevin O'Connor [mailto:kevin@koconnor.net]
>> Sent: Friday, May 22, 2015 1:20 AM
>> To: Stefan Berger
>> Cc: Xu, Quan; seabios@seabios.org; stefano.stabellini@eu.citrix.com;
>> xen-devel@lists.xen.org; Daniel De Graaf; wei.liu2@citrix.com
>> Subject: Re: [PATCH v10 3/6] Support for BIOS interrupt handler
>>
>> On Thu, Apr 30, 2015 at 11:34:20AM -0400, Stefan Berger wrote:
>>> On 04/30/2015 09:21 AM, Xu, Quan wrote:
>>>> Stefan,
>>>> Sorry to reply so late to you. I try to make it compatible with Xen
>>>> vTPM by below patch Against your serious of
>> patch(https://github.com/KevinOConnor/seabios/tree/tcg-testing ).
>>>> Also I have tested this patch, which is working.
>> [...]
>>> Kevin, I think this should be good to add after 3/6 ?
>> Thanks - I applied the first three tpm patches along with this patch.
>>
>> -Kevin
> Thanks :))
Yes, thanks!
Stefan
>
>
> Quan
>
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2015-05-22 14:27 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1427134940-402324-1-git-send-email-stefanb@linux.vnet.ibm.com>
[not found] ` <1427134940-402324-4-git-send-email-stefanb@linux.vnet.ibm.com>
[not found] ` <20150324001330.GA9637@morn.localdomain>
[not found] ` <55117E4B.4080001@linux.vnet.ibm.com>
[not found] ` <20150325224209.GA27838@morn.localdomain>
[not found] ` <5513DCED.40304@linux.vnet.ibm.com>
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8C9AB6@SHSMSX101.ccr.corp.intel.com>
[not found] ` <5513E7B3.6030005@linux.vnet.ibm.com>
2015-03-27 7:58 ` [PATCH v10 3/6] Support for BIOS interrupt handler Xu, Quan
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8CA80B@SHSMSX101.ccr.corp.intel.com>
2015-03-27 13:45 ` Stefan Berger
[not found] ` <55155EE8.4020203@linux.vnet.ibm.com>
2015-03-30 5:09 ` Xu, Quan
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8D6B72@SHSMSX101.ccr.corp.intel.com>
2015-04-21 15:22 ` Kevin O'Connor
[not found] ` <20150421152233.GA13648@morn.localdomain>
2015-04-29 11:18 ` Stefan Berger
2015-04-30 13:21 ` Xu, Quan
[not found] ` <945CA011AD5F084CBEA3E851C0AB28890E8F394A@SHSMSX101.ccr.corp.intel.com>
2015-04-30 15:34 ` Stefan Berger
[not found] ` <55424B7C.1090406@linux.vnet.ibm.com>
2015-05-21 17:20 ` Kevin O'Connor
[not found] ` <20150521172025.GE13894@morn.localdomain>
2015-05-22 1:18 ` Xu, Quan
2015-05-22 14:27 ` Stefan Berger
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.