sanchayanmaity/minix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fixed some select bugs related to pipes. Removed SELFD_* because they were

Browse source 
bogus and unused.
This commit is contained in:
Philip Homburg 2006-06-07 14:41:47 +00:00
parent 373ea53510
commit cb02a90b77
3 changed files with 5 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
servers/fs/pipe.c
View file

@ -406,15 +406,14 @@ PUBLIC int select_request_pipe(struct filp *f, int *ops, int block)
orig_ops = *ops; orig_ops = *ops;
if ((*ops & (SEL_RD|SEL_ERR))) { if ((*ops & (SEL_RD|SEL_ERR))) {
if ((err = pipe_check(f->filp_ino, READING, 0, if ((err = pipe_check(f->filp_ino, READING, 0,
1, f->filp_pos, &canwrite, 1)) != SUSPEND && err > 0) 1, f->filp_pos, &canwrite, 1)) != SUSPEND)
r |= SEL_RD; r |= SEL_RD;
if (err < 0 && err != SUSPEND) if (err < 0 && err != SUSPEND)
r |= SEL_ERR; r |= SEL_ERR;
} }
if ((*ops & (SEL_WR|SEL_ERR))) { if ((*ops & (SEL_WR|SEL_ERR))) {
if ((err = pipe_check(f->filp_ino, WRITING, 0, if ((err = pipe_check(f->filp_ino, WRITING, 0,
1, f->filp_pos, &canwrite, 1)) != SUSPEND && 1, f->filp_pos, &canwrite, 1)) != SUSPEND)
err > 0 && canwrite > 0)
r |= SEL_WR; r |= SEL_WR;
if (err < 0 && err != SUSPEND) if (err < 0 && err != SUSPEND)
r |= SEL_ERR; r |= SEL_ERR;

14
servers/fs/select.c
View file

@ -38,12 +38,6 @@ PRIVATE struct selectentry {
timer_t timer; /* if expiry > 0 */ timer_t timer; /* if expiry > 0 */
} selecttab[MAXSELECTS]; } selecttab[MAXSELECTS];
#define SELFD_FILE 0
#define SELFD_PIPE 1
#define SELFD_TTY 2
#define SELFD_INET 3
#define SELFD_LOG 4
#define SEL_FDS 5
FORWARD _PROTOTYPE(int select_reevaluate, (struct filp *fp)); FORWARD _PROTOTYPE(int select_reevaluate, (struct filp *fp));
@ -70,18 +64,14 @@ PRIVATE struct fdtype {
int (*select_request)(struct filp *, int *ops, int block); int (*select_request)(struct filp *, int *ops, int block);
int (*select_match)(struct filp *); int (*select_match)(struct filp *);
int select_major; int select_major;
} fdtypes[SEL_FDS] = { } fdtypes[] = {
/* SELFD_FILE */
{ select_request_file, select_match_file, 0 }, { select_request_file, select_match_file, 0 },
/* SELFD_TTY (also PTY) */
{ select_request_general, NULL, TTY_MAJOR }, { select_request_general, NULL, TTY_MAJOR },
/* SELFD_INET */
{ select_request_general, NULL, INET_MAJOR }, { select_request_general, NULL, INET_MAJOR },
/* SELFD_PIPE (pipe(2) pipes and FS FIFOs) */
{ select_request_pipe, select_match_pipe, 0 }, { select_request_pipe, select_match_pipe, 0 },
/* SELFD_LOG (/dev/klog) */
{ select_request_general, NULL, LOG_MAJOR }, { select_request_general, NULL, LOG_MAJOR },
}; };
#define SEL_FDS (sizeof(fdtypes) / sizeof(fdtypes[0]))
/* Open Group: /* Open Group:
* "File descriptors associated with regular files shall always select true * "File descriptors associated with regular files shall always select true