Re: sys_sendfile64 not in Linux 2.4.21-pre4

From: Christoph Hellwig (hch@infradead.org)
Date: Thu Jan 30 2003 - 06:16:34 EST


On Thu, Jan 30, 2003 at 12:02:01PM +0100, Marc-Christian Petersen wrote:
> > > > Why isn't sendfile64 included in 2.4.21-pre4? glibc 2.3 already
> > > > expects it, so programs with 64-bit off_t will not be able to use
> > > > sendfile otherwise. And the patch is IIUC very small . . .
> > >
> > > I sent patches to Marcelo a few times, but they got silently ignored..
> >
> > Can you forward me a copy ?
> to me too please :)

--- linux-2.4.18/arch/i386/kernel/entry.S~ Sun Jun 9 13:39:14 2002
+++ linux-2.4.18/arch/i386/kernel/entry.S Sun Jun 9 13:44:45 2002
@@ -645,7 +645,7 @@
         .long SYMBOL_NAME(sys_ni_syscall) /* reserved for lremovexattr */
         .long SYMBOL_NAME(sys_ni_syscall) /* reserved for fremovexattr */
          .long SYMBOL_NAME(sys_tkill)
- .long SYMBOL_NAME(sys_ni_syscall) /* reserved for sendfile64 */
+ .long SYMBOL_NAME(sys_sendfile64)
         .long SYMBOL_NAME(sys_ni_syscall) /* 240 reserved for futex */
         .long SYMBOL_NAME(sys_ni_syscall) /* reserved for sched_setaffinity */
         .long SYMBOL_NAME(sys_ni_syscall) /* reserved for sched_getaffinity */
--- linux-2.4.18/mm/filemap.c~ Sun Jun 9 13:39:21 2002
+++ linux-2.4.18/mm/filemap.c Sun Jun 9 13:48:38 2002
@@ -1889,7 +1889,7 @@
         return written;
 }
 
-asmlinkage ssize_t sys_sendfile(int out_fd, int in_fd, off_t *offset, size_t count)
+static ssize_t common_sendfile(int out_fd, int in_fd, loff_t *offset, size_t count)
 {
         ssize_t retval;
         struct file * in_file, * out_file;
@@ -1934,27 +1934,19 @@
         retval = 0;
         if (count) {
                 read_descriptor_t desc;
- loff_t pos = 0, *ppos;
-
- retval = -EFAULT;
- ppos = &in_file->f_pos;
- if (offset) {
- if (get_user(pos, offset))
- goto fput_out;
- ppos = &pos;
- }
+
+ if (!offset)
+ offset = &in_file->f_pos;
 
                 desc.written = 0;
                 desc.count = count;
                 desc.buf = (char *) out_file;
                 desc.error = 0;
- do_generic_file_read(in_file, ppos, &desc, file_send_actor);
+ do_generic_file_read(in_file, offset, &desc, file_send_actor);
 
                 retval = desc.written;
                 if (!retval)
                         retval = desc.error;
- if (offset)
- put_user(pos, offset);
         }
 
 fput_out:
@@ -1965,6 +1957,42 @@
         return retval;
 }
 
+asmlinkage ssize_t sys_sendfile(int out_fd, int in_fd, off_t *offset, size_t count)
+{
+ loff_t pos, *ppos = NULL;
+ ssize_t ret;
+
+ if (offset) {
+ off_t off;
+ if (unlikely(get_user(off, offset)))
+ return -EFAULT;
+ pos = off;
+ ppos = &pos;
+ }
+
+ ret = common_sendfile(out_fd, in_fd, ppos, count);
+ if (offset)
+ put_user((off_t)pos, offset);
+ return ret;
+}
+
+asmlinkage ssize_t sys_sendfile64(int out_fd, int in_fd, loff_t *offset, size_t count)
+{
+ loff_t pos, *ppos = NULL;
+ ssize_t ret;
+
+ if (offset) {
+ if (unlikely(copy_from_user(&pos, offset, sizeof(loff_t))))
+ return -EFAULT;
+ ppos = &pos;
+ }
+
+ ret = common_sendfile(out_fd, in_fd, ppos, count);
+ if (offset)
+ put_user(pos, offset);
+ return ret;
+}
+
 static ssize_t do_readahead(struct file *file, unsigned long index, unsigned long nr)
 {
         struct address_space *mapping = file->f_dentry->d_inode->i_mapping;
-
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/



This archive was generated by hypermail 2b29 : Fri Jan 31 2003 - 22:00:23 EST