[gentoo-commits] repo/gentoo:master commit in: sys-fs/zfs-kmod/files/

[Jonathan Vasquez]

commit:     b04e5ed88450f6754b6e63fc87af6e0daac5fe29
Author:     Jonathan Vasquez <fearedbliss <AT> gentoo <DOT> org>
AuthorDate: Sun Feb 12 16:31:26 2017 +0000
Commit:     Jonathan Vasquez <fearedbliss <AT> gentoo <DOT> org>
CommitDate: Sun Feb 12 16:32:16 2017 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=b04e5ed8

sys-fs/zfs-kmod: removing unused patches

Package-Manager: Portage-2.3.3, Repoman-2.3.1

 ...zfs-kmod-0.6.0_rc9-remove-pfmalloc-1-of-3.patch | 44 --------------
 ...zfs-kmod-0.6.0_rc9-remove-pfmalloc-2-of-3.patch | 56 -----------------
 ...zfs-kmod-0.6.0_rc9-remove-pfmalloc-3-of-3.patch | 70 ----------------------
 3 files changed, 170 deletions(-)

diff --git a/sys-fs/zfs-kmod/files/zfs-kmod-0.6.0_rc9-remove-pfmalloc-1-of-3.patch b/sys-fs/zfs-kmod/files/zfs-kmod-0.6.0_rc9-remove-pfmalloc-1-of-3.patch
deleted file mode 100644
index 51a81363ea..0000000000
--- a/sys-fs/zfs-kmod/files/zfs-kmod-0.6.0_rc9-remove-pfmalloc-1-of-3.patch
+++ /dev/null
@@ -1,44 +0,0 @@
-From e2060f922f8526aecf437de4cc4dff4c3cdfae16 Mon Sep 17 00:00:00 2001
-From: Richard Yao <ryao@×××××××××××××.edu>
-Date: Tue, 15 May 2012 23:19:32 -0400
-Subject: [PATCH] Revert Fix ASSERTION(!dsl_pool_sync_context(tx->tx_pool))
-
-Commit eec8164771bee067c3cd55ed0a16dadeeba276de worked around an issue
-involving direct reclaim through the use of PF_MEMALLOC.   Since we
-are reworking thing to use KM_PUSHPAGE so that swap works, we revert
-this patch in favor of the use of KM_PUSHPAGE in the affected areas.
-
-Signed-off-by: Richard Yao <ryao@×××××××××××××.edu>
-Signed-off-by: Brian Behlendorf <behlendorf1@××××.gov>
-Issue #726
----
- module/zfs/txg.c |   13 -------------
- 1 file changed, 13 deletions(-)
-
-diff --git a/module/zfs/txg.c b/module/zfs/txg.c
-index 5021e44..8d037f7 100644
---- a/module/zfs/txg.c
-+++ b/module/zfs/txg.c
-@@ -372,19 +372,6 @@
- 	callb_cpr_t cpr;
- 	uint64_t start, delta;
- 
--#ifdef _KERNEL
--	/*
--	 * Disable the normal reclaim path for the txg_sync thread.  This
--	 * ensures the thread will never enter dmu_tx_assign() which can
--	 * otherwise occur due to direct reclaim.  If this is allowed to
--	 * happen the system can deadlock.  Direct reclaim call path:
--	 *
--	 *   ->shrink_icache_memory->prune_icache->dispose_list->
--	 *   clear_inode->zpl_clear_inode->zfs_inactive->dmu_tx_assign
--	 */
--	current->flags |= PF_MEMALLOC;
--#endif /* _KERNEL */
--
- 	txg_thread_enter(tx, &cpr);
- 
- 	start = delta = 0;
--- 
-1.7.10
-

diff --git a/sys-fs/zfs-kmod/files/zfs-kmod-0.6.0_rc9-remove-pfmalloc-2-of-3.patch b/sys-fs/zfs-kmod/files/zfs-kmod-0.6.0_rc9-remove-pfmalloc-2-of-3.patch
deleted file mode 100644
index 0a4fd26040..0000000000
--- a/sys-fs/zfs-kmod/files/zfs-kmod-0.6.0_rc9-remove-pfmalloc-2-of-3.patch
+++ /dev/null
@@ -1,56 +0,0 @@
-From 78d26b8497b3845fc8130981c76214d6788f7a9d Mon Sep 17 00:00:00 2001
-From: Richard Yao <ryao@×××××××××××××.edu>
-Date: Mon, 7 May 2012 14:14:45 -0400
-Subject: [PATCH] Revert Fix zpl_writepage() deadlock
-
-The commit, cfc9a5c88f91f7b4d606fce89505e1f404691ea5, to fix deadlocks
-in zpl_writepage() relied on PF_MEMALLOC.   That had the effect of
-disabling the direct reclaim path on all allocations originating from
-calls to this function, but it failed to address the actual cause of
-those deadlocks.  This led to the same deadlocks being observed with
-swap on zvols, but not with swap on the loop device, which exercises
-this code.
-
-The use of PF_MEMALLOC also had the side effect of permitting
-allocations to be made from ZONE_DMA in instances that did not require
-it.  This contributes to the possibility of panics caused by depletion
-of pages from ZONE_DMA.
-
-As such, we revert this patch in favor of a proper fix for both issues.
-
-Signed-off-by: Richard Yao <ryao@×××××××××××××.edu>
-Signed-off-by: Brian Behlendorf <behlendorf1@××××.gov>
-Issue #726
----
- module/zfs/zpl_file.c |   15 +--------------
- 1 file changed, 1 insertion(+), 14 deletions(-)
-
-diff --git a/module/zfs/zpl_file.c b/module/zfs/zpl_file.c
-index 5ac41c9..2e9f72a 100644
---- a/module/zfs/zpl_file.c
-+++ b/module/zfs/zpl_file.c
-@@ -358,20 +358,7 @@
- 	ASSERT(PageLocked(pp));
- 	ASSERT(!PageWriteback(pp));
- 
--	/*
--	 * Disable the normal reclaim path for zpl_putpage().  This
--	 * ensures that all memory allocations under this call path
--	 * will never enter direct reclaim.  If this were to happen
--	 * the VM might try to write out additional pages by calling
--	 * zpl_putpage() again resulting in a deadlock.
--	 */
--	if (current->flags & PF_MEMALLOC) {
--		(void) zfs_putpage(mapping->host, pp, wbc);
--	} else {
--		current->flags |= PF_MEMALLOC;
--		(void) zfs_putpage(mapping->host, pp, wbc);
--		current->flags &= ~PF_MEMALLOC;
--	}
-+	(void) zfs_putpage(mapping->host, pp, wbc);
- 
- 	return (0);
- }
--- 
-1.7.10
-

diff --git a/sys-fs/zfs-kmod/files/zfs-kmod-0.6.0_rc9-remove-pfmalloc-3-of-3.patch b/sys-fs/zfs-kmod/files/zfs-kmod-0.6.0_rc9-remove-pfmalloc-3-of-3.patch
deleted file mode 100644
index 2eb85754b0..0000000000
--- a/sys-fs/zfs-kmod/files/zfs-kmod-0.6.0_rc9-remove-pfmalloc-3-of-3.patch
+++ /dev/null
@@ -1,70 +0,0 @@
-From a1c889dd6fb04d70439074a9399c9ea8f29f2cdb Mon Sep 17 00:00:00 2001
-From: Richard Yao <ryao@×××××××××××××.edu>
-Date: Wed, 16 May 2012 18:16:02 -0400
-Subject: [PATCH] Revert Disable direct reclaim for z_wr_* threads
-
-This commit used PF_MEMALLOC to prevent a memory reclaim deadlock.
-However, commit 49be0ccf1fdc2ce852271d4d2f8b7a9c2c4be6db eliminated
-the invocation of __cv_init(), which was the cause of the deadlock.
-PF_MEMALLOC has the side effect of permitting pages from ZONE_DMA
-to be allocated.  The use of PF_MEMALLOC was found to cause stability
-problems when doing swap on zvols. Since this technique is known to
-cause problems and no longer fixes anything, we revert it.
-
-Signed-off-by: Richard Yao <ryao@×××××××××××××.edu>
-Signed-off-by: Brian Behlendorf <behlendorf1@××××.gov>
-Issue #726
----
- include/sys/zfs_context.h |    1 -
- module/zfs/spa.c          |    9 +++------
- 2 files changed, 3 insertions(+), 7 deletions(-)
-
-diff --git a/include/sys/zfs_context.h b/include/sys/zfs_context.h
-index e4af6fc..77dded3 100644
---- a/include/sys/zfs_context.h
-+++ b/include/sys/zfs_context.h
-@@ -382,7 +382,6 @@ extern kstat_t *kstat_create(char *, int,
- #define	TASKQ_DYNAMIC		0x0004	/* Use dynamic thread scheduling */
- #define	TASKQ_THREADS_CPU_PCT	0x0008	/* Scale # threads by # cpus */
- #define	TASKQ_DC_BATCH		0x0010	/* Mark threads as batch */
--#define	TASKQ_NORECLAIM		0x0020	/* Disable direct memory reclaim */
- 
- #define	TQ_SLEEP	KM_SLEEP	/* Can block for memory */
- #define	TQ_NOSLEEP	KM_NOSLEEP	/* cannot block for memory; may fail */
-diff --git a/module/zfs/spa.c b/module/zfs/spa.c
-index c33a33a..436cd26 100644
---- a/module/zfs/spa.c
-+++ b/module/zfs/spa.c
-@@ -617,8 +617,9 @@ static inline int spa_load_impl(spa_t *spa, uint64_t, nvlist_t *config,
- 
- static taskq_t *
- spa_taskq_create(spa_t *spa, const char *name, enum zti_modes mode,
--    uint_t value, uint_t flags)
-+    uint_t value)
- {
-+	uint_t flags = TASKQ_PREPOPULATE;
- 	boolean_t batch = B_FALSE;
- 
- 	switch (mode) {
-@@ -668,17 +669,13 @@ static inline int spa_load_impl(spa_t *spa, uint64_t, nvlist_t *config,
- 			const zio_taskq_info_t *ztip = &zio_taskqs[t][q];
- 			enum zti_modes mode = ztip->zti_mode;
- 			uint_t value = ztip->zti_value;
--			uint_t flags = 0;
- 			char name[32];
- 
--			if (t == ZIO_TYPE_WRITE)
--				flags |= TASKQ_NORECLAIM;
--
- 			(void) snprintf(name, sizeof (name),
- 			    "%s_%s", zio_type_name[t], zio_taskq_types[q]);
- 
- 			spa->spa_zio_taskq[t][q] =
--			    spa_taskq_create(spa, name, mode, value, flags);
-+			    spa_taskq_create(spa, name, mode, value);
- 		}
- 	}
- }
--- 
-1.7.10
-