lkml.org 
[lkml]   [2020]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: [PATCH 02/12] xenbus: add freeze/thaw/restore callbacks support
From
Date


 CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you can confirm the sender and know the content is safe.



On 5/19/20 7:25 PM, Anchal Agarwal wrote:
>
> int xenbus_dev_resume(struct device *dev)
> {
> - int err;
> + int err = 0;


That's not necessary.
ACK.

> struct xenbus_driver *drv;
> struct xenbus_device *xdev
> = container_of(dev, struct xenbus_device, dev);
> -
> + bool xen_suspend = xen_suspend_mode_is_xen_suspend();
> DPRINTK("%s", xdev->nodename);
>
> if (dev->driver == NULL)
> @@ -627,24 +645,32 @@ int xenbus_dev_resume(struct device *dev)
> drv = to_xenbus_driver(dev->driver);
> err = talk_to_otherend(xdev);
> if (err) {
> - pr_warn("resume (talk_to_otherend) %s failed: %i\n",
> + pr_warn("%s (talk_to_otherend) %s failed: %i\n",


Please use dev_warn() everywhere, we just had a bunch of patches that
replaced pr_warn(). In fact, this is one of the lines that got changed.

ACK. Will send fixes in next series

>
> int xenbus_dev_cancel(struct device *dev)
> {
> - /* Do nothing */
> - DPRINTK("cancel");
> + int err = 0;


Again, no need to initialize.

ACK.
> + struct xenbus_driver *drv;
> + struct xenbus_device *xdev
> + = container_of(dev, struct xenbus_device, dev);


xendev please to be consistent with other code. And use to_xenbus_device().
ACK.

-boris

I will put the fixes in next round of patches.

Thanks,
Anchal


\
 
 \ /
  Last update: 2020-06-02 01:37    [W:0.091 / U:0.116 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site