It's pretty pointless as no one is using this structure, but even so the use of volatile is so wrong here it's sad... Cc: Hank Janssen <hjanssen@xxxxxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxx> --- drivers/staging/hv/include/HvSynicApi.h | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/include/HvSynicApi.h b/drivers/staging/hv/include/HvSynicApi.h index 7e3db63..3e6151d 100644 --- a/drivers/staging/hv/include/HvSynicApi.h +++ b/drivers/staging/hv/include/HvSynicApi.h @@ -212,7 +212,7 @@ struct hv_message { /* Define the synthetic interrupt message page layout. */ struct hv_message_page { - volatile struct hv_message SintMessage[HV_SYNIC_SINT_COUNT]; + struct hv_message SintMessage[HV_SYNIC_SINT_COUNT]; }; /* Define the synthetic interrupt controller event flags format. */ @@ -223,7 +223,7 @@ union hv_synic_event_flags { /* Define the synthetic interrupt flags page layout. */ struct hv_synic_event_flags_page { - volatile union hv_synic_event_flags SintEventFlags[HV_SYNIC_SINT_COUNT]; + union hv_synic_event_flags SintEventFlags[HV_SYNIC_SINT_COUNT]; }; /* Define SynIC control register. */ -- 1.6.4.2 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel