Skip to content

Commit

Permalink
Fix is_cdef_enabled in av1_alloc_cdef_buffers()
Browse files Browse the repository at this point in the history
Make the condition for is_cdef_enabled less strict in
av1_alloc_cdef_buffers(). Use !cm->tiles.single_tile_decoding instead of
!cm->tiles.large_scale because the code in
av1_decode_tg_tiles_and_wrapup() that calls av1_cdef_frame() or
av1_cdef_frame_mt() tests !cm->tiles.single_tile_decoding.

Bug: b:329485898
Bug: b:329810149
Bug: b:329813868
Bug: chromium:327882824
Bug: chromium:327698030
Bug: oss-fuzz:67057
Bug: oss-fuzz:67060
Change-Id: Iaedac01357c275ec7c3e89ad46ec1327be888d83
  • Loading branch information
wantehchang committed Mar 15, 2024
1 parent 74d1ea7 commit 158761d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion av1/common/alloccommon.c
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ void av1_alloc_cdef_buffers(AV1_COMMON *const cm,
const int is_num_workers_changed =
cdef_info->allocated_num_workers != num_workers;
const int is_cdef_enabled =
cm->seq_params->enable_cdef && !cm->tiles.large_scale;
cm->seq_params->enable_cdef && !cm->tiles.single_tile_decoding;

// num-bufs=3 represents ping-pong buffers for top linebuf,
// followed by bottom linebuf.
Expand Down

0 comments on commit 158761d

Please sign in to comment.