All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexey Dobriyan <adobriyan@gmail.com>
To: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-mm <linux-mm@kvack.org>, Hugh Dickins <hughd@google.com>,
	Dave Hansen <dave@linux.vnet.ibm.com>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Paul Mundt <lethal@linux-sh.org>,
	Russell King <linux@arm.linux.org.uk>
Subject: Re: mm: convert vma->vm_flags to 64bit
Date: Wed, 13 Apr 2011 09:44:32 +0300	[thread overview]
Message-ID: <20110413064432.GA4098@p183> (raw)
In-Reply-To: <20110413091301.41E1.A69D9226@jp.fujitsu.com>

On Wed, Apr 13, 2011 at 09:13:03AM +0900, KOSAKI Motohiro wrote:
> > On Tue, 2011-04-12 at 14:06 +0300, Alexey Dobriyan wrote:
> > > On Tue, Apr 12, 2011 at 10:12 AM, KOSAKI Motohiro
> > > <kosaki.motohiro@jp.fujitsu.com> wrote:
> > > > After next year? All developers don't have to ignore compiler warnings!
> > > 
> > > At least add vm_flags_t which is sparse-checked, just like we do with gfp_t.
> > > 
> > > VM_SAO is ppc64 only, so it could be moved into high part,
> > > freeing 1 bit?
> > 
> > My original series did use a type, I don't know what that was dropped,
> > it made conversion easier imho.
> 
> Yes, I take Hugh's version because vm_flags_t is ugly to me. And arch 
> dependent variable size is problematic.

Who said it should have arch-dependent size?

WARNING: multiple messages have this Message-ID (diff)
From: Alexey Dobriyan <adobriyan@gmail.com>
To: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-mm <linux-mm@kvack.org>, Hugh Dickins <hughd@google.com>,
	Dave Hansen <dave@linux.vnet.ibm.com>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Paul Mundt <lethal@linux-sh.org>,
	Russell King <linux@arm.linux.org.uk>
Subject: Re: mm: convert vma->vm_flags to 64bit
Date: Wed, 13 Apr 2011 09:44:32 +0300	[thread overview]
Message-ID: <20110413064432.GA4098@p183> (raw)
In-Reply-To: <20110413091301.41E1.A69D9226@jp.fujitsu.com>

On Wed, Apr 13, 2011 at 09:13:03AM +0900, KOSAKI Motohiro wrote:
> > On Tue, 2011-04-12 at 14:06 +0300, Alexey Dobriyan wrote:
> > > On Tue, Apr 12, 2011 at 10:12 AM, KOSAKI Motohiro
> > > <kosaki.motohiro@jp.fujitsu.com> wrote:
> > > > After next year? All developers don't have to ignore compiler warnings!
> > > 
> > > At least add vm_flags_t which is sparse-checked, just like we do with gfp_t.
> > > 
> > > VM_SAO is ppc64 only, so it could be moved into high part,
> > > freeing 1 bit?
> > 
> > My original series did use a type, I don't know what that was dropped,
> > it made conversion easier imho.
> 
> Yes, I take Hugh's version because vm_flags_t is ugly to me. And arch 
> dependent variable size is problematic.

Who said it should have arch-dependent size?

--
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-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2011-04-13  6:44 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-12  6:10 mm: convert vma->vm_flags to 64bit KOSAKI Motohiro
2011-04-12  6:10 ` KOSAKI Motohiro
2011-04-12  6:33 ` Andrew Morton
2011-04-12  6:33   ` Andrew Morton
2011-04-12  7:12   ` KOSAKI Motohiro
2011-04-12  7:12     ` KOSAKI Motohiro
2011-04-12 11:06     ` Alexey Dobriyan
2011-04-12 11:06       ` Alexey Dobriyan
2011-04-12 11:11       ` Alexey Dobriyan
2011-04-12 11:11         ` Alexey Dobriyan
2011-04-12 22:07       ` Benjamin Herrenschmidt
2011-04-12 22:07         ` Benjamin Herrenschmidt
2011-04-13  0:13         ` KOSAKI Motohiro
2011-04-13  0:13           ` KOSAKI Motohiro
2011-04-13  6:44           ` Alexey Dobriyan [this message]
2011-04-13  6:44             ` Alexey Dobriyan
2011-04-13  7:00             ` Benjamin Herrenschmidt
2011-04-13  7:00               ` Benjamin Herrenschmidt
2011-04-13  7:29               ` Russell King - ARM Linux
2011-04-13  7:29                 ` Russell King - ARM Linux
2011-04-13  8:56                 ` Benjamin Herrenschmidt
2011-04-13  8:56                   ` Benjamin Herrenschmidt
2011-04-13  8:34               ` KOSAKI Motohiro
2011-04-13  8:34                 ` KOSAKI Motohiro
2011-04-13  7:04             ` KOSAKI Motohiro
2011-04-13  7:04               ` KOSAKI Motohiro
2011-04-12 23:41       ` KOSAKI Motohiro
2011-04-12 23:41         ` KOSAKI Motohiro
2011-04-13  2:19         ` [PATCH 1/3] mm: add __nocast attribute to vm_flags KOSAKI Motohiro
2011-04-13  2:19           ` KOSAKI Motohiro
2011-04-13  2:20           ` [PATCH 2/3] fremap: convert vm_flags to unsigned long long KOSAKI Motohiro
2011-04-13  2:20             ` KOSAKI Motohiro
2011-04-13  2:21           ` [PATCH 3/3] procfs: " KOSAKI Motohiro
2011-04-13  2:21             ` KOSAKI Motohiro
2011-04-12 20:30   ` mm: convert vma->vm_flags to 64bit Russell King - ARM Linux
2011-04-12 20:30     ` Russell King - ARM Linux
2011-04-18  0:26 ` Hugh Dickins
2011-04-18  0:26   ` Hugh Dickins
2011-04-18  1:21   ` KOSAKI Motohiro
2011-04-18  1:21     ` KOSAKI Motohiro
2011-04-18  1:45   ` Benjamin Herrenschmidt
2011-04-18  1:45     ` Benjamin Herrenschmidt
2011-04-18  3:34     ` KOSAKI Motohiro
2011-04-18  3:34       ` KOSAKI Motohiro
2011-11-10  4:09 ` Nai Xia
2011-11-10  4:09   ` Nai Xia
2011-11-10  4:49   ` David Rientjes
2011-11-10  4:49     ` David Rientjes
2011-11-11  8:52     ` Russell King - ARM Linux
2011-11-11  8:52       ` Russell King - ARM Linux
2011-11-10 17:22   ` KOSAKI Motohiro
2011-11-10 17:22     ` KOSAKI Motohiro
2011-11-10 21:12     ` Benjamin Herrenschmidt
2011-11-10 21:12       ` Benjamin Herrenschmidt
2011-11-10 21:49       ` KOSAKI Motohiro
2011-11-10 21:49         ` KOSAKI Motohiro
2011-11-11  8:42         ` Russell King - ARM Linux
2011-11-11  8:42           ` Russell King - ARM Linux
2011-11-11  2:09       ` Hugh Dickins
2011-11-11  2:09         ` Hugh Dickins
2011-11-11  4:31         ` Benjamin Herrenschmidt
2011-11-11  4:31           ` Benjamin Herrenschmidt
2011-11-11  8:38           ` Nai Xia
2011-11-11  8:38             ` Nai Xia

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=20110413064432.GA4098@p183 \
    --to=adobriyan@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=benh@kernel.crashing.org \
    --cc=dave@linux.vnet.ibm.com \
    --cc=hughd@google.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=kosaki.motohiro@jp.fujitsu.com \
    --cc=lethal@linux-sh.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux@arm.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.