git://ftp.safe.ca
/
safe
/
jmp
/
linux-2.6
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
vfs: get_sb_single() - do not pass options twice
[safe/jmp/linux-2.6]
/
fs
/
fifo.c
diff --git
a/fs/fifo.c
b/fs/fifo.c
index
5455916
..
f8f97b8
100644
(file)
--- a/
fs/fifo.c
+++ b/
fs/fifo.c
@@
-11,37
+11,39
@@
#include <linux/mm.h>
#include <linux/slab.h>
#include <linux/mm.h>
#include <linux/slab.h>
-#include <linux/smp_lock.h>
#include <linux/fs.h>
#include <linux/fs.h>
+#include <linux/sched.h>
#include <linux/pipe_fs_i.h>
#include <linux/pipe_fs_i.h>
-static void wait_for_partner(struct inode* inode, unsigned int
*
cnt)
+static void wait_for_partner(struct inode* inode, unsigned int
*
cnt)
{
int cur = *cnt;
{
int cur = *cnt;
- while(cur == *cnt) {
- pipe_wait(inode);
- if(signal_pending(current))
+
+ while (cur == *cnt) {
+ pipe_wait(inode->i_pipe);
+ if (signal_pending(current))
break;
}
}
static void wake_up_partner(struct inode* inode)
{
break;
}
}
static void wake_up_partner(struct inode* inode)
{
- wake_up_interruptible(
PIPE_WAIT(*inode)
);
+ wake_up_interruptible(
&inode->i_pipe->wait
);
}
static int fifo_open(struct inode *inode, struct file *filp)
{
}
static int fifo_open(struct inode *inode, struct file *filp)
{
+ struct pipe_inode_info *pipe;
int ret;
int ret;
- ret = -ERESTARTSYS;
- if (down_interruptible(PIPE_SEM(*inode)))
- goto err_nolock_nocleanup;
-
- if (!inode->i_pipe) {
+ mutex_lock(&inode->i_mutex);
+ pipe = inode->i_pipe;
+ if (!pipe) {
ret = -ENOMEM;
ret = -ENOMEM;
- if(!pipe_new(inode))
+ pipe = alloc_pipe_info(inode);
+ if (!pipe)
goto err_nocleanup;
goto err_nocleanup;
+ inode->i_pipe = pipe;
}
filp->f_version = 0;
}
filp->f_version = 0;
@@
-49,67
+51,67
@@
static int fifo_open(struct inode *inode, struct file *filp)
filp->f_mode &= (FMODE_READ | FMODE_WRITE);
switch (filp->f_mode) {
filp->f_mode &= (FMODE_READ | FMODE_WRITE);
switch (filp->f_mode) {
- case
1
:
+ case
FMODE_READ
:
/*
* O_RDONLY
* POSIX.1 says that O_NONBLOCK means return with the FIFO
* opened, even when there is no process writing the FIFO.
*/
/*
* O_RDONLY
* POSIX.1 says that O_NONBLOCK means return with the FIFO
* opened, even when there is no process writing the FIFO.
*/
- filp->f_op = &read_fifo_fops;
-
PIPE_RCOUNTER(*inode)
++;
- if (
PIPE_READERS(*inode)
++ == 0)
+ filp->f_op = &read_
pipe
fifo_fops;
+
pipe->r_counter
++;
+ if (
pipe->readers
++ == 0)
wake_up_partner(inode);
wake_up_partner(inode);
- if (!
PIPE_WRITERS(*inode)
) {
+ if (!
pipe->writers
) {
if ((filp->f_flags & O_NONBLOCK)) {
/* suppress POLLHUP until we have
* seen a writer */
if ((filp->f_flags & O_NONBLOCK)) {
/* suppress POLLHUP until we have
* seen a writer */
- filp->f_version =
PIPE_WCOUNTER(*inode)
;
+ filp->f_version =
pipe->w_counter
;
} else
{
} else
{
- wait_for_partner(inode, &
PIPE_WCOUNTER(*inode)
);
+ wait_for_partner(inode, &
pipe->w_counter
);
if(signal_pending(current))
goto err_rd;
}
}
break;
if(signal_pending(current))
goto err_rd;
}
}
break;
- case
2
:
+ case
FMODE_WRITE
:
/*
* O_WRONLY
* POSIX.1 says that O_NONBLOCK means return -1 with
* errno=ENXIO when there is no process reading the FIFO.
*/
ret = -ENXIO;
/*
* O_WRONLY
* POSIX.1 says that O_NONBLOCK means return -1 with
* errno=ENXIO when there is no process reading the FIFO.
*/
ret = -ENXIO;
- if ((filp->f_flags & O_NONBLOCK) && !
PIPE_READERS(*inode)
)
+ if ((filp->f_flags & O_NONBLOCK) && !
pipe->readers
)
goto err;
goto err;
- filp->f_op = &write_fifo_fops;
-
PIPE_WCOUNTER(*inode)
++;
- if (!
PIPE_WRITERS(*inode)
++)
+ filp->f_op = &write_
pipe
fifo_fops;
+
pipe->w_counter
++;
+ if (!
pipe->writers
++)
wake_up_partner(inode);
wake_up_partner(inode);
- if (!
PIPE_READERS(*inode)
) {
- wait_for_partner(inode, &
PIPE_RCOUNTER(*inode)
);
+ if (!
pipe->readers
) {
+ wait_for_partner(inode, &
pipe->r_counter
);
if (signal_pending(current))
goto err_wr;
}
break;
if (signal_pending(current))
goto err_wr;
}
break;
- case
3
:
+ case
FMODE_READ | FMODE_WRITE
:
/*
* O_RDWR
* POSIX.1 leaves this case "undefined" when O_NONBLOCK is set.
* This implementation will NEVER block on a O_RDWR open, since
* the process can at least talk to itself.
*/
/*
* O_RDWR
* POSIX.1 leaves this case "undefined" when O_NONBLOCK is set.
* This implementation will NEVER block on a O_RDWR open, since
* the process can at least talk to itself.
*/
- filp->f_op = &rdwr_fifo_fops;
+ filp->f_op = &rdwr_
pipe
fifo_fops;
-
PIPE_READERS(*inode)
++;
-
PIPE_WRITERS(*inode)
++;
-
PIPE_RCOUNTER(*inode)
++;
-
PIPE_WCOUNTER(*inode)
++;
- if (
PIPE_READERS(*inode) == 1 || PIPE_WRITERS(*inode)
== 1)
+
pipe->readers
++;
+
pipe->writers
++;
+
pipe->r_counter
++;
+
pipe->w_counter
++;
+ if (
pipe->readers == 1 || pipe->writers
== 1)
wake_up_partner(inode);
break;
wake_up_partner(inode);
break;
@@
-119,29
+121,27
@@
static int fifo_open(struct inode *inode, struct file *filp)
}
/* Ok! */
}
/* Ok! */
-
up(PIPE_SEM(*inode)
);
+
mutex_unlock(&inode->i_mutex
);
return 0;
err_rd:
return 0;
err_rd:
- if (!--
PIPE_READERS(*inode)
)
- wake_up_interruptible(
PIPE_WAIT(*inode)
);
+ if (!--
pipe->readers
)
+ wake_up_interruptible(
&pipe->wait
);
ret = -ERESTARTSYS;
goto err;
err_wr:
ret = -ERESTARTSYS;
goto err;
err_wr:
- if (!--
PIPE_WRITERS(*inode)
)
- wake_up_interruptible(
PIPE_WAIT(*inode)
);
+ if (!--
pipe->writers
)
+ wake_up_interruptible(
&pipe->wait
);
ret = -ERESTARTSYS;
goto err;
err:
ret = -ERESTARTSYS;
goto err;
err:
- if (!
PIPE_READERS(*inode) && !PIPE_WRITERS(*inode)
)
+ if (!
pipe->readers && !pipe->writers
)
free_pipe_info(inode);
err_nocleanup:
free_pipe_info(inode);
err_nocleanup:
- up(PIPE_SEM(*inode));
-
-err_nolock_nocleanup:
+ mutex_unlock(&inode->i_mutex);
return ret;
}
return ret;
}
@@
-150,6
+150,6
@@
err_nolock_nocleanup:
* is contain the open that then fills in the correct operations
* depending on the access mode of the file...
*/
* is contain the open that then fills in the correct operations
* depending on the access mode of the file...
*/
-struct file_operations def_fifo_fops = {
- .open = fifo_open, /* will set read
or write pipe
_fops */
+
const
struct file_operations def_fifo_fops = {
+ .open = fifo_open, /* will set read
_ or write_pipefifo
_fops */
};
};