lkml.org 
[lkml]   [2000]   [Feb]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [FIX] rd.c (was Re: Linux Status For 2.3.x: v 2.3.43)


On Sun, 13 Feb 2000 willy@thepuffingroup.com wrote:

> On Sun, Feb 13, 2000 at 01:54:31AM -0500, Alexander Viro wrote:
> >
> >
> > On Thu, 10 Feb 2000 willy@thepuffingroup.com wrote:
> >
> > > + inode->i_rdev = device;
> > > + inode->i_bdev = bdget(kdev_t_to_nr(device));
> >
> > Erm... Why not init_special_inode(inode, S_IFBLK|S_IRUSR, kdev_to_nr(device));
> >
> > > + out_inode->i_rdev = ram_device;
> > > + out_inode->i_bdev = bdget(kdev_t_to_nr(ram_device));
> >
> > ... and similar here?
>
> Agreed. I merged this part of B.D. Elliott's patch a couple of days ago.
> (actually using S_IRWXU, but that's unimportant). I'm not terribly
> happy about constructing the dentries and struct file either; is there
> a better way of doing this?

Well, to start with, you don't need the outfile at all. You know where you
are writing to, don't you? So you can just get rid of the bloody thing.
Check where you are using crd_outfp and you'll see. BTW, there's no need
in messing with inode either. Input is trickier...


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:56    [W:0.053 / U:0.100 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site