Re: [PATCH 6/6] Drivers: hv: Execute shutdown in a thread context

[gregkh@xxxxxxxxxxxxxxxxxxx]

On Sat, Feb 09, 2013 at 12:31:49AM +0000, KY Srinivasan wrote:
> 
> 
> > -----Original Message-----
> > From: gregkh@xxxxxxxxxxxxxxxxxxx [mailto:gregkh@xxxxxxxxxxxxxxxxxxx]
> > Sent: Thursday, January 24, 2013 12:18 PM
> > To: KY Srinivasan
> > Cc: Jiri Kosina; olaf@xxxxxxxxx; jasowang@xxxxxxxxxx; linux-
> > kernel@xxxxxxxxxxxxxxx; James.Bottomley@xxxxxxxxxxxxxxxxxxxxx;
> > apw@xxxxxxxxxxxxx; devel@xxxxxxxxxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx
> > Subject: Re: [PATCH 6/6] Drivers: hv: Execute shutdown in a thread context
> > 
> > On Thu, Jan 24, 2013 at 05:06:27PM +0000, KY Srinivasan wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: Jiri Kosina [mailto:jkosina@xxxxxxx]
> > > > Sent: Thursday, January 24, 2013 5:10 AM
> > > > To: KY Srinivasan
> > > > Cc: gregkh@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> > > > devel@xxxxxxxxxxxxxxxxxxxxxx; olaf@xxxxxxxxx; apw@xxxxxxxxxxxxx;
> > > > jasowang@xxxxxxxxxx; davem@xxxxxxxxxxxxx;
> > > > James.Bottomley@xxxxxxxxxxxxxxxxxxxxx
> > > > Subject: Re: [PATCH 6/6] Drivers: hv: Execute shutdown in a thread context
> > > >
> > > > On Wed, 23 Jan 2013, K. Y. Srinivasan wrote:
> > > >
> > > > > Execute the shutdown code in a thread context. With recent changes made
> > to
> > > > the
> > > > > shutdown code, shutdown code cannot be invoked from an interrupt
> > context.
> > > > >
> > > > > Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
> > > > > Reviewed-by: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx>
> > > > > ---
> > > > >  drivers/hv/hv_util.c |   12 +++++++++++-
> > > > >  1 files changed, 11 insertions(+), 1 deletions(-)
> > > > >
> > > > > diff --git a/drivers/hv/hv_util.c b/drivers/hv/hv_util.c
> > > > > index 8b7868a..aceb67f 100644
> > > > > --- a/drivers/hv/hv_util.c
> > > > > +++ b/drivers/hv/hv_util.c
> > > > > @@ -49,6 +49,16 @@ static struct hv_util_service util_kvp = {
> > > > >  	.util_deinit = hv_kvp_deinit,
> > > > >  };
> > > > >
> > > > > +static void perform_shutdown(struct work_struct *dummy)
> > > > > +{
> > > > > +		orderly_poweroff(true);
> > > > > +}
> > > >
> > > > Is there any particular reason for this kind of crazy indentation?
> > > I don't know how this extra tab crept through! Greg, if you want I can resend
> > > this patch minus the extra tab. Let me know.
> > 
> > I'll edit it by hand, but someone owes me a beer for it...  :)
> 
> I just saw that you closed the tree. Did these patches go in?

They should have, didn't you get an email saying they were in?  If not,
check my git trees to verify.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/