diff options
author | KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> | 2009-11-17 16:21:09 +0900 |
---|---|---|
committer | David Woodhouse <David.Woodhouse@intel.com> | 2009-12-08 10:12:04 +0000 |
commit | 354bb65e6e0df0aaae0e5b1ea33948d8e0b61418 (patch) | |
tree | afcb7dca4c8362442ec50ce799290ae1211bde19 /drivers/pci/hotplug-pci.c | |
parent | 1672af1164d3d50ba8908014fd34cc0b58afdc1e (diff) | |
download | blackbird-op-linux-354bb65e6e0df0aaae0e5b1ea33948d8e0b61418.tar.gz blackbird-op-linux-354bb65e6e0df0aaae0e5b1ea33948d8e0b61418.zip |
Revert "Intel IOMMU: Avoid memory allocation failures in dma map api calls"
commit eb3fa7cb51 said Intel IOMMU
Intel IOMMU driver needs memory during DMA map calls to setup its
internal page tables and for other data structures. As we all know
that these DMA map calls are mostly called in the interrupt context
or with the spinlock held by the upper level drivers(network/storage
drivers), so in order to avoid any memory allocation failure due to
low memory issues, this patch makes memory allocation by temporarily
setting PF_MEMALLOC flags for the current task before making memory
allocation calls.
We evaluated mempools as a backup when kmem_cache_alloc() fails
and found that mempools are really not useful here because
1) We don't know for sure how much to reserve in advance
2) And mempools are not useful for GFP_ATOMIC case (as we call
memory alloc functions with GFP_ATOMIC)
(akpm: point 2 is wrong...)
The above description doesn't justify to waste system emergency memory
at all. Non MM subsystem must not use PF_MEMALLOC. Memory reclaim need
few memory, anyone must not prevent it. Otherwise the system cause
mysterious hang-up and/or OOM Killer invokation.
Plus, akpm already pointed out what we should do.
Then, this patch revert it.
Cc: Keshavamurthy Anil S <anil.s.keshavamurthy@intel.com>
Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Diffstat (limited to 'drivers/pci/hotplug-pci.c')
0 files changed, 0 insertions, 0 deletions