c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2024-12-25 18:59:33 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix stack lzjb

Browse Source 
Reduce kernel stack usage by lzjb_compress() by moving uint16 array
off the stack and on to the heap.  The exact performance implications
of this I have not measured but we absolutely need to keep stack
usage to a minimum.  If/when this becomes and issue we optimize.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
This commit is contained in:
Brian Behlendorf 2010-08-26 10:58:19 -07:00
parent bf701a83c5
commit 18a89ba43d
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
module/zfs/lzjb.c
View File

@ -36,7 +36,7 @@
* source length if compression would overflow the destination buffer. * source length if compression would overflow the destination buffer.
*/ */
#include <sys/types.h> #include <sys/zfs_context.h>
#define MATCH_BITS 6 #define MATCH_BITS 6
#define MATCH_MIN 3 #define MATCH_MIN 3
@ -54,12 +54,15 @@ lzjb_compress(void *s_start, void *d_start, size_t s_len, size_t d_len, int n)
int copymask = 1 << (NBBY - 1); int copymask = 1 << (NBBY - 1);
int mlen, offset, hash; int mlen, offset, hash;
uint16_t *hp; uint16_t *hp;
uint16_t lempel[LEMPEL_SIZE] = { 0 }; uint16_t *lempel;
lempel = kmem_zalloc(LEMPEL_SIZE * sizeof (uint16_t), KM_SLEEP);
while (src < (uchar_t *)s_start + s_len) { while (src < (uchar_t *)s_start + s_len) {
if ((copymask <<= 1) == (1 << NBBY)) { if ((copymask <<= 1) == (1 << NBBY)) {
if (dst >= (uchar_t *)d_start + d_len - 1 - 2 * NBBY) if (dst >= (uchar_t *)d_start + d_len - 1 - 2 * NBBY) {
kmem_free(lempel, LEMPEL_SIZE*sizeof(uint16_t));
return (s_len); return (s_len);
}
copymask = 1; copymask = 1;
copymap = dst; copymap = dst;
*dst++ = 0; *dst++ = 0;
@ -89,6 +92,8 @@ lzjb_compress(void *s_start, void *d_start, size_t s_len, size_t d_len, int n)
*dst++ = *src++; *dst++ = *src++;
} }
} }
kmem_free(lempel, LEMPEL_SIZE * sizeof (uint16_t));
return (dst - (uchar_t *)d_start); return (dst - (uchar_t *)d_start);
} }