lkml.org 
[lkml]   [2003]   [Sep]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 2.4.23-pre3] do we want negative file descriptors?
Date
As far as I can see it does not make much sense to have negative fds
(which are in fact indexes for an array), so we can declare them unsigned.
Or not? (If yes, this one avoids some compiler warnings with gcc 3.3)

Eike

--- linux-2.4.23-pre3/include/linux/sched.h 2003-09-05 15:04:48.000000000 +0200
+++ linux-2.4.23-pre3-caliban/include/linux/sched.h 2003-09-07 11:04:34.000000000 +0200
@@ -172,9 +172,9 @@
struct files_struct {
atomic_t count;
rwlock_t file_lock; /* Protects all the below members. Nests inside tsk->alloc_lock */
- int max_fds;
- int max_fdset;
- int next_fd;
+ unsigned int max_fds;
+ unsigned int max_fdset;
+ unsigned int next_fd;
struct file ** fd; /* current fd array */
fd_set *close_on_exec;
fd_set *open_fds;
--- linux-2.4.23-pre3/fs/file.c 2003-09-05 15:04:46.000000000 +0200
+++ linux-2.4.23-pre3-caliban/fs/file.c 2003-09-07 11:01:09.000000000 +0200
@@ -57,7 +57,8 @@
int expand_fd_array(struct files_struct *files, int nr)
{
struct file **new_fds;
- int error, nfds;
+ int error;
+ unsigned int nfds;


error = -EMFILE;
@@ -166,7 +167,8 @@
int expand_fdset(struct files_struct *files, int nr)
{
fd_set *new_openset = 0, *new_execset = 0;
- int error, nfds = 0;
+ int error;
+ unsigned int nfds = 0;

error = -EMFILE;
if (files->max_fdset >= NR_OPEN || nr >= NR_OPEN)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 13:48    [W:0.025 / U:0.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site