RE: [PATCH] hv: vmbus_free_channels(): remove the redundant free_channel()

2015-04-09 Thread Dexuan Cui
...@redhat.com; KY Srinivasan; > vkuzn...@redhat.com; dan.carpen...@oracle.com; Haiyang Zhang > Subject: Re: [PATCH] hv: vmbus_free_channels(): remove the redundant > free_channel() > > On Thu, Apr 09, 2015 at 09:15:30AM +, Dexuan Cui wrote: > > > -Original Me

Re: [PATCH] hv: vmbus_free_channels(): remove the redundant free_channel()

2015-04-09 Thread Greg KH
@linuxdriverproject.org; > > o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; KY Srinivasan; > > vkuzn...@redhat.com; dan.carpen...@oracle.com; Haiyang Zhang > > Subject: Re: [PATCH] hv: vmbus_free_channels(): remove the redundant > > free_channel() > > &g

RE: [PATCH] hv: vmbus_free_channels(): remove the redundant free_channel()

2015-04-09 Thread Dexuan Cui
rinivasan; > vkuzn...@redhat.com; dan.carpen...@oracle.com; Haiyang Zhang > Subject: Re: [PATCH] hv: vmbus_free_channels(): remove the redundant > free_channel() > > On Wed, Apr 08, 2015 at 11:13:47PM -0700, Dexuan Cui wrote: > > free_channel() has been invoked in > > vmbus

Re: [PATCH] hv: vmbus_free_channels(): remove the redundant free_channel()

2015-04-09 Thread Greg KH
On Wed, Apr 08, 2015 at 11:13:47PM -0700, Dexuan Cui wrote: > free_channel() has been invoked in > vmbus_remove() -> hv_process_channel_removal(), or vmbus_remove() -> > ... -> vmbus_close_internal() -> hv_process_channel_removal(). > > We also change to use list_for_each_entry_safe(), because the

[PATCH] hv: vmbus_free_channels(): remove the redundant free_channel()

2015-04-08 Thread Dexuan Cui
free_channel() has been invoked in vmbus_remove() -> hv_process_channel_removal(), or vmbus_remove() -> ... -> vmbus_close_internal() -> hv_process_channel_removal(). We also change to use list_for_each_entry_safe(), because the entry is removed in hv_process_channel_removal(). Thank Dan Carpente