From c4ca2da6ca875c0142a4e7ffd95e0aa0c573590b Mon Sep 17 00:00:00 2001 From: "Frank Ch. Eigler" Date: Tue, 13 Oct 2009 17:33:29 -0400 Subject: runtime shutdown errors: simplify, improve error specificity * transport/control.c (*_cmd): Return -Ecodes rather than "-1" from file_operations callbacks. * staprun/ctl.c (init_ctl_channel): Return distinct error codes. * staprun/staprun.c (remove_module): Skip connection attempt to .ctl file; just do delete_module() with O_NONBLOCK. --- runtime/transport/control.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'runtime/transport/control.c') diff --git a/runtime/transport/control.c b/runtime/transport/control.c index 0e18bb8b..8712a8ad 100644 --- a/runtime/transport/control.c +++ b/runtime/transport/control.c @@ -59,7 +59,7 @@ static ssize_t _stp_ctl_write_cmd(struct file *file, const char __user *buf, siz #ifdef STP_BULKMODE return count; #else - return -1; + return -EINVAL; #endif case STP_RELOCATION: _stp_do_relocation (buf, count); @@ -150,7 +150,7 @@ static int _stp_ctl_write(int type, void *data, unsigned len) /* get a buffer from the free pool */ bptr = _stp_mempool_alloc(_stp_pool_q); if (unlikely(bptr == NULL)) - return -1; + return -ENOMEM; bptr->type = type; memcpy(bptr->buf, data, len); @@ -225,7 +225,7 @@ static ssize_t _stp_ctl_read_cmd(struct file *file, char __user *buf, static int _stp_ctl_open_cmd(struct inode *inode, struct file *file) { if (_stp_ctl_attached) - return -1; + return -EBUSY; _stp_attach(); return 0; } -- cgit