Patchwork acpi: dmar: Make this test no longer experimental.

login
register
mail settings
Submitter Colin King
Date Dec. 7, 2012, 12:46 a.m.
Message ID <1354841199-8436-1-git-send-email-colin.king@canonical.com>
Download mbox | patch
Permalink /patch/204364/
State Accepted
Headers show

Comments

Colin King - Dec. 7, 2012, 12:46 a.m.
From: Colin Ian King <colin.king@canonical.com>

The dmar test has been experimental for a long while and I think
it is time to mark it as a normal batch test since nobody has been
complaining that it gives false results.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 src/acpi/dmar/dmar.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Keng-Yu Lin - Dec. 12, 2012, 8:06 a.m.
On Fri, Dec 7, 2012 at 8:46 AM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> The dmar test has been experimental for a long while and I think
> it is time to mark it as a normal batch test since nobody has been
> complaining that it gives false results.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/acpi/dmar/dmar.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/acpi/dmar/dmar.c b/src/acpi/dmar/dmar.c
> index ba1b0ee..c5bf9ae 100644
> --- a/src/acpi/dmar/dmar.c
> +++ b/src/acpi/dmar/dmar.c
> @@ -336,6 +336,6 @@ static fwts_framework_ops dmar_ops = {
>         .minor_tests = dmar_tests
>  };
>
> -FWTS_REGISTER(dmar, &dmar_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH_EXPERIMENTAL);
> +FWTS_REGISTER(dmar, &dmar_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH);
>
>  #endif
> --
> 1.8.0
>
Acked-by: Keng-Yu Lin <kengyu@canonical.com>
IvanHu - Dec. 13, 2012, 9:53 a.m.
On 12/07/2012 08:46 AM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> The dmar test has been experimental for a long while and I think
> it is time to mark it as a normal batch test since nobody has been
> complaining that it gives false results.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>   src/acpi/dmar/dmar.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/acpi/dmar/dmar.c b/src/acpi/dmar/dmar.c
> index ba1b0ee..c5bf9ae 100644
> --- a/src/acpi/dmar/dmar.c
> +++ b/src/acpi/dmar/dmar.c
> @@ -336,6 +336,6 @@ static fwts_framework_ops dmar_ops = {
>   	.minor_tests = dmar_tests
>   };
>
> -FWTS_REGISTER(dmar, &dmar_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH_EXPERIMENTAL);
> +FWTS_REGISTER(dmar, &dmar_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH);
>
>   #endif
>
Acked-by: Ivan Hu <ivan.hu@canonical.com>
Alex Hung - Dec. 17, 2012, 7 a.m.
On 12/07/2012 08:46 AM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> The dmar test has been experimental for a long while and I think
> it is time to mark it as a normal batch test since nobody has been
> complaining that it gives false results.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>   src/acpi/dmar/dmar.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/acpi/dmar/dmar.c b/src/acpi/dmar/dmar.c
> index ba1b0ee..c5bf9ae 100644
> --- a/src/acpi/dmar/dmar.c
> +++ b/src/acpi/dmar/dmar.c
> @@ -336,6 +336,6 @@ static fwts_framework_ops dmar_ops = {
>   	.minor_tests = dmar_tests
>   };
>
> -FWTS_REGISTER(dmar, &dmar_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH_EXPERIMENTAL);
> +FWTS_REGISTER(dmar, &dmar_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH);
>
>   #endif
>
Acked-by: Alex Hung <alex.hung@canonical.com>

Patch

diff --git a/src/acpi/dmar/dmar.c b/src/acpi/dmar/dmar.c
index ba1b0ee..c5bf9ae 100644
--- a/src/acpi/dmar/dmar.c
+++ b/src/acpi/dmar/dmar.c
@@ -336,6 +336,6 @@  static fwts_framework_ops dmar_ops = {
 	.minor_tests = dmar_tests
 };
 
-FWTS_REGISTER(dmar, &dmar_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH_EXPERIMENTAL);
+FWTS_REGISTER(dmar, &dmar_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH);
 
 #endif