X-Git-Url: https://wimlib.net/git/?a=blobdiff_plain;f=src%2Flzx-decompress.c;h=16b46cb253a264fc8b32b20a744e01176401acee;hb=4075b6ec704b03f387af8044762f228b60ffb2d6;hp=e1c0160cdfef246ab8e2eba26c2054dc5ea490d0;hpb=18f9daee201c3e79e6d508f1ce69206b8035ae4b;p=wimlib diff --git a/src/lzx-decompress.c b/src/lzx-decompress.c index e1c0160c..16b46cb2 100644 --- a/src/lzx-decompress.c +++ b/src/lzx-decompress.c @@ -107,18 +107,14 @@ # include "config.h" #endif -#include "wimlib.h" #include "wimlib/decompressor_ops.h" #include "wimlib/decompress_common.h" +#include "wimlib/error.h" #include "wimlib/lzx.h" #include "wimlib/util.h" #include -#ifdef __SSE2__ -# include -#endif - /* Huffman decoding tables and maps from symbols to code lengths. */ struct lzx_tables { @@ -527,9 +523,6 @@ lzx_decode_match(unsigned main_element, int block_type, unsigned num_extra_bits; u32 verbatim_bits; u32 aligned_bits; - unsigned i; - u8 *match_dest; - u8 *match_src; /* The main element is offset by 256 because values under 256 indicate a * literal value. */ @@ -552,22 +545,15 @@ lzx_decode_match(unsigned main_element, int block_type, /* If the position_slot is 0, 1, or 2, the match offset is retrieved * from the LRU queue. Otherwise, the match offset is not in the LRU * queue. */ - switch (position_slot) { - case 0: - match_offset = queue->R[0]; - break; - case 1: - match_offset = queue->R[1]; - swap(queue->R[0], queue->R[1]); - break; - case 2: - /* The queue doesn't work quite the same as a real LRU queue, - * since using the R2 offset doesn't bump the R1 offset down to - * R2. */ - match_offset = queue->R[2]; - swap(queue->R[0], queue->R[2]); - break; - default: + if (position_slot <= 2) { + /* Note: This isn't a real LRU queue, since using the R2 offset + * doesn't bump the R1 offset down to R2. This quirk allows all + * 3 recent offsets to be handled by the same code. (For R0, + * the swap is a no-op.) */ + match_offset = queue->R[position_slot]; + queue->R[position_slot] = queue->R[0]; + queue->R[0] = match_offset; + } else { /* Otherwise, the offset was not encoded as one the offsets in * the queue. Depending on the position slot, there is a * certain number of extra bits that need to be read to fully @@ -612,7 +598,6 @@ lzx_decode_match(unsigned main_element, int block_type, queue->R[2] = queue->R[1]; queue->R[1] = queue->R[0]; queue->R[0] = match_offset; - break; } /* Verify that the match is in the bounds of the part of the window @@ -633,171 +618,12 @@ lzx_decode_match(unsigned main_element, int block_type, return -1; } - match_dest = window + window_pos; - match_src = match_dest - match_offset; - -#if 0 - printf("Match: src %u, dst %u, len %u\n", match_src - window, - match_dest - window, - match_len); - putchar('|'); - for (i = 0; i < match_len; i++) { - match_dest[i] = match_src[i]; - putchar(match_src[i]); - } - putchar('|'); - putchar('\n'); -#else - for (i = 0; i < match_len; i++) - match_dest[i] = match_src[i]; -#endif + lz_copy(&window[window_pos], match_len, match_offset, + &window[window_pos + bytes_remaining]); return match_len; } -static void -undo_call_insn_translation(u32 *call_insn_target, s32 input_pos) -{ - s32 abs_offset; - s32 rel_offset; - - abs_offset = le32_to_cpu(*call_insn_target); - if (abs_offset >= 0) { - if (abs_offset < LZX_WIM_MAGIC_FILESIZE) { - /* "good translation" */ - rel_offset = abs_offset - input_pos; - - *call_insn_target = cpu_to_le32(rel_offset); - } - } else { - if (abs_offset >= -input_pos) { - /* "compensating translation" */ - rel_offset = abs_offset + LZX_WIM_MAGIC_FILESIZE; - - *call_insn_target = cpu_to_le32(rel_offset); - } - } -} - -/* Undo the 'E8' preprocessing, where the targets of x86 CALL instructions were - * changed from relative offsets to absolute offsets. - * - * Note that this call instruction preprocessing can and will be used on any - * data even if it is not actually x86 machine code. In fact, this type of - * preprocessing appears to always be used in LZX-compressed resources in WIM - * files; there is no bit to indicate whether it is used or not, unlike in the - * LZX compressed format as used in cabinet files, where a bit is reserved for - * that purpose. - * - * Call instruction preprocessing is disabled in the last 6 bytes of the - * uncompressed data, which really means the 5-byte call instruction cannot - * start in the last 10 bytes of the uncompressed data. This is one of the - * errors in the LZX documentation. - * - * Call instruction preprocessing does not appear to be disabled after the - * 32768th chunk of a WIM stream, which is apparently is yet another difference - * from the LZX compression used in cabinet files. - * - * Call instruction processing is supposed to take the file size as a parameter, - * as it is used in calculating the translated jump targets. But in WIM files, - * this file size is always the same (LZX_WIM_MAGIC_FILESIZE == 12000000).*/ -static void -undo_call_insn_preprocessing(u8 *uncompressed_data, size_t uncompressed_size) -{ -#ifdef __SSE2__ - - /* SSE2 vectorized implementation for x86_64. This speeds up LZX - * decompression by about 5-8% overall. (Usually --- the performance - * actually regresses slightly in the degenerate case that the data - * consists entirely of 0xe8 bytes.) */ - __m128i *p128 = (__m128i *)uncompressed_data; - u32 valid_mask = 0xFFFFFFFF; - - if (uncompressed_size >= 32 && - ((uintptr_t)uncompressed_data % 16 == 0)) - { - __m128i * const end128 = p128 + uncompressed_size / 16 - 1; - - /* Create a vector of all 0xe8 bytes */ - const __m128i e8_bytes = _mm_set1_epi8(0xe8); - - /* Iterate through the 16-byte vectors in the input. */ - do { - /* Compare the current 16-byte vector with the vector of - * all 0xe8 bytes. This produces 0xff where the byte is - * 0xe8 and 0x00 where it is not. */ - __m128i cmpresult = _mm_cmpeq_epi8(*p128, e8_bytes); - - /* Map the comparison results into a single 16-bit - * number. It will contain a 1 bit when the - * corresponding byte in the current 16-byte vector is - * an e8 byte. Note: the low-order bit corresponds to - * the first (lowest address) byte. */ - u32 e8_mask = _mm_movemask_epi8(cmpresult); - - if (!e8_mask) { - /* If e8_mask is 0, then none of these 16 bytes - * have value 0xe8. No e8 translation is - * needed, and there is no restriction that - * carries over to the next 16 bytes. */ - valid_mask = 0xFFFFFFFF; - } else { - /* At least one byte has value 0xe8. - * - * The AND with valid_mask accounts for the fact - * that we can't start an e8 translation that - * overlaps the previous one. */ - while ((e8_mask &= valid_mask)) { - - /* Count the number of trailing zeroes - * in e8_mask. This will produce the - * index of the byte, within the 16, at - * which the next e8 translation should - * be done. */ - u32 bit = __builtin_ctz(e8_mask); - - /* Do the e8 translation. */ - u8 *p8 = (u8 *)p128 + bit; - undo_call_insn_translation((s32 *)(p8 + 1), - p8 - uncompressed_data); - - /* Don't start an e8 translation in the - * next 4 bytes. */ - valid_mask &= ~((u32)0x1F << bit); - } - /* Moving on to the next vector. Shift and set - * valid_mask accordingly. */ - valid_mask >>= 16; - valid_mask |= 0xFFFF0000; - } - } while (++p128 < end128); - } - - u8 *p8 = (u8 *)p128; - while (!(valid_mask & 1)) { - p8++; - valid_mask >>= 1; - } -#else /* __SSE2__ */ - u8 *p8 = uncompressed_data; -#endif /* !__SSE2__ */ - - if (uncompressed_size > 10) { - /* Finish any bytes that weren't processed by the vectorized - * implementation. */ - u8 *p8_end = uncompressed_data + uncompressed_size - 10; - do { - if (*p8 == 0xe8) { - undo_call_insn_translation((s32 *)(p8 + 1), - p8 - uncompressed_data); - p8 += 5; - } else { - p8++; - } - } while (p8 < p8_end); - } -} - /* * Decompresses an LZX-compressed block of data from which the header has already * been read. @@ -955,7 +781,7 @@ lzx_decompress(const void *compressed_data, size_t compressed_size, } } if (e8_preprocessing_done) - undo_call_insn_preprocessing(uncompressed_data, uncompressed_size); + lzx_undo_e8_preprocessing(uncompressed_data, uncompressed_size); return 0; } @@ -964,20 +790,19 @@ lzx_free_decompressor(void *_ctx) { struct lzx_decompressor *ctx = _ctx; - FREE(ctx); + ALIGNED_FREE(ctx); } static int -lzx_create_decompressor(size_t max_window_size, - const struct wimlib_decompressor_params_header *params, - void **ctx_ret) +lzx_create_decompressor(size_t max_window_size, void **ctx_ret) { struct lzx_decompressor *ctx; if (!lzx_window_size_valid(max_window_size)) return WIMLIB_ERR_INVALID_PARAM; - ctx = MALLOC(sizeof(struct lzx_decompressor)); + ctx = ALIGNED_MALLOC(sizeof(struct lzx_decompressor), + DECODE_TABLE_ALIGNMENT); if (ctx == NULL) return WIMLIB_ERR_NOMEM;