diff for duplicates of <1500398136.2042.17.camel@hpe.com> diff --git a/a/1.txt b/N1/1.txt index f4171a6..a3e4059 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,35 +1,23 @@ -On Tue, 2017-07-18 at 18:43 +0200, Borislav Petkov wrote: -> On Tue, Jul 18, 2017 at 03:48:54PM +0000, Kani, Toshimitsu wrote: -> > This patch defines 'struct acpi_oemlist' in "include/linux/acpi.h" -> > as a -> -> I see that. -> -> > common structure, and replaces this specific 'struct -> > acpi_blacklist'. -> -> And what makes acpi_oemlist "common" and acpi_blacklist "specific"? -> -> So let me save you some time - "oemlist" is more specific than -> "blacklist" and I can imagine a blacklist item not always being -> oem-specific. -> -> What I'm hinting at is, don't change that name. acpi_blacklist is -> just fine. - -Well, a list does not need to be a black-list. It can be a white-list -or anything that matters. The caller defines the usage of a list. So, -I tried to avoid putting any usage to the structure name. - -> > Agreed. Will change to a shorter name like below. -> > -> > enum acpi_oemlist_pred predicate; -> -> enum acpi_predicate pred; -> -> is even better. - -Sounds good. - -Thanks, --Toshi +T24gVHVlLCAyMDE3LTA3LTE4IGF0IDE4OjQzICswMjAwLCBCb3Jpc2xhdiBQZXRrb3Ygd3JvdGU6 +DQo+IE9uIFR1ZSwgSnVsIDE4LCAyMDE3IGF0IDAzOjQ4OjU0UE0gKzAwMDAsIEthbmksIFRvc2hp +bWl0c3Ugd3JvdGU6DQo+ID4gVGhpcyBwYXRjaCBkZWZpbmVzICdzdHJ1Y3QgYWNwaV9vZW1saXN0 +JyBpbiAiaW5jbHVkZS9saW51eC9hY3BpLmgiDQo+ID4gYXMgYQ0KPiANCj4gSSBzZWUgdGhhdC4N +Cj4gDQo+ID4gY29tbW9uIHN0cnVjdHVyZSwgYW5kIHJlcGxhY2VzIHRoaXMgc3BlY2lmaWMgJ3N0 +cnVjdA0KPiA+IGFjcGlfYmxhY2tsaXN0Jy4NCj4gDQo+IEFuZCB3aGF0IG1ha2VzIGFjcGlfb2Vt +bGlzdCAiY29tbW9uIiBhbmQgYWNwaV9ibGFja2xpc3QgInNwZWNpZmljIj8NCj4gDQo+IFNvIGxl +dCBtZSBzYXZlIHlvdSBzb21lIHRpbWUgLSAib2VtbGlzdCIgaXMgbW9yZSBzcGVjaWZpYyB0aGFu +DQo+ICJibGFja2xpc3QiIGFuZCBJIGNhbiBpbWFnaW5lIGEgYmxhY2tsaXN0IGl0ZW0gbm90IGFs +d2F5cyBiZWluZw0KPiBvZW0tc3BlY2lmaWMuDQo+IA0KPiBXaGF0IEknbSBoaW50aW5nIGF0IGlz +LCBkb24ndCBjaGFuZ2UgdGhhdCBuYW1lLiBhY3BpX2JsYWNrbGlzdCBpcw0KPiBqdXN0IGZpbmUu +DQoNCldlbGwsIGEgbGlzdCBkb2VzIG5vdCBuZWVkIHRvIGJlIGEgYmxhY2stbGlzdC4gIEl0IGNh +biBiZSBhIHdoaXRlLWxpc3QNCm9yIGFueXRoaW5nIHRoYXQgbWF0dGVycy4gIFRoZSBjYWxsZXIg +ZGVmaW5lcyB0aGUgdXNhZ2Ugb2YgYSBsaXN0LiAgU28sDQpJIHRyaWVkIHRvIGF2b2lkIHB1dHRp +bmcgYW55IHVzYWdlIHRvIHRoZSBzdHJ1Y3R1cmUgbmFtZS4NCg0KPiA+IEFncmVlZC7CoMKgV2ls +bCBjaGFuZ2UgdG8gYSBzaG9ydGVyIG5hbWUgbGlrZSBiZWxvdy7CoA0KPiA+IA0KPiA+IAllbnVt +IGFjcGlfb2VtbGlzdF9wcmVkIHByZWRpY2F0ZTsNCj4gDQo+IAllbnVtIGFjcGlfcHJlZGljYXRl +IHByZWQ7DQo+IA0KPiBpcyBldmVuIGJldHRlci4NCg0KU291bmRzIGdvb2QuDQoNClRoYW5rcywN +Ci1Ub3NoaQ0K +--- +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 diff --git a/a/content_digest b/N1/content_digest index 031230e..22e27cc 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,10 +1,5 @@ - "ref\020170717215912.26070-1-toshi.kani@hpe.com\0" - "ref\020170717215912.26070-2-toshi.kani@hpe.com\0" - "ref\020170718053455.GA8736@nazgul.tnic\0" - "ref\01500392379.2042.13.camel@hpe.com\0" - "ref\020170718164350.GB22334@nazgul.tnic\0" - "From\0Kani, Toshimitsu <toshi.kani@hpe.com>\0" - "Subject\0Re: [PATCH 1/3] ACPI / blacklist: add acpi_match_oemlist() interface\0" + "From\0Toshi Kani <toshi.kani@hpe.com>\0" + "Subject\0[1/3] ACPI / blacklist: add acpi_match_oemlist() interface\0" "Date\0Tue, 18 Jul 2017 17:24:50 +0000\0" "To\0bp@alien8.de <bp@alien8.de>\0" "Cc\0linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>" @@ -17,40 +12,28 @@ " linux-edac@vger.kernel.org <linux-edac@vger.kernel.org>\0" "\00:1\0" "b\0" - "On Tue, 2017-07-18 at 18:43 +0200, Borislav Petkov wrote:\n" - "> On Tue, Jul 18, 2017 at 03:48:54PM +0000, Kani, Toshimitsu wrote:\n" - "> > This patch defines 'struct acpi_oemlist' in \"include/linux/acpi.h\"\n" - "> > as a\n" - "> \n" - "> I see that.\n" - "> \n" - "> > common structure, and replaces this specific 'struct\n" - "> > acpi_blacklist'.\n" - "> \n" - "> And what makes acpi_oemlist \"common\" and acpi_blacklist \"specific\"?\n" - "> \n" - "> So let me save you some time - \"oemlist\" is more specific than\n" - "> \"blacklist\" and I can imagine a blacklist item not always being\n" - "> oem-specific.\n" - "> \n" - "> What I'm hinting at is, don't change that name. acpi_blacklist is\n" - "> just fine.\n" - "\n" - "Well, a list does not need to be a black-list. It can be a white-list\n" - "or anything that matters. The caller defines the usage of a list. So,\n" - "I tried to avoid putting any usage to the structure name.\n" - "\n" - "> > Agreed.\302\240\302\240Will change to a shorter name like below.\302\240\n" - "> > \n" - "> > \tenum acpi_oemlist_pred predicate;\n" - "> \n" - "> \tenum acpi_predicate pred;\n" - "> \n" - "> is even better.\n" - "\n" - "Sounds good.\n" - "\n" - "Thanks,\n" - -Toshi + "T24gVHVlLCAyMDE3LTA3LTE4IGF0IDE4OjQzICswMjAwLCBCb3Jpc2xhdiBQZXRrb3Ygd3JvdGU6\n" + "DQo+IE9uIFR1ZSwgSnVsIDE4LCAyMDE3IGF0IDAzOjQ4OjU0UE0gKzAwMDAsIEthbmksIFRvc2hp\n" + "bWl0c3Ugd3JvdGU6DQo+ID4gVGhpcyBwYXRjaCBkZWZpbmVzICdzdHJ1Y3QgYWNwaV9vZW1saXN0\n" + "JyBpbiAiaW5jbHVkZS9saW51eC9hY3BpLmgiDQo+ID4gYXMgYQ0KPiANCj4gSSBzZWUgdGhhdC4N\n" + "Cj4gDQo+ID4gY29tbW9uIHN0cnVjdHVyZSwgYW5kIHJlcGxhY2VzIHRoaXMgc3BlY2lmaWMgJ3N0\n" + "cnVjdA0KPiA+IGFjcGlfYmxhY2tsaXN0Jy4NCj4gDQo+IEFuZCB3aGF0IG1ha2VzIGFjcGlfb2Vt\n" + "bGlzdCAiY29tbW9uIiBhbmQgYWNwaV9ibGFja2xpc3QgInNwZWNpZmljIj8NCj4gDQo+IFNvIGxl\n" + "dCBtZSBzYXZlIHlvdSBzb21lIHRpbWUgLSAib2VtbGlzdCIgaXMgbW9yZSBzcGVjaWZpYyB0aGFu\n" + "DQo+ICJibGFja2xpc3QiIGFuZCBJIGNhbiBpbWFnaW5lIGEgYmxhY2tsaXN0IGl0ZW0gbm90IGFs\n" + "d2F5cyBiZWluZw0KPiBvZW0tc3BlY2lmaWMuDQo+IA0KPiBXaGF0IEknbSBoaW50aW5nIGF0IGlz\n" + "LCBkb24ndCBjaGFuZ2UgdGhhdCBuYW1lLiBhY3BpX2JsYWNrbGlzdCBpcw0KPiBqdXN0IGZpbmUu\n" + "DQoNCldlbGwsIGEgbGlzdCBkb2VzIG5vdCBuZWVkIHRvIGJlIGEgYmxhY2stbGlzdC4gIEl0IGNh\n" + "biBiZSBhIHdoaXRlLWxpc3QNCm9yIGFueXRoaW5nIHRoYXQgbWF0dGVycy4gIFRoZSBjYWxsZXIg\n" + "ZGVmaW5lcyB0aGUgdXNhZ2Ugb2YgYSBsaXN0LiAgU28sDQpJIHRyaWVkIHRvIGF2b2lkIHB1dHRp\n" + "bmcgYW55IHVzYWdlIHRvIHRoZSBzdHJ1Y3R1cmUgbmFtZS4NCg0KPiA+IEFncmVlZC7CoMKgV2ls\n" + "bCBjaGFuZ2UgdG8gYSBzaG9ydGVyIG5hbWUgbGlrZSBiZWxvdy7CoA0KPiA+IA0KPiA+IAllbnVt\n" + "IGFjcGlfb2VtbGlzdF9wcmVkIHByZWRpY2F0ZTsNCj4gDQo+IAllbnVtIGFjcGlfcHJlZGljYXRl\n" + "IHByZWQ7DQo+IA0KPiBpcyBldmVuIGJldHRlci4NCg0KU291bmRzIGdvb2QuDQoNClRoYW5rcywN\n" + "Ci1Ub3NoaQ0K\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 -5ee60428da7b5e4a156e5f7178b6b60aaa652acfbe50d9eb280e54ed1c203e5b +0066f24935ced67286be409e9153de651f322f45aa1d368a1322d68a904b273b
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.