[PATCH] erofs-utils: generate preallocated extents for tarerofs

[Jingbo Xu]

On 7/29/23 9:32 PM, Gao Xiang wrote:
> -int erofs_blob_remap(struct erofs_sb_info *sbi)
> +int tarerofs_write_chunk_data(struct erofs_inode *inode, erofs_off_t data_offset)
> +{
> +	struct erofs_sb_info *sbi = inode->sbi;
> +	unsigned int chunkbits = ilog2(inode->i_size - 1) + 1;
> +	unsigned int count, unit, device_id;
> +	erofs_off_t chunksize, len, pos;
> +	erofs_blk_t blkaddr;
> +	struct erofs_inode_chunk_index *idx;
> +
> +	if (chunkbits < sbi->blkszbits)
> +		chunkbits = sbi->blkszbits;
> +	if (chunkbits - sbi->blkszbits > EROFS_CHUNK_FORMAT_BLKBITS_MASK)
> +		chunkbits = EROFS_CHUNK_FORMAT_BLKBITS_MASK + sbi->blkszbits;
> +
> +	inode->u.chunkformat |= chunkbits - sbi->blkszbits;
> +	if (sbi->extra_devices) {
> +		device_id = 1;
> +		inode->u.chunkformat |= EROFS_CHUNK_FORMAT_INDEXES;
> +		unit = sizeof(struct erofs_inode_chunk_index);
> +		DBG_BUGON(erofs_blkoff(sbi, data_offset));
> +		blkaddr = erofs_blknr(sbi, data_offset);
> +	} else {
> +		device_id = 0;
> +		unit = EROFS_BLOCK_MAP_ENTRY_SIZE;
> +		DBG_BUGON(erofs_blkoff(sbi, datablob_size));
> +		blkaddr = erofs_blknr(sbi, datablob_size);
> +		datablob_size += round_up(inode->i_size, erofs_blksiz(sbi));
> +	}
> +	chunksize = 1ULL << chunkbits;
> +	count = DIV_ROUND_UP(inode->i_size, chunksize);
> +
> +	inode->extent_isize = count * unit;
> +	idx = calloc(count, max(sizeof(*idx), sizeof(void *)));
> +	if (!idx)
> +		return -ENOMEM;
> +	inode->chunkindexes = idx;
> +
> +	for (pos = 0; pos < inode->i_size; pos += len) {
> +		struct erofs_blobchunk *chunk;
> +
> +		len = min_t(erofs_off_t, inode->i_size - pos, chunksize);
> +
> +		chunk = erofs_get_unhashed_chunk(device_id, blkaddr,
> +						 data_offset);
> +		if (IS_ERR(chunk))
> +			return PTR_ERR(chunk);

inode->chunkindexes needs to be freed on error path.

> +
> +		*(void **)idx++ = chunk;
> +		blkaddr += erofs_blknr(sbi, len);
> +		data_offset += len;
> +	}
> +	inode->datalayout = EROFS_INODE_CHUNK_BASED;
> +	return 0;
> +}
> +


-- 
Thanks,
Jingbo