Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added callback function in uzfs_get_txg_diff_tree #11

Merged

Conversation

mynktl
Copy link
Member

@mynktl mynktl commented Mar 12, 2018

Changes in uzfs_get_txg_diff_tree API
This API will traverse zvol and execute callback function provided by caller with offset/len/blockid.

Signed-off-by: mayank <[email protected]>
@mynktl mynktl force-pushed the modified_block_tree_changes branch from a92afd6 to ab043fe Compare March 12, 2018 10:57
int
uzfs_get_txg_diff_tree(zvol_state_t *zv, uint64_t start_txg, uint64_t end_txg,
avl_tree_t **tree)
uzfs_zvol_traverse_t *func, void *arg)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uzfs_txg_diff_traverse_cb_t

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@@ -22,10 +22,30 @@
#ifndef _UZFS_MTREE_H
#define _UZFS_MTREE_H

/*
* API to get modified blocks between start_txg and end_txg
* Note: API will alocate condensed tree and populate it witch

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update the comments

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

int
uzfs_get_txg_diff_tree(zvol_state_t *zv, uint64_t start_txg, uint64_t end_txg,
avl_tree_t **tree)
uzfs_zvol_traverse_t *func, void *arg)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should remove "tree" from function name for better clarity.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@vishnuitta vishnuitta merged commit a55ebd7 into mayadata-io:zfs-0.7-release Mar 13, 2018
jkryl pushed a commit that referenced this pull request Jun 26, 2018
…11)

* [CCS 59] uZFS API to call dmu from istgt

* [CCS-59] fix in travis and header for pkg-utils

* Adding other header files for make pkg build

* [CCS 59] added few header files to build pkg mod

* [CCS 59]cstyle fixes, adding cstyle check to travis

* Remove unnecessary header files,added TOTAL_TIME env

* Incorporating review comments [CCS 59]

* cstyle fix, removed getenv in uzfs_test

* Merged libuzfs_ioctl into zfs_ioctl

* cstyle fix, removed cstyle from travis

* compilation issue with non-uzfs and userspace build

Once zvol is created using 'zfs create -V' command, it can be opened by doing 'uzfs_open_dataset()', and use uzfs_write_data() and uzfs_read_data() APIs for doing IOs.
jkryl pushed a commit that referenced this pull request Jun 26, 2018
added callback function in uzfs_get_txg_diff
Signed-off-by: mayank <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants