Re: [PATCH ] drivers/base: cacheinfo: remove warning in resume

2016-09-02 Thread Sudeep Holla
On 02/09/16 13:58, Sumit Gupta wrote: Hi Sudeep, Thank you for your comments. I understand the warning we get but the patch is completely wrong. One it removes the feature of adding/removing the cache devices on cpu hotplug events. Have you tested your patch with simple cpu hotplug and seen

Re: [PATCH ] drivers/base: cacheinfo: remove warning in resume

2016-09-02 Thread Sudeep Holla
On 02/09/16 13:58, Sumit Gupta wrote: Hi Sudeep, Thank you for your comments. I understand the warning we get but the patch is completely wrong. One it removes the feature of adding/removing the cache devices on cpu hotplug events. Have you tested your patch with simple cpu hotplug and seen

RE: [PATCH ] drivers/base: cacheinfo: remove warning in resume

2016-09-02 Thread Sumit Gupta
Hi Sudeep, Thank you for your comments. > > I understand the warning we get but the patch is completely wrong. > One it removes the feature of adding/removing the cache devices on cpu > hotplug events. Have you tested your patch with simple cpu hotplug and seen no > change before and after this

RE: [PATCH ] drivers/base: cacheinfo: remove warning in resume

2016-09-02 Thread Sumit Gupta
Hi Sudeep, Thank you for your comments. > > I understand the warning we get but the patch is completely wrong. > One it removes the feature of adding/removing the cache devices on cpu > hotplug events. Have you tested your patch with simple cpu hotplug and seen no > change before and after this

Re: [PATCH ] drivers/base: cacheinfo: remove warning in resume

2016-08-30 Thread Sudeep Holla
Hi Sumit, I understand the warning we get but the patch is completely wrong. One it removes the feature of adding/removing the cache devices on cpu hotplug events. Have you tested your patch with simple cpu hotplug and seen no change before and after this change ? On 29/08/16 08:20, Sumit Gupta

Re: [PATCH ] drivers/base: cacheinfo: remove warning in resume

2016-08-30 Thread Sudeep Holla
Hi Sumit, I understand the warning we get but the patch is completely wrong. One it removes the feature of adding/removing the cache devices on cpu hotplug events. Have you tested your patch with simple cpu hotplug and seen no change before and after this change ? On 29/08/16 08:20, Sumit Gupta