Re: [PATCH] Export shmem_file_setup and shmem_getpage for DRM-GEM

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Nick Piggin
Date: Monday, August 4, 2008 - 2:02 am

On Monday 04 August 2008 11:54, Keith Packard wrote:

This is not an argument. shmem_file_setup is public because that's
how ipc is implemented, not because it makes sense to be used
anywhere else. Lots of other things in mm/ are public that should
never be used outside that directory for example.



This is how I'd suggested it work as well. I think a little bit
more effort should be spent looking at making this work.



Mapping the file into an address space might be a way to make it
work (using get_user_pages to get the struct page). splice might
also work. read_mapping_page or similar could also be something to
look at. But using shmem_getpage seems wrong because it circumvents
the vfs API.

If you genuinely have problems that can't be fit into existing
APIs without significant modification, and that is specific just to
your app, then we could always look at making special cases for you.
But it would be nice if we generically solve problems you have with
processes manipulating thousands of files.
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] PCI: Add pci_read_base() API, Eric Anholt, (Thu Jul 31, 11:58 pm)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Christoph Hellwig, (Fri Aug 1, 1:50 pm)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Nick Piggin, (Mon Aug 4, 2:02 am)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Stephane Marchesin, (Wed Aug 6, 9:20 am)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Arjan van de Ven, (Wed Aug 6, 10:24 am)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Stephane Marchesin, (Wed Aug 6, 10:32 am)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Stephane Marchesin, (Wed Aug 6, 11:09 am)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Stephane Marchesin, (Wed Aug 6, 7:16 pm)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Christoph Hellwig, (Sun Aug 10, 6:23 pm)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Christoph Hellwig, (Sun Aug 10, 6:30 pm)
Re: [PATCH] Export shmem_file_setup and shmem_getpage for ..., Christoph Hellwig, (Sun Aug 10, 6:34 pm)
Re: [PATCH] Export shmem_file_setup for DRM-GEM, Keith Packard, (Sun Aug 10, 8:03 pm)