ACK: [SRU][Utopic][Vivid][PATCH 0/2][Hyper-V] LP:#1463584 Add support for VMBus panic notifier handler

[Brad Figg]

On Tue, Jun 16, 2015 at 02:55:28PM -0400, Joseph Salisbury wrote:
> BugLink: http://bugs.launchpad.net/bugs/1463584
> 
> == Utopic and Vivid SRU Justification ==
> These patches are specific to the Hyper-V VMBus driver to enable kernel crash notifications.
> 
> In the event of a kernel panic, VMBus driver supports sending five 64-bit values
>  to be recorded in the log. On Hyper-V this is recorded in the event log, and on
>  Azure we can use this data to better detect failures like this in the guest VM
>  and surface that as part of our diagnostics information.
> 
> Commit 5ef5b69 is a clean cherry pick.  However, commit 96c1d05 requeired a backport.  
> These patches are also needed in Trusty, but the backport for 96c1d05 was different
> becuase there are two spaces after '#include "hyperv_vmbus.h"' in Trusty, where Utopic and 
> Vivid only had once space.  I'll send a seperate SRU request for Trusty.  
> Wily will get these patches when it is rebased to v4.1 final.  
> 
> These commits will also be requested in upstream stable. 
> 
> == Fixes ==
> commit 96c1d0581d00f7abe033350edb021a9d947d8d81
> Author: Nick Meier <nmeier at microsoft.com>
> Date:   Sat Feb 28 11:39:01 2015 -0800
> 
>     Drivers: hv: vmbus: Add support for VMBus panic notifier handler
> 
> commit 5ef5b6927f14f29cacd78fa1fb861661a5367f13
> Author: Nick Meier <nmeier at microsoft.com>
> Date:   Wed Mar 18 12:29:24 2015 -0700
> 
>     Drivers: hv: vmbus: Correcting truncation error for constant HV_CRASH_CTL_CRASH_NOTIFY
> 
> 
> 
> == Test Case ==
> 
> A test kernel was built with these patches and tested by the original bug reporter.
> The bug reporter states the test kernel resolved the bug.
> 
> Nick Meier (2):
>   Drivers: hv: vmbus: Add support for VMBus panic notifier handler
>   Drivers: hv: vmbus: Correcting truncation error for constant
>     HV_CRASH_CTL_CRASH_NOTIFY
> 
>  drivers/hv/hyperv_vmbus.h | 11 +++++++++++
>  drivers/hv/vmbus_drv.c    | 35 +++++++++++++++++++++++++++++++++++
>  2 files changed, 46 insertions(+)
> 
> -- 
> 2.1.0
> 
> 
> -- 
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

Looks reasonable.

-- 
Brad Figg brad.figg at canonical.com http://www.canonical.com