diff for duplicates of <CAB=+i9SbDgMJSo91Wt4ZfkgWW3+8R49_7Uhn5FCbBXt-SX+osg@mail.gmail.com>
diff --git a/a/2.hdr b/N1/2.hdr
index a0dd15a..127ac4a 100644
--- a/a/2.hdr
+++ b/N1/2.hdr
@@ -1,2 +1,4 @@
-Content-Type: text/html; charset="UTF-8"
-Content-Transfer-Encoding: quoted-printable
+Content-Type: text/html
+MIME-Version: 1.0
+Content-Transfer-Encoding: base64
+Content-Disposition: attachment; filename="attachment.htm"
diff --git a/a/content_digest b/N1/content_digest
index 247da8c..92c06d0 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -1,45 +1,3 @@
-[
- "ref\000202106051442.G1VJubTz-lkp\@intel.com\0"
-]
-[
- "ref\00020210606110839.GA13828\@hyeyoo\0"
-]
-[
- "ref\0eb652efc-7695-ded7-350d-4373dad94460\@suse.cz\0"
-]
-[
- "ref\00020210607122550.GA752464\@hyeyoo\0"
-]
-[
- "ref\00006af75da-ffe9-7070-1da8-bcb2cb7881d2\@suse.cz\0"
-]
-[
- "ref\00020210607154957.GB927582\@hyeyoo\0"
-]
-[
- "ref\0006e1d48f2-409c-0a71-4d04-a907fe4183b8\@suse.cz\0"
-]
-[
- "ref\00020210608170528.GA28015\@hyeyoo\0"
-]
-[
- "ref\0002d2d792e-e189-99a4-36cb-f1473a4df9ad\@suse.cz\0"
-]
-[
- "ref\00020210608184501.GA5505\@hyeyoo\0"
-]
-[
- "ref\000513f82e6-175c-d040-691c-5d0e7dacfb83\@suse.cz\0"
-]
-[
- "ref\0CAB=+i9StdrGQWXXoQHKU5oLK3eKuNcuCAbrd88kPLzM_Yw==Jg\@mail.gmail.com\0"
-]
-[
- "ref\0d89798b1-ac4f-ab3e-27be-b1d40b8d7193\@suse.cz\0"
-]
-[
- "ref\0CAB=+i9Snmu7ML3Zqrbii1-jtS0BF_KeGEhn-R49Z2bh=uW-rGg\@mail.gmail.com\0"
-]
[
"ref\0ab90e117-6457-bf01-fdce-9682e972ed27\@suse.cz\0"
]
@@ -53,15 +11,7 @@
"Date\0Sat, 12 Jun 2021 09:19:05 +0900\0"
]
[
- "To\0Vlastimil Babka <vbabka\@suse.cz>\0"
-]
-[
- "Cc\0kernel test robot <lkp\@intel.com>",
- " kbuild-all\@lists.01.org",
- " Linux Memory Management List <linux-mm\@kvack.org>",
- " Andrew Morton <akpm\@linux-foundation.org>",
- " Nathan Chancellor <nathan\@kernel.org>",
- " Nick Desaulniers <ndesaulniers\@google.com>\0"
+ "To\0kbuild-all\@lists.01.org\0"
]
[
"\0001:1\0"
@@ -132,48 +82,51 @@
[
"\0001:2\0"
]
+[
+ "fn\0attachment.htm\0"
+]
[
"b\0"
]
[
- "<div dir=\"auto\"><div><div class=\"gmail_quote\"><div dir=\"ltr\" class=\"gmail_attr\">On Sat, Jun 12, 2021, 8:59 AM Vlastimil Babka <<a href=\"mailto:vbabka\@suse.cz\">vbabka\@suse.cz</a>> wrote:<br></div><blockquote class=\"gmail_quote\" style=\"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex\">On 6/11/21 1:56 PM, Hyeonggon Yoo wrote:<br>\r\n",
- "> On Fri, Jun 11, 2021, 7:27 PM Vlastimil Babka <<a href=\"mailto:vbabka\@suse.cz\" target=\"_blank\" rel=\"noreferrer\">vbabka\@suse.cz</a><br>\r\n",
- "> <mailto:<a href=\"mailto:vbabka\@suse.cz\" target=\"_blank\" rel=\"noreferrer\">vbabka\@suse.cz</a>>> wrote:<br>\r\n",
- ">\302\240 \302\240 \302\240I meant the the condition to use BUILD_BUG_ON instead of BUG_ON would include<br>\r\n",
- ">\302\240 \302\240 \302\240!IS_ENABLED(CONFIG_PROFILE_ALL_BRANCHES)<br>\r\n",
- "> <br>\r\n",
- "> <br>\r\n",
- "> You mean this? (This will make kmalloc_index return -1 without BUG())<br>\r\n",
- "> <br>\r\n",
- "> if ((IS_ENABLED(CONFIG_CC_IS_GCC) || CONFIG_CLANG_VERSION >= 110000) &&<br>\r\n",
- "> size_is_constant)<br>\r\n",
- "> \302\240 \302\240 \302\240 \302\240 BUILD_BUG_ON_MSG(!IS_ENABLED(CONFIG_PROFILE_ALL_BRANCHES), "unexpected<br>\r\n",
- "> size in kmalloc_index()");<br>\r\n",
- "> else<br>\r\n",
- "> \302\240 \302\240 \302\240 \302\240 BUG();<br>\r\n",
- "<br>\r\n",
- "No,<br>\r\n",
- "<br>\r\n",
- "> Or This?<br>\r\n",
- "> <br>\r\n",
- "> if ((IS_ENABLED(CONFIG_CC_IS_GCC) || CONFIG_CLANG_VERSION >= 110000) &&<br>\r\n",
- "> size_is_constant && !IS_ENABLED(CONFIG_PROFILE_ALL_BRANCHES))<br>\r\n",
- "> \302\240 \302\240 \302\240 \302\240 BUILD_BUG_ON_MSG(1, "unexpected size in kmalloc_index()");<br>\r\n",
- "> else<br>\r\n",
- "> \302\240 \302\240 \302\240 \302\240 BUG();<br>\r\n",
- "> <br>\r\n",
- "> Maybe this version seems better<br>\r\n",
- "<br>\r\n",
- "Yeah, meant that.<br></blockquote></div></div><div dir=\"auto\"><br></div><div dir=\"auto\">Ah, okay!<br></div><div dir=\"auto\"><br></div><div dir=\"auto\"><div class=\"gmail_quote\"><blockquote class=\"gmail_quote\" style=\"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex\">\r\n",
- "<br>\r\n",
- "> But little bit worried :(<br>\r\n",
- "> The code is getting too complicated...<br>\r\n",
- "> How do you think?<br>\r\n",
- "<br>\r\n",
- "Yeah, I expected that problems like this could occur as we're poking at some<br>\r\n",
- "rare corner cases of compiler implementations here. But if that leads to fixes<br>\r\n",
- "in compilers, good for everyone I'd say.</blockquote></div></div><div dir=\"auto\"><div class=\"gmail_quote\"><blockquote class=\"gmail_quote\" style=\"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex\">So I would try this, even if it becomes complicated.<br></blockquote></div></div><div dir=\"auto\"><br></div><div dir=\"auto\">Okay, I see the code is okay, but one thing to suggest:</div><div dir=\"auto\">\302\240 \302\240 The problem already existed in kmalloc_node before the patch And we found it by adding BUILD_BUG_ON in kmalloc_index.</div><div dir=\"auto\"><br></div><div dir=\"auto\">\302\240 \302\240 So I suggest adding the condition in kmalloc_node. How about this?</div><div dir=\"auto\"><div class=\"gmail_quote\"><blockquote class=\"gmail_quote\" style=\"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex\">\r\n",
- "</blockquote></div></div></div>\r\n"
+ "<div dir=\"auto\"><div><div class=\"gmail_quote\"><div dir=\"ltr\" class=\"gmail_attr\">On Sat, Jun 12, 2021, 8:59 AM Vlastimil Babka <<a href=\"mailto:vbabka\@suse.cz\">vbabka\@suse.cz</a>> wrote:<br></div><blockquote class=\"gmail_quote\" style=\"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex\">On 6/11/21 1:56 PM, Hyeonggon Yoo wrote:<br>\n",
+ "> On Fri, Jun 11, 2021, 7:27 PM Vlastimil Babka <<a href=\"mailto:vbabka\@suse.cz\" target=\"_blank\" rel=\"noreferrer\">vbabka\@suse.cz</a><br>\n",
+ "> <mailto:<a href=\"mailto:vbabka\@suse.cz\" target=\"_blank\" rel=\"noreferrer\">vbabka\@suse.cz</a>>> wrote:<br>\n",
+ ">\302\240 \302\240 \302\240I meant the the condition to use BUILD_BUG_ON instead of BUG_ON would include<br>\n",
+ ">\302\240 \302\240 \302\240!IS_ENABLED(CONFIG_PROFILE_ALL_BRANCHES)<br>\n",
+ "> <br>\n",
+ "> <br>\n",
+ "> You mean this? (This will make kmalloc_index return -1 without BUG())<br>\n",
+ "> <br>\n",
+ "> if ((IS_ENABLED(CONFIG_CC_IS_GCC) || CONFIG_CLANG_VERSION >= 110000) &&<br>\n",
+ "> size_is_constant)<br>\n",
+ "> \302\240 \302\240 \302\240 \302\240 BUILD_BUG_ON_MSG(!IS_ENABLED(CONFIG_PROFILE_ALL_BRANCHES), "unexpected<br>\n",
+ "> size in kmalloc_index()");<br>\n",
+ "> else<br>\n",
+ "> \302\240 \302\240 \302\240 \302\240 BUG();<br>\n",
+ "<br>\n",
+ "No,<br>\n",
+ "<br>\n",
+ "> Or This?<br>\n",
+ "> <br>\n",
+ "> if ((IS_ENABLED(CONFIG_CC_IS_GCC) || CONFIG_CLANG_VERSION >= 110000) &&<br>\n",
+ "> size_is_constant && !IS_ENABLED(CONFIG_PROFILE_ALL_BRANCHES))<br>\n",
+ "> \302\240 \302\240 \302\240 \302\240 BUILD_BUG_ON_MSG(1, "unexpected size in kmalloc_index()");<br>\n",
+ "> else<br>\n",
+ "> \302\240 \302\240 \302\240 \302\240 BUG();<br>\n",
+ "> <br>\n",
+ "> Maybe this version seems better<br>\n",
+ "<br>\n",
+ "Yeah, meant that.<br></blockquote></div></div><div dir=\"auto\"><br></div><div dir=\"auto\">Ah, okay!<br></div><div dir=\"auto\"><br></div><div dir=\"auto\"><div class=\"gmail_quote\"><blockquote class=\"gmail_quote\" style=\"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex\">\n",
+ "<br>\n",
+ "> But little bit worried :(<br>\n",
+ "> The code is getting too complicated...<br>\n",
+ "> How do you think?<br>\n",
+ "<br>\n",
+ "Yeah, I expected that problems like this could occur as we're poking at some<br>\n",
+ "rare corner cases of compiler implementations here. But if that leads to fixes<br>\n",
+ "in compilers, good for everyone I'd say.</blockquote></div></div><div dir=\"auto\"><div class=\"gmail_quote\"><blockquote class=\"gmail_quote\" style=\"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex\">So I would try this, even if it becomes complicated.<br></blockquote></div></div><div dir=\"auto\"><br></div><div dir=\"auto\">Okay, I see the code is okay, but one thing to suggest:</div><div dir=\"auto\">\302\240 \302\240 The problem already existed in kmalloc_node before the patch And we found it by adding BUILD_BUG_ON in kmalloc_index.</div><div dir=\"auto\"><br></div><div dir=\"auto\">\302\240 \302\240 So I suggest adding the condition in kmalloc_node. How about this?</div><div dir=\"auto\"><div class=\"gmail_quote\"><blockquote class=\"gmail_quote\" style=\"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex\">\n",
+ "</blockquote></div></div></div>\n"
]
-017652501e91aa0a77b436cb19204ca145cf535399bc09311da114a1c066e648
+583a32fd77f9c749f50bd34dcea97af849c8d5b6acaff749d21edc88ba20cdf1
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.