Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-11-03 Thread James Bottomley
On Tue, 2015-11-03 at 21:57 +0100, Rasmus Villemoes wrote: > On Tue, Nov 03 2015, James Bottomley wrote: > > > > > It was a suggestion when I explained what the missing sources of > > precision were, I don't think it's really a suggestion when it comes > > with an exemplary patch. > > ex·em·pla·

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-11-03 Thread Rasmus Villemoes
On Tue, Nov 03 2015, James Bottomley wrote: > > It was a suggestion when I explained what the missing sources of > precision were, I don't think it's really a suggestion when it comes > with an exemplary patch. ex·em·pla·ry adjective 1. serving as a desirable model; representing the bes

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-11-03 Thread James Bottomley
On Tue, 2015-11-03 at 14:13 +0100, Vitaly Kuznetsov wrote: > James Bottomley writes: > > > On Mon, 2015-11-02 at 16:58 +0100, Vitaly Kuznetsov wrote: > >> James Bottomley writes: > >> > >> > On Fri, 2015-10-30 at 11:46 +0100, Vitaly Kuznetsov wrote: > >> >> James Bottomley writes: > >> >> > >

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-11-03 Thread Vitaly Kuznetsov
James Bottomley writes: > On Mon, 2015-11-02 at 16:58 +0100, Vitaly Kuznetsov wrote: >> James Bottomley writes: >> >> > On Fri, 2015-10-30 at 11:46 +0100, Vitaly Kuznetsov wrote: >> >> James Bottomley writes: >> >> >> >> > On Thu, 2015-10-29 at 17:30 +0100, Vitaly Kuznetsov wrote: >> >> >> st

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-11-02 Thread James Bottomley
On Mon, 2015-11-02 at 16:58 +0100, Vitaly Kuznetsov wrote: > James Bottomley writes: > > > On Fri, 2015-10-30 at 11:46 +0100, Vitaly Kuznetsov wrote: > >> James Bottomley writes: > >> > >> > On Thu, 2015-10-29 at 17:30 +0100, Vitaly Kuznetsov wrote: > >> >> string_get_size() can't really handle

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-11-02 Thread Vitaly Kuznetsov
James Bottomley writes: > On Fri, 2015-10-30 at 11:46 +0100, Vitaly Kuznetsov wrote: >> James Bottomley writes: >> >> > On Thu, 2015-10-29 at 17:30 +0100, Vitaly Kuznetsov wrote: >> >> string_get_size() can't really handle huge block sizes, especially >> >> blk_size > U32_MAX but string_get_siz

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-10-30 Thread James Bottomley
On Fri, 2015-10-30 at 11:46 +0100, Vitaly Kuznetsov wrote: > James Bottomley writes: > > > On Thu, 2015-10-29 at 17:30 +0100, Vitaly Kuznetsov wrote: > >> string_get_size() can't really handle huge block sizes, especially > >> blk_size > U32_MAX but string_get_size() interface states the opposite

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-10-30 Thread Vitaly Kuznetsov
James Bottomley writes: > On Thu, 2015-10-29 at 17:30 +0100, Vitaly Kuznetsov wrote: >> string_get_size() can't really handle huge block sizes, especially >> blk_size > U32_MAX but string_get_size() interface states the opposite. >> Change blk_size from u64 to u32 to reflect the reality. > > What

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-10-29 Thread James Bottomley
On Fri, 2015-10-30 at 00:19 +0100, Rasmus Villemoes wrote: > On Thu, Oct 29 2015, James Bottomley wrote: > > > On Thu, 2015-10-29 at 17:30 +0100, Vitaly Kuznetsov wrote: > >> string_get_size() can't really handle huge block sizes, especially > >> blk_size > U32_MAX but string_get_size() interface

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-10-29 Thread Rasmus Villemoes
On Fri, Oct 30 2015, Rasmus Villemoes wrote: > block sizes as well. For ~13000 semirandom size,blk_size pairs, Sorry, that should have been ~2. Rasmus -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majord

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-10-29 Thread Rasmus Villemoes
On Thu, Oct 29 2015, James Bottomley wrote: > On Thu, 2015-10-29 at 17:30 +0100, Vitaly Kuznetsov wrote: >> string_get_size() can't really handle huge block sizes, especially >> blk_size > U32_MAX but string_get_size() interface states the opposite. >> Change blk_size from u64 to u32 to reflect t

Re: [PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-10-29 Thread James Bottomley
On Thu, 2015-10-29 at 17:30 +0100, Vitaly Kuznetsov wrote: > string_get_size() can't really handle huge block sizes, especially > blk_size > U32_MAX but string_get_size() interface states the opposite. > Change blk_size from u64 to u32 to reflect the reality. What is the actual evidence for this?

[PATCH v3 1/4] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-10-29 Thread Vitaly Kuznetsov
string_get_size() can't really handle huge block sizes, especially blk_size > U32_MAX but string_get_size() interface states the opposite. Change blk_size from u64 to u32 to reflect the reality. Signed-off-by: Vitaly Kuznetsov --- include/linux/string_helpers.h | 2 +- lib/string_helpers.c