From: Joe Perches Subject: xen: add __attribute__((format(printf... where appropriate Use the compiler to verify printf formats and arguments. Fix fallout. Signed-off-by: Joe Perches Make apply, slightly re-format, and fix more fallout (in blktap). Signed-off-by: Jan Beulcih --- a/drivers/xen/blkback/xenbus.c +++ b/drivers/xen/blkback/xenbus.c @@ -473,7 +473,7 @@ again: err = xenbus_switch_state(dev, XenbusStateConnected); if (err) - xenbus_dev_fatal(dev, err, "switching to Connected state", + xenbus_dev_fatal(dev, err, "%s: switching to Connected state", dev->nodename); return; --- a/drivers/xen/blktap/xenbus.c +++ b/drivers/xen/blktap/xenbus.c @@ -426,7 +426,7 @@ again: err = xenbus_switch_state(dev, XenbusStateConnected); if (err) - xenbus_dev_fatal(dev, err, "switching to Connected state", + xenbus_dev_fatal(dev, err, "%s: switching to Connected state", dev->nodename); return; --- a/include/xen/xenbus.h +++ b/include/xen/xenbus.h @@ -276,7 +276,7 @@ enum xenbus_state xenbus_read_driver_sta * formatted message. */ void xenbus_dev_error(struct xenbus_device *dev, int err, const char *fmt, - ...); + ...) __attribute__((__format__(__printf__, 3, 4))); /*** @@ -285,7 +285,7 @@ void xenbus_dev_error(struct xenbus_devi * closedown of this driver and its peer. */ void xenbus_dev_fatal(struct xenbus_device *dev, int err, const char *fmt, - ...); + ...) __attribute__((__format__(__printf__, 3, 4))); int xenbus_dev_init(void);