Module Name:src
Committed By: pho
Date: Fri Dec 3 13:08:10 UTC 2021
Modified Files:
src/sys/fs/puffs: puffs_msgif.h
Log Message:
Avoid using register_t in
The purpose of this header file is to interface between the
kernel-space and user-space, and is #include'd by a
Module Name:src
Committed By: pho
Date: Fri Dec 3 13:08:10 UTC 2021
Modified Files:
src/sys/fs/puffs: puffs_msgif.h
Log Message:
Avoid using register_t in
The purpose of this header file is to interface between the
kernel-space and user-space, and is #include'd by a
Module Name:src
Committed By: christos
Date: Fri Sep 27 22:36:57 UTC 2019
Modified Files:
src/sys/fs/puffs: puffs_vfsops.c
Log Message:
Fix copying issue that was causing errors in unit_test puffs_tstavfs by
removing code.
To generate a diff of this commit:
cvs rdiff -u
Module Name:src
Committed By: christos
Date: Fri Sep 27 22:36:57 UTC 2019
Modified Files:
src/sys/fs/puffs: puffs_vfsops.c
Log Message:
Fix copying issue that was causing errors in unit_test puffs_tstavfs by
removing code.
To generate a diff of this commit:
cvs rdiff -u
On Tue, Oct 07, 2014 at 12:40:04AM +0700, Robert Elz wrote:
Date:Mon, 6 Oct 2014 14:26:44 +
From:Havard Eidnes h...@netbsd.org
Message-ID: 20141006142644.7693...@cvs.netbsd.org
| Make this build again without debugging enabled; DPRINTF() can end up
| as
On Wed, 15 Oct 2014, David Laight wrote:
Consider what happens if you write:
if (error)
DPRINTF((...));
else
fubar();
When DPRINTF() expands 'if (xxx) yyy' it all goes horribly wrong.
That's why I changed it to
do { if (xxx) yyy; }
Date:Wed, 15 Oct 2014 19:41:40 +0100
From:David Laight da...@l8s.co.uk
Message-ID: 20141015184140.gl23...@snowdrop.l8s.co.uk
| Consider what happens if you write:
| When DPRINTF() expands 'if (xxx) yyy' it all goes horribly wrong.
Of course, the do { } while(0)
Date:Mon, 6 Oct 2014 14:26:44 +
From:Havard Eidnes h...@netbsd.org
Message-ID: 20141006142644.7693...@cvs.netbsd.org
| Make this build again without debugging enabled; DPRINTF() can end up
| as empty, and in an if conditional, you then need braces if that's
| Make this build again without debugging enabled; DPRINTF() can end up
| as empty, and in an if conditional, you then need braces if that's the
| only potential body.
That change makes no sense to me - the original code was
if (error)
DPRINTF(());
and even
On Sun, Oct 05, 2014 at 02:13:15PM +, Alan Barrett wrote:
#ifdef PUFFSDEBUG
extern int puffsdebug; /* puffs_subr.c */
-#define DPRINTF(x) if (puffsdebug 0) printf x
-#define DPRINTF_VERBOSE(x) if (puffsdebug 1) printf x
+#define DPRINTF(x) do { \
+ if (puffsdebug 0)
David Holland dholland-sourcechan...@netbsd.org wrote:
Di not call cache_enter with path components bigger than NCHNAMLEN, as it
panics the kernel.
This is wrong; you're reverting rmind's cleanup.
I backed out the change. Will someone pullup the change to netbsd-6?
--
Emmanuel Dreyfus
On Mon, Jul 23, 2012 at 10:07:06PM +, Martin Husemann wrote:
This is wrong; you're reverting rmind's cleanup.
I backed out the change. Will someone pullup the change to netbsd-6?
Can you prepare a patch that applies before rmind's cleanup? (Or just
one that applies on netbsd-6?)
David Holland dholland-sourcechan...@netbsd.org wrote:
OTOH for netbsd-6 maybe a better patch would be to change cache_enter
to return instead of asserting for long filenames.
If nobody opposes, I can do that.
--
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
m...@netbsd.org
On Tue, Jul 24, 2012 at 07:13:34AM +0200, Emmanuel Dreyfus wrote:
OTOH for netbsd-6 maybe a better patch would be to change cache_enter
to return instead of asserting for long filenames.
If nobody opposes, I can do that.
Best to ask on tech-kern, but it seems unlikely to be
On Tue, Oct 18, 2011 at 03:39:10PM +, Emmanuel Dreyfus wrote:
Module Name: src
Committed By: manu
Date: Tue Oct 18 15:39:09 UTC 2011
Modified Files:
src/sys/fs/puffs: puffs_msgif.c puffs_node.c puffs_vfsops.c
puffs_vnops.c
Log Message:
Make sure pagedaemon
Jukka Ruohonen jruoho...@iki.fi wrote:
+#ifdef DIAGNOSTIC
+ KASSERT(curlwp != uvm.pagedaemon_lwp);
+#endif
Why pollute code with redundant #ifdefs?
Right, I missed the point that KASSERT contains #ifdef DIAGNOSTIC. I
will fix that tomorrow morning.
--
Emmanuel Dreyfus
hi,
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote:
if it trapped forever, it's a bug and should be fixed. my point was
that your change didn't fix the bug. blocking ioflush is merely a symptom.
The problem with userland filesystems is that we may have little control
as theses may be
On Wed, Oct 12, 2011 at 06:02:05AM +, YAMAMOTO Takashi wrote:
One way to fix that may be to have one ioflush thread for each userland
filesystem. That way a broken filesystem will not prevent ioflush from
working for others.
sure, it can be a good idea.
I have a patch that does
hi,
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote:
Sure ioflush do not directly free memory, but vnodes' dirty page use
memory, don't they? If ioflush stops working, is pageadaemon able to
pageout that kind of memory?
pagedaemon flushes dirty pages by itself, yes.
So this is not a
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote:
if it trapped forever, it's a bug and should be fixed. my point was
that your change didn't fix the bug. blocking ioflush is merely a symptom.
The problem with userland filesystems is that we may have little control
as theses may be third pary
On Sat, Sep 24, 2011 at 03:31:02AM +, YAMAMOTO Takashi wrote:
pagedaemon flushes dirty pages by itself, yes.
I added a printf() at the beginning of puffs_vnop_strategy() to check for
calls by kernel threads. Only ioflush calls puffs_vnop_strategy(),
pagedaemon never goes there. This may
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote:
This avoids deadlocks in the following situations:
1) when memory is low: ioflush waits the fileystem, the fielsystem waits
for memory
can you explain how it is a problem?
As I understand, one way to free memory is to flush vnode backed
hi,
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote:
This avoids deadlocks in the following situations:
1) when memory is low: ioflush waits the fileystem, the fielsystem waits
for memory
can you explain how it is a problem?
As I understand, one way to free memory is to flush vnode
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote:
as i told you a few times, ioflush is not a thread to free memory.
pagedaemon is.
Sure ioflush do not directly free memory, but vnodes' dirty page use
memory, don't they? If ioflush stops working, is pageadaemon able to
pageout that kind of
hi,
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote:
as i told you a few times, ioflush is not a thread to free memory.
pagedaemon is.
Sure ioflush do not directly free memory, but vnodes' dirty page use
memory, don't they? If ioflush stops working, is pageadaemon able to
pageout that
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote:
Sure ioflush do not directly free memory, but vnodes' dirty page use
memory, don't they? If ioflush stops working, is pageadaemon able to
pageout that kind of memory?
pagedaemon flushes dirty pages by itself, yes.
So this is not a problem to
hi,
Module Name: src
Committed By: manu
Date: Wed Sep 21 15:36:33 UTC 2011
Modified Files:
src/sys/fs/puffs: puffs_vfsops.c puffs_vnops.c
Log Message:
Make sure ioflush does not sleep in PUFFS code path, waiting for a mutex,
a memory allocation, or a response from the
On Tue Jan 11 2011 at 14:04:54 +, Mihai Chelaru wrote:
Module Name: src
Committed By: kefren
Date: Tue Jan 11 14:04:54 UTC 2011
Modified Files:
src/sys/fs/puffs: puffs_sys.h puffs_vnops.c
Log Message:
add advlock to puffs. ok pooka@
should fix kern/43321
Thanks!
On Wed, Jul 14, 2010 at 02:07:37PM +, Antti Kantee wrote:
Modified Files:
src/sys/fs/puffs: puffs_vnops.c
Log Message:
RENAME lookup semantics say return EISDIR if dvp = *vpp for the
last component obviously(!!)
I take it that when you say *vpp, you mean tvp?
(the
On Thu Jul 15 2010 at 20:03:37 +, David Holland wrote:
On Wed, Jul 14, 2010 at 02:07:37PM +, Antti Kantee wrote:
Modified Files:
src/sys/fs/puffs: puffs_vnops.c
Log Message:
RENAME lookup semantics say return EISDIR if dvp = *vpp for the
last component
Module Name:src
Committed By: pooka
Date: Thu Jan 14 14:44:13 UTC 2010
Modified Files:
src/sys/fs/puffs: puffs_vnops.c
Log Message:
Since VOP_GETATTR() does not require a locked vnode, resolve and
reference the puffs_node before sending the request to the file
server.
Module Name:src
Committed By: pooka
Date: Thu Jan 7 22:45:31 UTC 2010
Modified Files:
src/sys/fs/puffs: puffs_msgif.c puffs_msgif.h puffs_sys.h
Log Message:
Add a PUFFS_UNMOUNT server-kernel request, which causes the kernel
to initiate self destruct, i.e.
Module Name:src
Committed By: pooka
Date: Thu Jan 7 22:59:27 UTC 2010
Modified Files:
src/sys/fs/puffs: puffs_msgif.c
Log Message:
Fix variable name in my commit tree too.
To generate a diff of this commit:
cvs rdiff -u -r1.77 -r1.78 src/sys/fs/puffs/puffs_msgif.c
33 matches
Mail list logo