public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Arun Sharma <arun.sharma@google.com>
To: Andrew Morton <akpm@osdl.org>
Cc: rohit.seth@intel.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Expose SHM_HUGETLB in shmctl(id, IPC_STAT, ...)
Date: Thu, 10 Nov 2005 13:41:33 -0800	[thread overview]
Message-ID: <4373BE8D.2070104@google.com> (raw)
In-Reply-To: <20051110115941.1cbe1ae7.akpm@osdl.org>

Andrew Morton wrote:

>>The man page on my system says:
>>
>>               unsigned short mode;  /* Permissions + SHM_DEST and
>>                                         SHM_LOCKED flags */
>>
>>I looked for a precendent before sending the patch and thought that 
>>SHM_LOCKED was a good one.
>>
> 
> 
> hm, OK.   But an app could still do
> 
> 	if (mode == 0666|SHM_LOCKED)
> 

I'd argue that the app should really be doing (perm.mode & 0777 = 0666)

> 
> How important is this feature?

Without this feature, an application has no way to figure out if a given 
segment is hugetlb or not. Applications need to know this to be able to 
handle alignment issues properly.

Also, if the flag is exported via ipcs, the system administrator would 
have a better idea about how the hugetlb pages she configured on the 
system are getting used.

	-Arun

  reply	other threads:[~2005-11-10 21:41 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-09 18:46 [PATCH] Expose SHM_HUGETLB in shmctl(id, IPC_STAT, ...) Arun Sharma
2005-11-10  6:22 ` Andrew Morton
2005-11-10 18:35   ` Arun Sharma
2005-11-10 19:59     ` Andrew Morton
2005-11-10 21:41       ` Arun Sharma [this message]
2005-11-10 22:06         ` Andrew Morton
2005-11-11  2:49           ` Arun Sharma
2005-11-11  3:12             ` Andrew Morton
2005-11-12  7:19               ` Michael Kerrisk
2005-11-12  7:38                 ` Andrew Morton
2005-11-11  7:30             ` William Lee Irwin III

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=4373BE8D.2070104@google.com \
    --to=arun.sharma@google.com \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rohit.seth@intel.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