All of lore.kernel.org
 help / color / mirror / Atom feed
From: Carsten Otte <cotte@de.ibm.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Nick Piggin <npiggin@suse.de>, Jared Hulbert <jaredeh@gmail.com>,
	carsteno@de.ibm.com, Andrew Morton <akpm@linux-foundation.org>,
	mschwid2@linux.vnet.ibm.com, heicars2@linux.vnet.ibm.com,
	linux-mm@kvack.org, linux-fsdevel@vger.kernel.org
Subject: Re: [patch 4/6] xip: support non-struct page backed memory
Date: Tue, 04 Mar 2008 10:06:42 +0100	[thread overview]
Message-ID: <47CD1122.6090807@de.ibm.com> (raw)
In-Reply-To: <alpine.LFD.1.00.0803031420430.2979@woody.linux-foundation.org>

Linus Torvalds wrote:
> Implementing a kmap_pfn() sounds like a perfectly sane idea. But why does 
> it need to even be mapped into kernel space? Is it for the ELF header 
> reading or something (not having looked at the patch, just reacting to the 
> wrongness of using virt_to_phys())?
It needs to be mapped into kernel space to do regular file operations 
other than mmap. In mm/filemap_xip.c we do access the xip memory from 
kernel to fulfill sys_read/sys_write and friends [copy_from/to_user to 
user's buffer].

so long,
Carsten

WARNING: multiple messages have this Message-ID (diff)
From: Carsten Otte <cotte@de.ibm.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Nick Piggin <npiggin@suse.de>, Jared Hulbert <jaredeh@gmail.com>,
	carsteno@de.ibm.com, Andrew Morton <akpm@linux-foundation.org>,
	mschwid2@linux.vnet.ibm.com, heicars2@linux.vnet.ibm.com,
	linux-mm@kvack.org, linux-fsdevel@vger.kernel.org
Subject: Re: [patch 4/6] xip: support non-struct page backed memory
Date: Tue, 04 Mar 2008 10:06:42 +0100	[thread overview]
Message-ID: <47CD1122.6090807@de.ibm.com> (raw)
In-Reply-To: <alpine.LFD.1.00.0803031420430.2979@woody.linux-foundation.org>

Linus Torvalds wrote:
> Implementing a kmap_pfn() sounds like a perfectly sane idea. But why does 
> it need to even be mapped into kernel space? Is it for the ELF header 
> reading or something (not having looked at the patch, just reacting to the 
> wrongness of using virt_to_phys())?
It needs to be mapped into kernel space to do regular file operations 
other than mmap. In mm/filemap_xip.c we do access the xip memory from 
kernel to fulfill sys_read/sys_write and friends [copy_from/to_user to 
user's buffer].

so long,
Carsten

--
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/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  parent reply	other threads:[~2008-03-04  9:07 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20080118045649.334391000@suse.de>
2008-01-18  4:56 ` [patch 1/6] mm: introduce VM_MIXEDMAP npiggin, Jared Hulbert
2008-01-18  4:56 ` [patch 2/6] mm: introduce pte_special pte bit npiggin-l3A5Bk7waGM
2008-01-18  4:56   ` npiggin
2008-01-18  4:56   ` npiggin
2008-01-18 16:41   ` Linus Torvalds
2008-01-18 16:41     ` Linus Torvalds
2008-01-18 18:04     ` Sam Ravnborg
2008-01-18 18:04       ` Sam Ravnborg
2008-01-18 18:28       ` Linus Torvalds
2008-01-18 18:28         ` Linus Torvalds
2008-01-18 18:53         ` Sam Ravnborg
2008-01-18 18:53           ` Sam Ravnborg
2008-01-18 22:46     ` Nick Piggin
2008-01-18 22:46       ` Nick Piggin
2008-01-18 23:03       ` Linus Torvalds
2008-01-18 23:03         ` Linus Torvalds
2008-01-19  5:07         ` Nick Piggin
2008-01-19  5:07           ` Nick Piggin
2008-01-21  9:43       ` Nick Piggin
2008-01-21  9:43         ` Nick Piggin
2008-01-18  4:56 ` [patch 3/6] mm: add vm_insert_mixed npiggin
2008-01-18  4:56 ` [patch 4/6] xip: support non-struct page backed memory npiggin
2008-01-18  4:56   ` npiggin
2008-03-01  8:14   ` Jared Hulbert
2008-03-01  8:14     ` Jared Hulbert
2008-03-03  5:29     ` Nick Piggin
2008-03-03  5:29       ` Nick Piggin
2008-03-03  8:30       ` Carsten Otte
2008-03-03  8:30         ` Carsten Otte
2008-03-03 15:59       ` Jared Hulbert
2008-03-03 15:59         ` Jared Hulbert
2008-03-03  8:18     ` Carsten Otte
2008-03-03  8:18       ` Carsten Otte
2008-03-03 15:44       ` Jared Hulbert
2008-03-03 15:44         ` Jared Hulbert
2008-03-03 18:40       ` Linus Torvalds
2008-03-03 18:40         ` Linus Torvalds
2008-03-03 19:38         ` Jared Hulbert
2008-03-03 19:38           ` Jared Hulbert
2008-03-03 20:04           ` Linus Torvalds
2008-03-03 20:04             ` Linus Torvalds
2008-03-03 20:32             ` Nick Piggin
2008-03-03 20:32               ` Nick Piggin
2008-03-03 22:21               ` Linus Torvalds
2008-03-03 22:21                 ` Linus Torvalds
2008-03-03 23:25                 ` Jared Hulbert
2008-03-03 23:25                   ` Jared Hulbert
2008-03-04  9:06                 ` Carsten Otte [this message]
2008-03-04  9:06                   ` Carsten Otte

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=47CD1122.6090807@de.ibm.com \
    --to=cotte@de.ibm.com \
    --cc=akpm@linux-foundation.org \
    --cc=carsteno@de.ibm.com \
    --cc=heicars2@linux.vnet.ibm.com \
    --cc=jaredeh@gmail.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mschwid2@linux.vnet.ibm.com \
    --cc=npiggin@suse.de \
    --cc=torvalds@linux-foundation.org \
    /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.