From: Al Viro <viro@ftp.linux.org.uk>
To: Blaisorblade <blaisorblade@yahoo.it>
Cc: user-mode-linux-devel@lists.sourceforge.net,
Al Viro <viro@zeniv.linux.org.uk>, Jeff Dike <jdike@addtoit.com>,
linux-kernel@vger.kernel.org
Subject: Re: [uml-devel] Re: [patch 1/2] Fixup symlink function pointers for hppfs [for 2.6.13]
Date: Wed, 21 Sep 2005 04:23:53 +0100 [thread overview]
Message-ID: <20050921032353.GU7992@ftp.linux.org.uk> (raw)
In-Reply-To: <200509181400.35765.blaisorblade@yahoo.it>
On Sun, Sep 18, 2005 at 02:00:35PM +0200, Blaisorblade wrote:
> On Friday 26 August 2005 23:48, Al Viro wrote:
> > On Fri, Aug 26, 2005 at 10:04:43PM +0200, Blaisorblade wrote:
> > > And beyond that what? I cannot even think what's the rest *. And
> > > "obvious" doesn't hold with me.
> Al, while at it, can I get a bit of help from you?
>
> We have a commented out version of
> arch/um/drivers/mconsole_kern.c:mconsole_proc(), which is supposed to read
> the contents of procfs from the internal kernel mount, rather than /proc (to
> avoid being faked out by hppfs).
>
> As remarked in comments, that code is broken (run on the host uml_mconsole
> <umid> proc <filename>, which will call that code, for 4-5 times gives you an
> oops inside UML). Can you help with that?
nd.mnt is what? NULL? There's your oops.
Grab fs/nfsctl.c::do_open() and s/nfsd/proc/g in there. FWIW, it's probably
better off in libfs.c with fs type as argument... For now, just copy it -
it's trivial and small enough; when I move the sucker to libfs, we'll
switch to that. And lose that deactivate_super(), obviously - you won't
be touching superblock anymore.
And for pity sake,
...
goto out;
...
out: ;
}
is spelled
...
return;
...
-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server. Download
it for free - -and be entered to win a 42" plasma tv or your very own
Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
WARNING: multiple messages have this Message-ID (diff)
From: Al Viro <viro@ftp.linux.org.uk>
To: Blaisorblade <blaisorblade@yahoo.it>
Cc: user-mode-linux-devel@lists.sourceforge.net,
Al Viro <viro@zeniv.linux.org.uk>, Jeff Dike <jdike@addtoit.com>,
linux-kernel@vger.kernel.org
Subject: Re: [uml-devel] Re: [patch 1/2] Fixup symlink function pointers for hppfs [for 2.6.13]
Date: Wed, 21 Sep 2005 04:23:53 +0100 [thread overview]
Message-ID: <20050921032353.GU7992@ftp.linux.org.uk> (raw)
In-Reply-To: <200509181400.35765.blaisorblade@yahoo.it>
On Sun, Sep 18, 2005 at 02:00:35PM +0200, Blaisorblade wrote:
> On Friday 26 August 2005 23:48, Al Viro wrote:
> > On Fri, Aug 26, 2005 at 10:04:43PM +0200, Blaisorblade wrote:
> > > And beyond that what? I cannot even think what's the rest *. And
> > > "obvious" doesn't hold with me.
> Al, while at it, can I get a bit of help from you?
>
> We have a commented out version of
> arch/um/drivers/mconsole_kern.c:mconsole_proc(), which is supposed to read
> the contents of procfs from the internal kernel mount, rather than /proc (to
> avoid being faked out by hppfs).
>
> As remarked in comments, that code is broken (run on the host uml_mconsole
> <umid> proc <filename>, which will call that code, for 4-5 times gives you an
> oops inside UML). Can you help with that?
nd.mnt is what? NULL? There's your oops.
Grab fs/nfsctl.c::do_open() and s/nfsd/proc/g in there. FWIW, it's probably
better off in libfs.c with fs type as argument... For now, just copy it -
it's trivial and small enough; when I move the sucker to libfs, we'll
switch to that. And lose that deactivate_super(), obviously - you won't
be touching superblock anymore.
And for pity sake,
...
goto out;
...
out: ;
}
is spelled
...
return;
...
next prev parent reply other threads:[~2005-09-21 3:23 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-08-26 14:57 [uml-devel] [patch 1/2] Fixup symlink function pointers for hppfs [for 2.6.13] blaisorblade
2005-08-26 14:57 ` blaisorblade
2005-08-26 19:03 ` [uml-devel] " Al Viro
2005-08-26 19:03 ` Al Viro
2005-08-26 20:04 ` [uml-devel] " Blaisorblade
2005-08-26 20:04 ` Blaisorblade
2005-08-26 21:48 ` [uml-devel] " Al Viro
2005-08-26 21:48 ` Al Viro
2005-09-18 12:00 ` [uml-devel] " Blaisorblade
2005-09-18 12:00 ` Blaisorblade
2005-09-21 3:23 ` Al Viro [this message]
2005-09-21 3:23 ` Al Viro
2005-09-21 3:40 ` Al Viro
2005-09-21 3:40 ` Al Viro
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20050921032353.GU7992@ftp.linux.org.uk \
--to=viro@ftp.linux.org.uk \
--cc=blaisorblade@yahoo.it \
--cc=jdike@addtoit.com \
--cc=linux-kernel@vger.kernel.org \
--cc=user-mode-linux-devel@lists.sourceforge.net \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.