diff for duplicates of <DM5PR12MB1916E752AF9164228359BC9EF8F50@DM5PR12MB1916.namprd12.prod.outlook.com>
diff --git a/a/1.txt b/N1/1.txt
index 09553e7..53771b8 100644
--- a/a/1.txt
+++ b/N1/1.txt
@@ -1,40 +1,31 @@
-> -----Original Message-----
-> From: Borislav Petkov [mailto:bp@alien8.de]
-> Sent: Thursday, February 8, 2018 10:05 AM
-> To: Ghannam, Yazen <Yazen.Ghannam@amd.com>
-> Cc: linux-edac@vger.kernel.org; linux-kernel@vger.kernel.org; bp@suse.de;
-> tony.luck@intel.com; x86@kernel.org
-> Subject: Re: [PATCH 1/3] x86/MCE/AMD: Redo function to get SMCA bank type
->
-> On Thu, Feb 01, 2018 at 12:48:11PM -0600, Yazen Ghannam wrote:
-> > arch/x86/kernel/cpu/mcheck/mce_amd.c | 15 +++++++--------
-> > 1 file changed, 7 insertions(+), 8 deletions(-)
-> >
-> > diff --git a/arch/x86/kernel/cpu/mcheck/mce_amd.c
-> b/arch/x86/kernel/cpu/mcheck/mce_amd.c
-> > index 0f32ad242324..4e16afc0794d 100644
-> > --- a/arch/x86/kernel/cpu/mcheck/mce_amd.c
-> > +++ b/arch/x86/kernel/cpu/mcheck/mce_amd.c
-> > @@ -110,14 +110,14 @@ const char *smca_get_long_name(enum
-> smca_bank_types t)
-> > }
-> > EXPORT_SYMBOL_GPL(smca_get_long_name);
-> >
-> > -static enum smca_bank_types smca_get_bank_type(struct mce *m)
-> > +static enum smca_bank_types smca_get_bank_type(unsigned int bank)
-> > {
-> > struct smca_bank *b;
-> >
-> > - if (m->bank >= N_SMCA_BANK_TYPES)
-> > + if (bank >= ARRAY_SIZE(smca_banks))
->
-> MAX_NR_BANKS
->
-
-I know that we're declaring smca_banks[] to have MAX_NR_BANKS items. But
-shouldn't we directly check that an index is within the bounds of the array?
-We'll have a bug if we check against MAX_NR_BANKS and the definition of
-smca_banks[] changes.
-
-Thanks,
-Yazen
\ No newline at end of file
+PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBCb3Jpc2xhdiBQZXRrb3YgW21h
+aWx0bzpicEBhbGllbjguZGVdDQo+IFNlbnQ6IFRodXJzZGF5LCBGZWJydWFyeSA4LCAyMDE4IDEw
+OjA1IEFNDQo+IFRvOiBHaGFubmFtLCBZYXplbiA8WWF6ZW4uR2hhbm5hbUBhbWQuY29tPg0KPiBD
+YzogbGludXgtZWRhY0B2Z2VyLmtlcm5lbC5vcmc7IGxpbnV4LWtlcm5lbEB2Z2VyLmtlcm5lbC5v
+cmc7IGJwQHN1c2UuZGU7DQo+IHRvbnkubHVja0BpbnRlbC5jb207IHg4NkBrZXJuZWwub3JnDQo+
+IFN1YmplY3Q6IFJlOiBbUEFUQ0ggMS8zXSB4ODYvTUNFL0FNRDogUmVkbyBmdW5jdGlvbiB0byBn
+ZXQgU01DQSBiYW5rIHR5cGUNCj4gDQo+IE9uIFRodSwgRmViIDAxLCAyMDE4IGF0IDEyOjQ4OjEx
+UE0gLTA2MDAsIFlhemVuIEdoYW5uYW0gd3JvdGU6DQo+ID4gIGFyY2gveDg2L2tlcm5lbC9jcHUv
+bWNoZWNrL21jZV9hbWQuYyB8IDE1ICsrKysrKystLS0tLS0tLQ0KPiA+ICAxIGZpbGUgY2hhbmdl
+ZCwgNyBpbnNlcnRpb25zKCspLCA4IGRlbGV0aW9ucygtKQ0KPiA+DQo+ID4gZGlmZiAtLWdpdCBh
+L2FyY2gveDg2L2tlcm5lbC9jcHUvbWNoZWNrL21jZV9hbWQuYw0KPiBiL2FyY2gveDg2L2tlcm5l
+bC9jcHUvbWNoZWNrL21jZV9hbWQuYw0KPiA+IGluZGV4IDBmMzJhZDI0MjMyNC4uNGUxNmFmYzA3
+OTRkIDEwMDY0NA0KPiA+IC0tLSBhL2FyY2gveDg2L2tlcm5lbC9jcHUvbWNoZWNrL21jZV9hbWQu
+Yw0KPiA+ICsrKyBiL2FyY2gveDg2L2tlcm5lbC9jcHUvbWNoZWNrL21jZV9hbWQuYw0KPiA+IEBA
+IC0xMTAsMTQgKzExMCwxNCBAQCBjb25zdCBjaGFyICpzbWNhX2dldF9sb25nX25hbWUoZW51bQ0K
+PiBzbWNhX2JhbmtfdHlwZXMgdCkNCj4gPiAgfQ0KPiA+ICBFWFBPUlRfU1lNQk9MX0dQTChzbWNh
+X2dldF9sb25nX25hbWUpOw0KPiA+DQo+ID4gLXN0YXRpYyBlbnVtIHNtY2FfYmFua190eXBlcyBz
+bWNhX2dldF9iYW5rX3R5cGUoc3RydWN0IG1jZSAqbSkNCj4gPiArc3RhdGljIGVudW0gc21jYV9i
+YW5rX3R5cGVzIHNtY2FfZ2V0X2JhbmtfdHlwZSh1bnNpZ25lZCBpbnQgYmFuaykNCj4gPiAgew0K
+PiA+ICAJc3RydWN0IHNtY2FfYmFuayAqYjsNCj4gPg0KPiA+IC0JaWYgKG0tPmJhbmsgPj0gTl9T
+TUNBX0JBTktfVFlQRVMpDQo+ID4gKwlpZiAoYmFuayA+PSBBUlJBWV9TSVpFKHNtY2FfYmFua3Mp
+KQ0KPiANCj4gTUFYX05SX0JBTktTDQo+IA0KDQpJIGtub3cgdGhhdCB3ZSdyZSBkZWNsYXJpbmcg
+c21jYV9iYW5rc1tdIHRvIGhhdmUgTUFYX05SX0JBTktTIGl0ZW1zLiBCdXQNCnNob3VsZG4ndCB3
+ZSBkaXJlY3RseSBjaGVjayB0aGF0IGFuIGluZGV4IGlzIHdpdGhpbiB0aGUgYm91bmRzIG9mIHRo
+ZSBhcnJheT8NCldlJ2xsIGhhdmUgYSBidWcgaWYgd2UgY2hlY2sgYWdhaW5zdCBNQVhfTlJfQkFO
+S1MgYW5kIHRoZSBkZWZpbml0aW9uIG9mDQpzbWNhX2JhbmtzW10gY2hhbmdlcy4NCg0KVGhhbmtz
+LA0KWWF6ZW4NCg0K
+---
+To unsubscribe from this list: send the line "unsubscribe linux-edac" in
+the body of a message to majordomo@vger.kernel.org
+More majordomo info at http://vger.kernel.org/majordomo-info.html
\ No newline at end of file
diff --git a/a/content_digest b/N1/content_digest
index 98cbb77..f939984 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -1,14 +1,8 @@
[
- "ref\00020180201184813.82253-1-Yazen.Ghannam\@amd.com\0"
+ "From\0Yazen Ghannam <yazen.ghannam\@amd.com>\0"
]
[
- "ref\00020180208150433.GD7964\@pd.tnic\0"
-]
-[
- "From\0Ghannam, Yazen <Yazen.Ghannam\@amd.com>\0"
-]
-[
- "Subject\0RE: [PATCH 1/3] x86/MCE/AMD: Redo function to get SMCA bank type\0"
+ "Subject\0[1/3] x86/MCE/AMD: Redo function to get SMCA bank type\0"
]
[
"Date\0Wed, 14 Feb 2018 16:38:34 +0000\0"
@@ -30,46 +24,37 @@
"b\0"
]
[
- "> -----Original Message-----\n",
- "> From: Borislav Petkov [mailto:bp\@alien8.de]\n",
- "> Sent: Thursday, February 8, 2018 10:05 AM\n",
- "> To: Ghannam, Yazen <Yazen.Ghannam\@amd.com>\n",
- "> Cc: linux-edac\@vger.kernel.org; linux-kernel\@vger.kernel.org; bp\@suse.de;\n",
- "> tony.luck\@intel.com; x86\@kernel.org\n",
- "> Subject: Re: [PATCH 1/3] x86/MCE/AMD: Redo function to get SMCA bank type\n",
- "> \n",
- "> On Thu, Feb 01, 2018 at 12:48:11PM -0600, Yazen Ghannam wrote:\n",
- "> > arch/x86/kernel/cpu/mcheck/mce_amd.c | 15 +++++++--------\n",
- "> > 1 file changed, 7 insertions(+), 8 deletions(-)\n",
- "> >\n",
- "> > diff --git a/arch/x86/kernel/cpu/mcheck/mce_amd.c\n",
- "> b/arch/x86/kernel/cpu/mcheck/mce_amd.c\n",
- "> > index 0f32ad242324..4e16afc0794d 100644\n",
- "> > --- a/arch/x86/kernel/cpu/mcheck/mce_amd.c\n",
- "> > +++ b/arch/x86/kernel/cpu/mcheck/mce_amd.c\n",
- "> > \@\@ -110,14 +110,14 \@\@ const char *smca_get_long_name(enum\n",
- "> smca_bank_types t)\n",
- "> > }\n",
- "> > EXPORT_SYMBOL_GPL(smca_get_long_name);\n",
- "> >\n",
- "> > -static enum smca_bank_types smca_get_bank_type(struct mce *m)\n",
- "> > +static enum smca_bank_types smca_get_bank_type(unsigned int bank)\n",
- "> > {\n",
- "> > \tstruct smca_bank *b;\n",
- "> >\n",
- "> > -\tif (m->bank >= N_SMCA_BANK_TYPES)\n",
- "> > +\tif (bank >= ARRAY_SIZE(smca_banks))\n",
- "> \n",
- "> MAX_NR_BANKS\n",
- "> \n",
- "\n",
- "I know that we're declaring smca_banks[] to have MAX_NR_BANKS items. But\n",
- "shouldn't we directly check that an index is within the bounds of the array?\n",
- "We'll have a bug if we check against MAX_NR_BANKS and the definition of\n",
- "smca_banks[] changes.\n",
- "\n",
- "Thanks,\n",
- "Yazen"
+ "PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBCb3Jpc2xhdiBQZXRrb3YgW21h\n",
+ "aWx0bzpicEBhbGllbjguZGVdDQo+IFNlbnQ6IFRodXJzZGF5LCBGZWJydWFyeSA4LCAyMDE4IDEw\n",
+ "OjA1IEFNDQo+IFRvOiBHaGFubmFtLCBZYXplbiA8WWF6ZW4uR2hhbm5hbUBhbWQuY29tPg0KPiBD\n",
+ "YzogbGludXgtZWRhY0B2Z2VyLmtlcm5lbC5vcmc7IGxpbnV4LWtlcm5lbEB2Z2VyLmtlcm5lbC5v\n",
+ "cmc7IGJwQHN1c2UuZGU7DQo+IHRvbnkubHVja0BpbnRlbC5jb207IHg4NkBrZXJuZWwub3JnDQo+\n",
+ "IFN1YmplY3Q6IFJlOiBbUEFUQ0ggMS8zXSB4ODYvTUNFL0FNRDogUmVkbyBmdW5jdGlvbiB0byBn\n",
+ "ZXQgU01DQSBiYW5rIHR5cGUNCj4gDQo+IE9uIFRodSwgRmViIDAxLCAyMDE4IGF0IDEyOjQ4OjEx\n",
+ "UE0gLTA2MDAsIFlhemVuIEdoYW5uYW0gd3JvdGU6DQo+ID4gIGFyY2gveDg2L2tlcm5lbC9jcHUv\n",
+ "bWNoZWNrL21jZV9hbWQuYyB8IDE1ICsrKysrKystLS0tLS0tLQ0KPiA+ICAxIGZpbGUgY2hhbmdl\n",
+ "ZCwgNyBpbnNlcnRpb25zKCspLCA4IGRlbGV0aW9ucygtKQ0KPiA+DQo+ID4gZGlmZiAtLWdpdCBh\n",
+ "L2FyY2gveDg2L2tlcm5lbC9jcHUvbWNoZWNrL21jZV9hbWQuYw0KPiBiL2FyY2gveDg2L2tlcm5l\n",
+ "bC9jcHUvbWNoZWNrL21jZV9hbWQuYw0KPiA+IGluZGV4IDBmMzJhZDI0MjMyNC4uNGUxNmFmYzA3\n",
+ "OTRkIDEwMDY0NA0KPiA+IC0tLSBhL2FyY2gveDg2L2tlcm5lbC9jcHUvbWNoZWNrL21jZV9hbWQu\n",
+ "Yw0KPiA+ICsrKyBiL2FyY2gveDg2L2tlcm5lbC9jcHUvbWNoZWNrL21jZV9hbWQuYw0KPiA+IEBA\n",
+ "IC0xMTAsMTQgKzExMCwxNCBAQCBjb25zdCBjaGFyICpzbWNhX2dldF9sb25nX25hbWUoZW51bQ0K\n",
+ "PiBzbWNhX2JhbmtfdHlwZXMgdCkNCj4gPiAgfQ0KPiA+ICBFWFBPUlRfU1lNQk9MX0dQTChzbWNh\n",
+ "X2dldF9sb25nX25hbWUpOw0KPiA+DQo+ID4gLXN0YXRpYyBlbnVtIHNtY2FfYmFua190eXBlcyBz\n",
+ "bWNhX2dldF9iYW5rX3R5cGUoc3RydWN0IG1jZSAqbSkNCj4gPiArc3RhdGljIGVudW0gc21jYV9i\n",
+ "YW5rX3R5cGVzIHNtY2FfZ2V0X2JhbmtfdHlwZSh1bnNpZ25lZCBpbnQgYmFuaykNCj4gPiAgew0K\n",
+ "PiA+ICAJc3RydWN0IHNtY2FfYmFuayAqYjsNCj4gPg0KPiA+IC0JaWYgKG0tPmJhbmsgPj0gTl9T\n",
+ "TUNBX0JBTktfVFlQRVMpDQo+ID4gKwlpZiAoYmFuayA+PSBBUlJBWV9TSVpFKHNtY2FfYmFua3Mp\n",
+ "KQ0KPiANCj4gTUFYX05SX0JBTktTDQo+IA0KDQpJIGtub3cgdGhhdCB3ZSdyZSBkZWNsYXJpbmcg\n",
+ "c21jYV9iYW5rc1tdIHRvIGhhdmUgTUFYX05SX0JBTktTIGl0ZW1zLiBCdXQNCnNob3VsZG4ndCB3\n",
+ "ZSBkaXJlY3RseSBjaGVjayB0aGF0IGFuIGluZGV4IGlzIHdpdGhpbiB0aGUgYm91bmRzIG9mIHRo\n",
+ "ZSBhcnJheT8NCldlJ2xsIGhhdmUgYSBidWcgaWYgd2UgY2hlY2sgYWdhaW5zdCBNQVhfTlJfQkFO\n",
+ "S1MgYW5kIHRoZSBkZWZpbml0aW9uIG9mDQpzbWNhX2JhbmtzW10gY2hhbmdlcy4NCg0KVGhhbmtz\n",
+ "LA0KWWF6ZW4NCg0K\n",
+ "---\n",
+ "To unsubscribe from this list: send the line \"unsubscribe linux-edac\" in\n",
+ "the body of a message to majordomo\@vger.kernel.org\n",
+ "More majordomo info at http://vger.kernel.org/majordomo-info.html"
]
-b4f1601ae20c8ecf61f325503c53a75fbc48990961f5a98a6e281f54d08c3f42
+d3ca562a69c4de49e9a027cb70aec5d44f507ea2c1dc38b076a1fd629c44a52a
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.