RE: [PATCH 3/3] hv: vmbus_open(): reset the channel state on ENOMEM

2015-02-01 Thread Dexuan Cui
; jasow...@redhat.com; Haiyang Zhang > Subject: RE: [PATCH 3/3] hv: vmbus_open(): reset the channel state on ENOMEM > > -Original Message- > > From: Dexuan Cui > > Sent: Thursday, January 29, 2015 9:03 PM > > To: Vitaly Kuznetsov > > Cc: gre...@linuxfoundatio

RE: [PATCH 3/3] hv: vmbus_open(): reset the channel state on ENOMEM

2015-02-01 Thread KY Srinivasan
at.com; KY Srinivasan; Haiyang Zhang > Subject: RE: [PATCH 3/3] hv: vmbus_open(): reset the channel state on > ENOMEM > > > -Original Message- > > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > > Sent: Thursday, January 29, 2015 21:22 PM > > To: Dexua

RE: [PATCH 3/3] hv: vmbus_open(): reset the channel state on ENOMEM

2015-01-29 Thread Dexuan Cui
nical.com; > jasow...@redhat.com; KY Srinivasan; Haiyang Zhang > Subject: Re: [PATCH 3/3] hv: vmbus_open(): reset the channel state on ENOMEM > > Dexuan Cui writes: > > > Without this patch, the state is put to CHANNEL_OPENING_STATE, and when > > the driver is loaded next

Re: [PATCH 3/3] hv: vmbus_open(): reset the channel state on ENOMEM

2015-01-29 Thread Vitaly Kuznetsov
Dexuan Cui writes: > Without this patch, the state is put to CHANNEL_OPENING_STATE, and when > the driver is loaded next time, vmbus_open() will fail immediately due to > newchannel->state != CHANNEL_OPEN_STATE. The patch makes sense, but I have one small doubt. We call vmbus_open from probe fun