Re: [PATCH] bsg: update bsg_device_list to utilize static hash table implementation.

2017-12-01 Thread Tim Hansen
This patch was submitted as an effort to standardize on Sasha's hashtable implementation. Just a friendly ping to get some comments on this patch, been about 2 months with no comments on it at all. Are there any changes to be requested or issues raised with this change?

Re: [PATCH] bsg: update bsg_device_list to utilize static hash table implementation.

2017-10-15 Thread Tim Hansen
Mon, Oct 02, 2017 at 12:10:01PM -0400, Tim Hansen wrote: > Updates bsg_device_list to utilize static hashtable implementation. > > This is done in an effort to standarize the hashtables used across the > kernel. > > Signed-off-by: Tim Hansen <devtimhan...@gmail.com> &

Re: [PATCH] block/bio: Remove null checks before mempool_destroy in bioset_free

2017-10-06 Thread Tim Hansen
On Fri, Oct 06, 2017 at 01:05:01PM -0600, Jens Axboe wrote: > On 10/06/2017 12:45 PM, Tim Hansen wrote: > > This patch removes redundant checks for null values on bio_pool and > > bvec_pool. > > > > Found using make coccicheck M=block/ on linux-net tree on

Re: [PATCH] block: remove unnecessary NULL checks in bioset_integrity_free()

2017-10-06 Thread Tim Hansen
On Fri, Oct 06, 2017 at 01:04:25PM -0600, Jens Axboe wrote: > On 10/05/2017 12:09 PM, Tim Hansen wrote: > > mempool_destroy() already checks for a NULL value being passed in, this > > eliminates duplicate checks. > > > > This was caught by running make coccich

[PATCH] block/bio: Remove null checks before mempool_destroy in bioset_free

2017-10-06 Thread Tim Hansen
This patch removes redundant checks for null values on bio_pool and bvec_pool. Found using make coccicheck M=block/ on linux-net tree on the next-20170929 tag. Related to patch 9987695 that removed similar checks in bio-integrity. Signed-off-by: Tim Hansen <devtimhan...@gmail.com> ---

[PATCH] block: remove unnecessary NULL checks in bioset_integrity_free()

2017-10-05 Thread Tim Hansen
mempool_destroy() already checks for a NULL value being passed in, this eliminates duplicate checks. This was caught by running make coccicheck M=block/ on linus' tree on commit 77ede3a014a32746002f7889211f0cecf4803163 (current head as of this patch). Signed-off-by: Tim Hansen <devtim

[PATCH] bsg: update bsg_device_list to utilize static hash table implementation.

2017-10-02 Thread Tim Hansen
Updates bsg_device_list to utilize static hashtable implementation. This is done in an effort to standarize the hashtables used across the kernel. Signed-off-by: Tim Hansen <devtimhan...@gmail.com> --- block/bsg.c | 20 ++-- 1 file changed, 6 insertions(+), 14 deletions(-)