| Message ID | 20210602060758.5173-1-ivan.hu@canonical.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [V2] uefirttime: set test time for the settime and setwakeuptime tests(LP: #1929509) | expand |
On 02/06/2021 07:07, Ivan Hu wrote: > Set test time for settime and setwakeuptime unsupported tests, in case > we won't get EFI_INVALID_PARAMETER returned instead. > > Signed-off-by: Ivan Hu <ivan.hu@canonical.com> > --- > src/uefi/uefirttime/uefirttime.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/src/uefi/uefirttime/uefirttime.c b/src/uefi/uefirttime/uefirttime.c > index 41503843..eb5e1683 100644 > --- a/src/uefi/uefirttime/uefirttime.c > +++ b/src/uefi/uefirttime/uefirttime.c > @@ -1272,6 +1272,12 @@ static int uefirttime_test38(fwts_framework *fw) > } else > fwts_skipped(fw, "GetTime runtime service supported, skip test."); > > + /* set test time for settime and setwakeuptime tests */ > + memset(&efi_time, 0, sizeof(efi_time)); > + efi_time.Year = 2000; > + efi_time.Month = 1; > + efi_time.Day = 1; > + > if (!(runtimeservicessupported & EFI_RT_SUPPORTED_SET_TIME)) { > settime.Time = &efi_time; > status = ~0ULL; > Thanks Ivan. Acked-by: Colin Ian King <colin.king@canonical.com>
On 2021-06-02 12:07 a.m., Ivan Hu wrote: > Set test time for settime and setwakeuptime unsupported tests, in case > we won't get EFI_INVALID_PARAMETER returned instead. > > Signed-off-by: Ivan Hu <ivan.hu@canonical.com> > --- > src/uefi/uefirttime/uefirttime.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/src/uefi/uefirttime/uefirttime.c b/src/uefi/uefirttime/uefirttime.c > index 41503843..eb5e1683 100644 > --- a/src/uefi/uefirttime/uefirttime.c > +++ b/src/uefi/uefirttime/uefirttime.c > @@ -1272,6 +1272,12 @@ static int uefirttime_test38(fwts_framework *fw) > } else > fwts_skipped(fw, "GetTime runtime service supported, skip test."); > > + /* set test time for settime and setwakeuptime tests */ > + memset(&efi_time, 0, sizeof(efi_time)); > + efi_time.Year = 2000; > + efi_time.Month = 1; > + efi_time.Day = 1; > + > if (!(runtimeservicessupported & EFI_RT_SUPPORTED_SET_TIME)) { > settime.Time = &efi_time; > status = ~0ULL; > Acked-by: Alex Hung <alex.hung@canonical.com>
diff --git a/src/uefi/uefirttime/uefirttime.c b/src/uefi/uefirttime/uefirttime.c index 41503843..eb5e1683 100644 --- a/src/uefi/uefirttime/uefirttime.c +++ b/src/uefi/uefirttime/uefirttime.c @@ -1272,6 +1272,12 @@ static int uefirttime_test38(fwts_framework *fw) } else fwts_skipped(fw, "GetTime runtime service supported, skip test."); + /* set test time for settime and setwakeuptime tests */ + memset(&efi_time, 0, sizeof(efi_time)); + efi_time.Year = 2000; + efi_time.Month = 1; + efi_time.Day = 1; + if (!(runtimeservicessupported & EFI_RT_SUPPORTED_SET_TIME)) { settime.Time = &efi_time; status = ~0ULL;
Set test time for settime and setwakeuptime unsupported tests, in case we won't get EFI_INVALID_PARAMETER returned instead. Signed-off-by: Ivan Hu <ivan.hu@canonical.com> --- src/uefi/uefirttime/uefirttime.c | 6 ++++++ 1 file changed, 6 insertions(+)