diff for duplicates of <1544784449.3270.30.camel@synopsys.com> diff --git a/a/1.txt b/N1/1.txt index 05433f3..b82de98 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,6 @@ Hi Vineet, -On Thu, 2018-12-13@18:23 +0000, Vineet Gupta wrote: +On Thu, 2018-12-13 at 18:23 +0000, Vineet Gupta wrote: > On 12/13/18 7:43 AM, Eugeniy Paltsev wrote: > > Change __ffs return value from 'int' to 'unsigned long' as it > > is done in other implementations (like asm-generic, x86, etc...) @@ -10,7 +10,7 @@ On Thu, 2018-12-13@18:23 +0000, Vineet Gupta wrote: > > As __ffs may return values in [0-31] interval changing return > > type to unsigned is valid. > > -> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev at synopsys.com> +> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> > > Can you please paste a couple of lines of offending output ! @@ -23,20 +23,20 @@ In file included from ./include/asm-generic/bug.h:18:0, from ./include/linux/gfp.h:5, from ./include/linux/slab.h:15, from mm/nobootmem.c:14: -mm/nobootmem.c: In function ?__free_pages_memory?: +mm/nobootmem.c: In function ‘__free_pages_memory’: ./include/linux/kernel.h:845:29: warning: comparison of distinct pointer types lacks a cast (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1))) ^ -./include/linux/kernel.h:859:4: note: in expansion of macro ?__typecheck? +./include/linux/kernel.h:859:4: note: in expansion of macro ‘__typecheck’ (__typecheck(x, y) && __no_side_effects(x, y)) ^~~~~~~~~~~ -./include/linux/kernel.h:869:24: note: in expansion of macro ?__safe_cmp? +./include/linux/kernel.h:869:24: note: in expansion of macro ‘__safe_cmp’ __builtin_choose_expr(__safe_cmp(x, y), \ ^~~~~~~~~~ -./include/linux/kernel.h:878:19: note: in expansion of macro ?__careful_cmp? +./include/linux/kernel.h:878:19: note: in expansion of macro ‘__careful_cmp’ #define min(x, y) __careful_cmp(x, y, <) ^~~~~~~~~~~~~ -mm/nobootmem.c:104:11: note: in expansion of macro ?min? +mm/nobootmem.c:104:11: note: in expansion of macro ‘min’ order = min(MAX_ORDER - 1UL, __ffs(start)); ^~~ ---------------------------------->8--------------------------------- diff --git a/a/content_digest b/N1/content_digest index db95258..e96a8d4 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,14 +1,18 @@ "ref\020181213154257.23033-1-Eugeniy.Paltsev@synopsys.com\0" "ref\0C2D7FE5348E1B147BCA15975FBA23075014641F6E3@US01WEMBX2.internal.synopsys.com\0" - "From\0eugeniy.paltsev@synopsys.com (Eugeniy Paltsev)\0" - "Subject\0[PATCH] ARC: fix __ffs return value to avoid build warnings\0" + "From\0Eugeniy Paltsev <eugeniy.paltsev@synopsys.com>\0" + "Subject\0Re: [PATCH] ARC: fix __ffs return value to avoid build warnings\0" "Date\0Fri, 14 Dec 2018 10:47:30 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0Eugeniy.Paltsev@synopsys.com <Eugeniy.Paltsev@synopsys.com>" + Vineet Gupta <vineet.gupta1@synopsys.com> + " linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org>\0" + "Cc\0linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>" + " Alexey Brodkin <alexey.brodkin@synopsys.com>\0" "\00:1\0" "b\0" "Hi Vineet,\n" "\n" - "On Thu, 2018-12-13@18:23 +0000, Vineet Gupta wrote:\n" + "On Thu, 2018-12-13 at 18:23 +0000, Vineet Gupta wrote:\n" "> On 12/13/18 7:43 AM, Eugeniy Paltsev wrote:\n" "> > Change __ffs return value from 'int' to 'unsigned long' as it\n" "> > is done in other implementations (like asm-generic, x86, etc...)\n" @@ -18,7 +22,7 @@ "> > As __ffs may return values in [0-31] interval changing return\n" "> > type to unsigned is valid.\n" "> > \n" - "> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev at synopsys.com>\n" + "> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>\n" "> \n" "> Can you please paste a couple of lines of offending output !\n" "\n" @@ -31,20 +35,20 @@ " from ./include/linux/gfp.h:5,\n" " from ./include/linux/slab.h:15,\n" " from mm/nobootmem.c:14:\n" - "mm/nobootmem.c: In function ?__free_pages_memory?:\n" + "mm/nobootmem.c: In function \342\200\230__free_pages_memory\342\200\231:\n" "./include/linux/kernel.h:845:29: warning: comparison of distinct pointer types lacks a cast\n" " (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))\n" " ^\n" - "./include/linux/kernel.h:859:4: note: in expansion of macro ?__typecheck?\n" + "./include/linux/kernel.h:859:4: note: in expansion of macro \342\200\230__typecheck\342\200\231\n" " (__typecheck(x, y) && __no_side_effects(x, y))\n" " ^~~~~~~~~~~\n" - "./include/linux/kernel.h:869:24: note: in expansion of macro ?__safe_cmp?\n" + "./include/linux/kernel.h:869:24: note: in expansion of macro \342\200\230__safe_cmp\342\200\231\n" " __builtin_choose_expr(__safe_cmp(x, y), \\\n" " ^~~~~~~~~~\n" - "./include/linux/kernel.h:878:19: note: in expansion of macro ?__careful_cmp?\n" + "./include/linux/kernel.h:878:19: note: in expansion of macro \342\200\230__careful_cmp\342\200\231\n" " #define min(x, y) __careful_cmp(x, y, <)\n" " ^~~~~~~~~~~~~\n" - "mm/nobootmem.c:104:11: note: in expansion of macro ?min?\n" + "mm/nobootmem.c:104:11: note: in expansion of macro \342\200\230min\342\200\231\n" " order = min(MAX_ORDER - 1UL, __ffs(start));\n" " ^~~\n" "---------------------------------->8---------------------------------\n" @@ -54,4 +58,4 @@ "-- \n" Eugeniy Paltsev -9eeac667e2b7310b7f67002d9ae124b7b355977c316d5921a7b5ffe56534d5d4 +f0f356afecd50a0af588a68e49bbf0d131c4e370fa35611f8cdc440d72158faa
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.