linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: GOTO Masanori <gotom@debian.or.jp>
To: hans-christoph.rohland@sap.com
Cc: linux-kernel@vger.rutgers.edu, linux-MM@kvack.org,
	Linus Torvalds <torvalds@transmeta.com>,
	gotom@debian.or.jp
Subject: Re: [PATCH] shm fs v2 against 2.3.41
Date: Tue, 01 Feb 2000 19:07:20 +0900	[thread overview]
Message-ID: <20000201190720E.gotom@fe.dis.titech.ac.jp> (raw)
In-Reply-To: <qwwemazzj8u.fsf@sap.com>

Calling shmget( key, size, shmflg ) with size = 0,
I got an error EINVAL. The below patch fix it,
please apply into 2.3.41+shmfs14 patch.

---------------------
--- linux-2.3.41_shmfs14/ipc/shm.c      Tue Feb  1 18:49:02 2000
+++ linux-2.3.41_shmfs14_fixed/ipc/shm.c        Tue Feb  1 18:57:52 2000
@@ -660,7 +660,7 @@
                return -EINVAL;
        }
 
-       if (size < SHMMIN)
+       if ((size != 0) && (size < SHMMIN))
                return -EINVAL;
 
        down(&shm_ids.sem);
---------------------

And now I have a question:
I guess almost all users have no shmpath (default: /var/shm),
and they maybe make a dir and have to mount it.
IMHO, it is better to change that sysv shared memory works
samely, whenever shmfs is not mounted. Is it feasible, 
or only my mistaken ?


From: Christoph Rohland <hans-christoph.rohland@sap.com>
Subject: [PATCH] shm fs v2 against 2.3.41
> Hi Folks,
> 
> Here is my newest version of shm over a filesystem:
> 
> Changes to the previous versio:
> 
> - It does not try to autodetect the path any more. Per default it
>   expects to be mounted at /var/shm. If you mount it somewhere else,
>   put the new path into /proc/sys/kernel/shmpath
> - No more /proc/sys/kernel/{shmall,shmmni}. Use mount options
>   nr_blocks and nr_pages. You can change these parameters with remount.
> - You can set the initial mode of the directory with mount option 'mode'.
> - It frees all objects on umount.
> 
> I tested the shm fs heavily on 2.3.40+some patches to make smp and
> page_cache stable. It survived one day swap test.

Regards,
-- GOTO Masanori
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux.eu.org/Linux-MM/

  reply	other threads:[~2000-02-01 10:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-01-30 15:12 Christoph Rohland
2000-02-01 10:07 ` GOTO Masanori [this message]
2000-02-01 12:46   ` Christoph Rohland
2000-02-02 14:24   ` Stephen C. Tweedie

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=20000201190720E.gotom@fe.dis.titech.ac.jp \
    --to=gotom@debian.or.jp \
    --cc=hans-christoph.rohland@sap.com \
    --cc=linux-MM@kvack.org \
    --cc=linux-kernel@vger.rutgers.edu \
    --cc=torvalds@transmeta.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