This is part [5/7] of the v9fs-2.0.2 patch against Linux 2.6.13-rc2-mm2.
This part of the patch contains the 9P protocol function changes related
to hch's comments.
Signed-off-by: Eric Van Hensbergen <[email protected]>
----------
fs/9p/9p.c | 2 +-
fs/9p/conv.c | 54 ++++++++++++++++++------------------------------------
2 files changed, 19 insertions(+), 37 deletions(-)
----------
--- a/fs/9p/9p.c
+++ b/fs/9p/9p.c
@@ -28,9 +28,9 @@
#include <linux/module.h>
#include <linux/errno.h>
#include <linux/fs.h>
+#include <linux/idr.h>
#include "debug.h"
-#include "idpool.h"
#include "v9fs.h"
#include "9p.h"
#include "mux.h"
diff --git a/fs/9p/Makefile b/fs/9p/Makefile
--- a/fs/9p/conv.c
+++ b/fs/9p/conv.c
@@ -28,9 +28,9 @@
#include <linux/module.h>
#include <linux/errno.h>
#include <linux/fs.h>
+#include <linux/idr.h>
#include "debug.h"
-#include "idpool.h"
#include "v9fs.h"
#include "9p.h"
#include "conv.h"
@@ -55,39 +55,21 @@ static inline int buf_check_overflow(str
return buf->p > buf->ep;
}
-#define buf_check_sizep(buf, len) \
- if (buf->p+len > buf->ep) { \
- if (buf->p < buf->ep) { \
- eprintk(KERN_ERR, "buffer overflow\n"); \
- buf->p = buf->ep + 1; \
- } \
- return NULL; \
- } \
-
-
-#define buf_check_size(buf, len) \
- if (buf->p+len > buf->ep) { \
- if (buf->p < buf->ep) { \
- eprintk(KERN_ERR, "buffer overflow\n"); \
- buf->p = buf->ep + 1; \
- } \
- return 0; \
- } \
-
-#define buf_check_sizev(buf, len) \
- if (buf->p+len > buf->ep) { \
- if (buf->p < buf->ep) { \
- eprintk(KERN_ERR, "buffer overflow\n"); \
- buf->p = buf->ep + 1; \
- } \
- return; \
- } \
+static inline void buf_check_size(struct cbuf *buf, int len)
+{
+ if (buf->p+len > buf->ep) {
+ if (buf->p < buf->ep) {
+ eprintk(KERN_ERR, "buffer overflow\n");
+ buf->p = buf->ep + 1;
+ }
+ }
+}
static inline void *buf_alloc(struct cbuf *buf, int len)
{
void *ret = NULL;
- buf_check_sizep(buf, len);
+ buf_check_size(buf, len);
ret = buf->p;
buf->p += len;
@@ -96,7 +78,7 @@ static inline void *buf_alloc(struct cbu
static inline void buf_put_int8(struct cbuf *buf, u8 val)
{
- buf_check_sizev(buf, 1);
+ buf_check_size(buf, 1);
buf->p[0] = val;
buf->p++;
@@ -104,7 +86,7 @@ static inline void buf_put_int8(struct c
static inline void buf_put_int16(struct cbuf *buf, u16 val)
{
- buf_check_sizev(buf, 2);
+ buf_check_size(buf, 2);
buf->p[0] = val;
buf->p[1] = val >> 8;
@@ -113,7 +95,7 @@ static inline void buf_put_int16(struct
static inline void buf_put_int32(struct cbuf *buf, u32 val)
{
- buf_check_sizev(buf, 4);
+ buf_check_size(buf, 4);
buf->p[0] = val;
buf->p[1] = val >> 8;
@@ -124,7 +106,7 @@ static inline void buf_put_int32(struct
static inline void buf_put_int64(struct cbuf *buf, u64 val)
{
- buf_check_sizev(buf, 8);
+ buf_check_size(buf, 8);
buf->p[0] = val;
buf->p[1] = val >> 8;
@@ -139,7 +121,7 @@ static inline void buf_put_int64(struct
static inline void buf_put_stringn(struct cbuf *buf, const char *s, u16 slen)
{
- buf_check_sizev(buf, slen + 2);
+ buf_check_size(buf, slen + 2);
buf_put_int16(buf, slen);
memcpy(buf->p, s, slen);
@@ -153,7 +135,7 @@ static inline void buf_put_string(struct
static inline void buf_put_data(struct cbuf *buf, void *data, u32 datalen)
{
- buf_check_sizev(buf, datalen);
+ buf_check_size(buf, datalen);
memcpy(buf->p, data, datalen);
buf->p += datalen;
@@ -257,7 +239,7 @@ static inline void *buf_get_datab(struct
char *ret = NULL;
int n = 0;
- buf_check_sizep(dbuf, datalen);
+ buf_check_size(dbuf, datalen);
n = buf_get_data(buf, dbuf->p, datalen);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
| |
 |