From: "Petr Vandrovec" <VANDROVE@vc.cvut.cz>
To: Manik Raina <manik@cisco.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH] small compile warning fix
Date: Tue, 30 Oct 2001 21:21:57 MET-1 [thread overview]
Message-ID: <7C6E1BF47C4@vcnet.vc.cvut.cz> (raw)
> > You can remove it completely. It should not be '#ifdef LATER', but
> > '#ifdef OLDVERSION'... ncp_add_mem_fromfs was invoked with lock on
> > ncp_server structure, but then it directly accessed userspace. It was
> > possible to use this to cause deadlock, so now ncpfs uses bounce buffers
> > and double copy instead of this.
> >
>
> would you include this in your patches or would you like me to make
> it #ifdef OLDVERSION ?
I removed ncp_add_mem_fromfs() from my tree...
> > I have some ncpfs patches, but I though that I'll leave them for 2.5.x.
> > Maybe it is time to change this decision.
... but as diff between my ncpfs and Alan's is about 30KB, I still did not
decide whether I'll send it for 2.4.x...
Petr Vandrovec
vandrove@vc.cvut.cz
next reply other threads:[~2001-10-30 20:22 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-10-30 21:21 Petr Vandrovec [this message]
-- strict thread matches above, loose matches on Subject: below --
2001-10-29 16:13 [PATCH] small compile warning fix Petr Vandrovec
2001-10-30 4:23 ` Manik Raina
2001-10-29 10:13 Manik Raina
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=7C6E1BF47C4@vcnet.vc.cvut.cz \
--to=vandrove@vc.cvut.cz \
--cc=linux-kernel@vger.kernel.org \
--cc=manik@cisco.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox