diff for duplicates of <20161013233742.GA5854@hgst.com> diff --git a/a/1.txt b/N1/1.txt index f09dd33..0b3a396 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,26 +1,32 @@ -SGVsbG8gVGVqdW4sCgpUaGUgMTAvMTMvMjAxNiAxOToyMiwgVGVqdW4gSGVvIHdyb3RlOgo+IEhl -bGxvLCBBZGFtLgo+IAo+IFNvcnJ5IGFib3V0IGxhdGUgcmVwbHkuICBXYXMgb24gdmFjYXRpb24u -CgpOUCBJIHdhcyBvbiB2YWNhdGlvbiBhdCB0aGUgZW5kIG9mIHRoZSB3ZWVrIGxhc3Qgd2Vlay4K -Cj4gCj4gT24gVGh1LCBPY3QgMTMsIDIwMTYgYXQgMDQ6MDA6MzFQTSAtMDcwMCwgQWRhbSBNYW56 -YW5hcmVzIHdyb3RlOgo+ID4gQWRkIGEgc3lzZnMgZW50cnkgdG8gdHVybiBvbiBwcmlvcml0eSBp -bmZvcm1hdGlvbiBiZWluZyBwYXNzZWQKPiA+IHRvIGEgQVRBIGRldmljZS4gQnkgZGVmYXVsdCB0 -aGlzIGZlYXR1cmUgaXMgdHVybmVkIG9mZi4KPiA+IAo+ID4gVGhpcyBwYXRjaCBkZXBlbmRzIG9u -IGF0YTogRW5hYmxpbmcgQVRBIENvbW1hbmQgUHJpb3JpdGllcwo+IAo+IExvb2tzIGdlbmVyYWxs -eSBnb29kIGJ1dCBjYW4gd2UgcGxlYXNlIHVzZSBhIGRldmljZSBhdHRyaWJ1dGUgbmFtZQo+IHdo -aWNoIGlzIG1vcmUgc3BlY2lmaWMgLSBpZS4gZW5hYmxlX25jcV9wcmlvPwoKV2lsbCBkbywgSSds -bCBhbHNvIGRvdWJsZSBjaGVjayB0aGUgbmFtaW5nIHNjaGVtZSBvZiBmdW5jdGlvbnMgYW5kIHZh -cmlhYmxlcyAKYWxzby4gVGhlIGZ1bmN0aW9ucyB0aGF0IGNoZWNrIGlmIHRoZSBkZXZpY2UgaGFz -IHRoZSBuY3EgcHJpbyBjYXBhYmlsaXR5IG1pZ2h0CmJlIHRvbyBzaW1pbGFyIHRvIHRoZSBmdW5j -dGlvbiB0aGF0IGNoZWNrcyBpZiB0aGUgZGV2aWNlIGF0dHJpYnV0ZSBpcyAKZW5hYmxlZC4KCj4g -Cj4gVGhhbmtzLgo+IAo+IC0tIAo+IHRlanVuCgpUYWtlIGNhcmUsCkFkYW0KV2VzdGVybiBEaWdp -dGFsIENvcnBvcmF0aW9uIChhbmQgaXRzIHN1YnNpZGlhcmllcykgRS1tYWlsIENvbmZpZGVudGlh -bGl0eSBOb3RpY2UgJiBEaXNjbGFpbWVyOgoKVGhpcyBlLW1haWwgYW5kIGFueSBmaWxlcyB0cmFu -c21pdHRlZCB3aXRoIGl0IG1heSBjb250YWluIGNvbmZpZGVudGlhbCBvciBsZWdhbGx5IHByaXZp -bGVnZWQgaW5mb3JtYXRpb24gb2YgV0RDIGFuZC9vciBpdHMgYWZmaWxpYXRlcywgYW5kIGFyZSBp -bnRlbmRlZCBzb2xlbHkgZm9yIHRoZSB1c2Ugb2YgdGhlIGluZGl2aWR1YWwgb3IgZW50aXR5IHRv -IHdoaWNoIHRoZXkgYXJlIGFkZHJlc3NlZC4gSWYgeW91IGFyZSBub3QgdGhlIGludGVuZGVkIHJl -Y2lwaWVudCwgYW55IGRpc2Nsb3N1cmUsIGNvcHlpbmcsIGRpc3RyaWJ1dGlvbiBvciBhbnkgYWN0 -aW9uIHRha2VuIG9yIG9taXR0ZWQgdG8gYmUgdGFrZW4gaW4gcmVsaWFuY2Ugb24gaXQsIGlzIHBy -b2hpYml0ZWQuIElmIHlvdSBoYXZlIHJlY2VpdmVkIHRoaXMgZS1tYWlsIGluIGVycm9yLCBwbGVh -c2Ugbm90aWZ5IHRoZSBzZW5kZXIgaW1tZWRpYXRlbHkgYW5kIGRlbGV0ZSB0aGUgZS1tYWlsIGlu -IGl0cyBlbnRpcmV0eSBmcm9tIHlvdXIgc3lzdGVtLgo= +Hello Tejun, + +The 10/13/2016 19:22, Tejun Heo wrote: +> Hello, Adam. +> +> Sorry about late reply. Was on vacation. + +NP I was on vacation at the end of the week last week. + +> +> On Thu, Oct 13, 2016 at 04:00:31PM -0700, Adam Manzanares wrote: +> > Add a sysfs entry to turn on priority information being passed +> > to a ATA device. By default this feature is turned off. +> > +> > This patch depends on ata: Enabling ATA Command Priorities +> +> Looks generally good but can we please use a device attribute name +> which is more specific - ie. enable_ncq_prio? + +Will do, I'll also double check the naming scheme of functions and variables +also. The functions that check if the device has the ncq prio capability might +be too similar to the function that checks if the device attribute is +enabled. + +> +> Thanks. +> +> -- +> tejun + +Take care, +Adam diff --git a/a/content_digest b/N1/content_digest index 3750ea1..87675e5 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -6,48 +6,54 @@ "Date\0Thu, 13 Oct 2016 16:37:43 -0700\0" "To\0Tejun Heo <tj@kernel.org>\0" "Cc\0Adam Manzanares <adam.manzanares@hgst.com>" - <axboe@kernel.dk> - <dan.j.williams@intel.com> - <hare@suse.de> - <martin.petersen@oracle.com> - <mchristi@redhat.com> - <toshi.kani@hpe.com> - <ming.lei@canonical.com> - <sathya.prakash@broadcom.com> - <chaitra.basappa@broadcom.com> - <suganath-prabu.subramani@broadcom.com> - <linux-block@vger.kernel.org> - <linux-ide@vger.kernel.org> - <linux-kernel@vger.kernel.org> - <MPT-FusionLinux.pdl@broadcom.com> - " <linux-scsi@vger.kernel.org>\0" + axboe@kernel.dk + dan.j.williams@intel.com + hare@suse.de + martin.petersen@oracle.com + mchristi@redhat.com + toshi.kani@hpe.com + ming.lei@canonical.com + sathya.prakash@broadcom.com + chaitra.basappa@broadcom.com + suganath-prabu.subramani@broadcom.com + linux-block@vger.kernel.org + linux-ide@vger.kernel.org + linux-kernel@vger.kernel.org + MPT-FusionLinux.pdl@broadcom.com + " linux-scsi@vger.kernel.org\0" "\00:1\0" "b\0" - "SGVsbG8gVGVqdW4sCgpUaGUgMTAvMTMvMjAxNiAxOToyMiwgVGVqdW4gSGVvIHdyb3RlOgo+IEhl\n" - "bGxvLCBBZGFtLgo+IAo+IFNvcnJ5IGFib3V0IGxhdGUgcmVwbHkuICBXYXMgb24gdmFjYXRpb24u\n" - "CgpOUCBJIHdhcyBvbiB2YWNhdGlvbiBhdCB0aGUgZW5kIG9mIHRoZSB3ZWVrIGxhc3Qgd2Vlay4K\n" - "Cj4gCj4gT24gVGh1LCBPY3QgMTMsIDIwMTYgYXQgMDQ6MDA6MzFQTSAtMDcwMCwgQWRhbSBNYW56\n" - "YW5hcmVzIHdyb3RlOgo+ID4gQWRkIGEgc3lzZnMgZW50cnkgdG8gdHVybiBvbiBwcmlvcml0eSBp\n" - "bmZvcm1hdGlvbiBiZWluZyBwYXNzZWQKPiA+IHRvIGEgQVRBIGRldmljZS4gQnkgZGVmYXVsdCB0\n" - "aGlzIGZlYXR1cmUgaXMgdHVybmVkIG9mZi4KPiA+IAo+ID4gVGhpcyBwYXRjaCBkZXBlbmRzIG9u\n" - "IGF0YTogRW5hYmxpbmcgQVRBIENvbW1hbmQgUHJpb3JpdGllcwo+IAo+IExvb2tzIGdlbmVyYWxs\n" - "eSBnb29kIGJ1dCBjYW4gd2UgcGxlYXNlIHVzZSBhIGRldmljZSBhdHRyaWJ1dGUgbmFtZQo+IHdo\n" - "aWNoIGlzIG1vcmUgc3BlY2lmaWMgLSBpZS4gZW5hYmxlX25jcV9wcmlvPwoKV2lsbCBkbywgSSds\n" - "bCBhbHNvIGRvdWJsZSBjaGVjayB0aGUgbmFtaW5nIHNjaGVtZSBvZiBmdW5jdGlvbnMgYW5kIHZh\n" - "cmlhYmxlcyAKYWxzby4gVGhlIGZ1bmN0aW9ucyB0aGF0IGNoZWNrIGlmIHRoZSBkZXZpY2UgaGFz\n" - "IHRoZSBuY3EgcHJpbyBjYXBhYmlsaXR5IG1pZ2h0CmJlIHRvbyBzaW1pbGFyIHRvIHRoZSBmdW5j\n" - "dGlvbiB0aGF0IGNoZWNrcyBpZiB0aGUgZGV2aWNlIGF0dHJpYnV0ZSBpcyAKZW5hYmxlZC4KCj4g\n" - "Cj4gVGhhbmtzLgo+IAo+IC0tIAo+IHRlanVuCgpUYWtlIGNhcmUsCkFkYW0KV2VzdGVybiBEaWdp\n" - "dGFsIENvcnBvcmF0aW9uIChhbmQgaXRzIHN1YnNpZGlhcmllcykgRS1tYWlsIENvbmZpZGVudGlh\n" - "bGl0eSBOb3RpY2UgJiBEaXNjbGFpbWVyOgoKVGhpcyBlLW1haWwgYW5kIGFueSBmaWxlcyB0cmFu\n" - "c21pdHRlZCB3aXRoIGl0IG1heSBjb250YWluIGNvbmZpZGVudGlhbCBvciBsZWdhbGx5IHByaXZp\n" - "bGVnZWQgaW5mb3JtYXRpb24gb2YgV0RDIGFuZC9vciBpdHMgYWZmaWxpYXRlcywgYW5kIGFyZSBp\n" - "bnRlbmRlZCBzb2xlbHkgZm9yIHRoZSB1c2Ugb2YgdGhlIGluZGl2aWR1YWwgb3IgZW50aXR5IHRv\n" - "IHdoaWNoIHRoZXkgYXJlIGFkZHJlc3NlZC4gSWYgeW91IGFyZSBub3QgdGhlIGludGVuZGVkIHJl\n" - "Y2lwaWVudCwgYW55IGRpc2Nsb3N1cmUsIGNvcHlpbmcsIGRpc3RyaWJ1dGlvbiBvciBhbnkgYWN0\n" - "aW9uIHRha2VuIG9yIG9taXR0ZWQgdG8gYmUgdGFrZW4gaW4gcmVsaWFuY2Ugb24gaXQsIGlzIHBy\n" - "b2hpYml0ZWQuIElmIHlvdSBoYXZlIHJlY2VpdmVkIHRoaXMgZS1tYWlsIGluIGVycm9yLCBwbGVh\n" - "c2Ugbm90aWZ5IHRoZSBzZW5kZXIgaW1tZWRpYXRlbHkgYW5kIGRlbGV0ZSB0aGUgZS1tYWlsIGlu\n" - IGl0cyBlbnRpcmV0eSBmcm9tIHlvdXIgc3lzdGVtLgo= + "Hello Tejun,\n" + "\n" + "The 10/13/2016 19:22, Tejun Heo wrote:\n" + "> Hello, Adam.\n" + "> \n" + "> Sorry about late reply. Was on vacation.\n" + "\n" + "NP I was on vacation at the end of the week last week.\n" + "\n" + "> \n" + "> On Thu, Oct 13, 2016 at 04:00:31PM -0700, Adam Manzanares wrote:\n" + "> > Add a sysfs entry to turn on priority information being passed\n" + "> > to a ATA device. By default this feature is turned off.\n" + "> > \n" + "> > This patch depends on ata: Enabling ATA Command Priorities\n" + "> \n" + "> Looks generally good but can we please use a device attribute name\n" + "> which is more specific - ie. enable_ncq_prio?\n" + "\n" + "Will do, I'll also double check the naming scheme of functions and variables \n" + "also. The functions that check if the device has the ncq prio capability might\n" + "be too similar to the function that checks if the device attribute is \n" + "enabled.\n" + "\n" + "> \n" + "> Thanks.\n" + "> \n" + "> -- \n" + "> tejun\n" + "\n" + "Take care,\n" + Adam -dfcbdfe4e2fdfa4fbfde468ba4aad640d2be1eb322c07e637cb75dbf0c42c993 +e9f94721675178912665224b2fb5921562d25f65b6e76061df34edb958348efc
diff --git a/a/1.txt b/N2/1.txt index f09dd33..0b3a396 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,26 +1,32 @@ -SGVsbG8gVGVqdW4sCgpUaGUgMTAvMTMvMjAxNiAxOToyMiwgVGVqdW4gSGVvIHdyb3RlOgo+IEhl -bGxvLCBBZGFtLgo+IAo+IFNvcnJ5IGFib3V0IGxhdGUgcmVwbHkuICBXYXMgb24gdmFjYXRpb24u -CgpOUCBJIHdhcyBvbiB2YWNhdGlvbiBhdCB0aGUgZW5kIG9mIHRoZSB3ZWVrIGxhc3Qgd2Vlay4K -Cj4gCj4gT24gVGh1LCBPY3QgMTMsIDIwMTYgYXQgMDQ6MDA6MzFQTSAtMDcwMCwgQWRhbSBNYW56 -YW5hcmVzIHdyb3RlOgo+ID4gQWRkIGEgc3lzZnMgZW50cnkgdG8gdHVybiBvbiBwcmlvcml0eSBp -bmZvcm1hdGlvbiBiZWluZyBwYXNzZWQKPiA+IHRvIGEgQVRBIGRldmljZS4gQnkgZGVmYXVsdCB0 -aGlzIGZlYXR1cmUgaXMgdHVybmVkIG9mZi4KPiA+IAo+ID4gVGhpcyBwYXRjaCBkZXBlbmRzIG9u -IGF0YTogRW5hYmxpbmcgQVRBIENvbW1hbmQgUHJpb3JpdGllcwo+IAo+IExvb2tzIGdlbmVyYWxs -eSBnb29kIGJ1dCBjYW4gd2UgcGxlYXNlIHVzZSBhIGRldmljZSBhdHRyaWJ1dGUgbmFtZQo+IHdo -aWNoIGlzIG1vcmUgc3BlY2lmaWMgLSBpZS4gZW5hYmxlX25jcV9wcmlvPwoKV2lsbCBkbywgSSds -bCBhbHNvIGRvdWJsZSBjaGVjayB0aGUgbmFtaW5nIHNjaGVtZSBvZiBmdW5jdGlvbnMgYW5kIHZh -cmlhYmxlcyAKYWxzby4gVGhlIGZ1bmN0aW9ucyB0aGF0IGNoZWNrIGlmIHRoZSBkZXZpY2UgaGFz -IHRoZSBuY3EgcHJpbyBjYXBhYmlsaXR5IG1pZ2h0CmJlIHRvbyBzaW1pbGFyIHRvIHRoZSBmdW5j -dGlvbiB0aGF0IGNoZWNrcyBpZiB0aGUgZGV2aWNlIGF0dHJpYnV0ZSBpcyAKZW5hYmxlZC4KCj4g -Cj4gVGhhbmtzLgo+IAo+IC0tIAo+IHRlanVuCgpUYWtlIGNhcmUsCkFkYW0KV2VzdGVybiBEaWdp -dGFsIENvcnBvcmF0aW9uIChhbmQgaXRzIHN1YnNpZGlhcmllcykgRS1tYWlsIENvbmZpZGVudGlh -bGl0eSBOb3RpY2UgJiBEaXNjbGFpbWVyOgoKVGhpcyBlLW1haWwgYW5kIGFueSBmaWxlcyB0cmFu -c21pdHRlZCB3aXRoIGl0IG1heSBjb250YWluIGNvbmZpZGVudGlhbCBvciBsZWdhbGx5IHByaXZp -bGVnZWQgaW5mb3JtYXRpb24gb2YgV0RDIGFuZC9vciBpdHMgYWZmaWxpYXRlcywgYW5kIGFyZSBp -bnRlbmRlZCBzb2xlbHkgZm9yIHRoZSB1c2Ugb2YgdGhlIGluZGl2aWR1YWwgb3IgZW50aXR5IHRv -IHdoaWNoIHRoZXkgYXJlIGFkZHJlc3NlZC4gSWYgeW91IGFyZSBub3QgdGhlIGludGVuZGVkIHJl -Y2lwaWVudCwgYW55IGRpc2Nsb3N1cmUsIGNvcHlpbmcsIGRpc3RyaWJ1dGlvbiBvciBhbnkgYWN0 -aW9uIHRha2VuIG9yIG9taXR0ZWQgdG8gYmUgdGFrZW4gaW4gcmVsaWFuY2Ugb24gaXQsIGlzIHBy -b2hpYml0ZWQuIElmIHlvdSBoYXZlIHJlY2VpdmVkIHRoaXMgZS1tYWlsIGluIGVycm9yLCBwbGVh -c2Ugbm90aWZ5IHRoZSBzZW5kZXIgaW1tZWRpYXRlbHkgYW5kIGRlbGV0ZSB0aGUgZS1tYWlsIGlu -IGl0cyBlbnRpcmV0eSBmcm9tIHlvdXIgc3lzdGVtLgo= +Hello Tejun, + +The 10/13/2016 19:22, Tejun Heo wrote: +> Hello, Adam. +> +> Sorry about late reply. Was on vacation. + +NP I was on vacation at the end of the week last week. + +> +> On Thu, Oct 13, 2016 at 04:00:31PM -0700, Adam Manzanares wrote: +> > Add a sysfs entry to turn on priority information being passed +> > to a ATA device. By default this feature is turned off. +> > +> > This patch depends on ata: Enabling ATA Command Priorities +> +> Looks generally good but can we please use a device attribute name +> which is more specific - ie. enable_ncq_prio? + +Will do, I'll also double check the naming scheme of functions and variables +also. The functions that check if the device has the ncq prio capability might +be too similar to the function that checks if the device attribute is +enabled. + +> +> Thanks. +> +> -- +> tejun + +Take care, +Adam diff --git a/a/content_digest b/N2/content_digest index 3750ea1..41c5045 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -23,31 +23,37 @@ " <linux-scsi@vger.kernel.org>\0" "\00:1\0" "b\0" - "SGVsbG8gVGVqdW4sCgpUaGUgMTAvMTMvMjAxNiAxOToyMiwgVGVqdW4gSGVvIHdyb3RlOgo+IEhl\n" - "bGxvLCBBZGFtLgo+IAo+IFNvcnJ5IGFib3V0IGxhdGUgcmVwbHkuICBXYXMgb24gdmFjYXRpb24u\n" - "CgpOUCBJIHdhcyBvbiB2YWNhdGlvbiBhdCB0aGUgZW5kIG9mIHRoZSB3ZWVrIGxhc3Qgd2Vlay4K\n" - "Cj4gCj4gT24gVGh1LCBPY3QgMTMsIDIwMTYgYXQgMDQ6MDA6MzFQTSAtMDcwMCwgQWRhbSBNYW56\n" - "YW5hcmVzIHdyb3RlOgo+ID4gQWRkIGEgc3lzZnMgZW50cnkgdG8gdHVybiBvbiBwcmlvcml0eSBp\n" - "bmZvcm1hdGlvbiBiZWluZyBwYXNzZWQKPiA+IHRvIGEgQVRBIGRldmljZS4gQnkgZGVmYXVsdCB0\n" - "aGlzIGZlYXR1cmUgaXMgdHVybmVkIG9mZi4KPiA+IAo+ID4gVGhpcyBwYXRjaCBkZXBlbmRzIG9u\n" - "IGF0YTogRW5hYmxpbmcgQVRBIENvbW1hbmQgUHJpb3JpdGllcwo+IAo+IExvb2tzIGdlbmVyYWxs\n" - "eSBnb29kIGJ1dCBjYW4gd2UgcGxlYXNlIHVzZSBhIGRldmljZSBhdHRyaWJ1dGUgbmFtZQo+IHdo\n" - "aWNoIGlzIG1vcmUgc3BlY2lmaWMgLSBpZS4gZW5hYmxlX25jcV9wcmlvPwoKV2lsbCBkbywgSSds\n" - "bCBhbHNvIGRvdWJsZSBjaGVjayB0aGUgbmFtaW5nIHNjaGVtZSBvZiBmdW5jdGlvbnMgYW5kIHZh\n" - "cmlhYmxlcyAKYWxzby4gVGhlIGZ1bmN0aW9ucyB0aGF0IGNoZWNrIGlmIHRoZSBkZXZpY2UgaGFz\n" - "IHRoZSBuY3EgcHJpbyBjYXBhYmlsaXR5IG1pZ2h0CmJlIHRvbyBzaW1pbGFyIHRvIHRoZSBmdW5j\n" - "dGlvbiB0aGF0IGNoZWNrcyBpZiB0aGUgZGV2aWNlIGF0dHJpYnV0ZSBpcyAKZW5hYmxlZC4KCj4g\n" - "Cj4gVGhhbmtzLgo+IAo+IC0tIAo+IHRlanVuCgpUYWtlIGNhcmUsCkFkYW0KV2VzdGVybiBEaWdp\n" - "dGFsIENvcnBvcmF0aW9uIChhbmQgaXRzIHN1YnNpZGlhcmllcykgRS1tYWlsIENvbmZpZGVudGlh\n" - "bGl0eSBOb3RpY2UgJiBEaXNjbGFpbWVyOgoKVGhpcyBlLW1haWwgYW5kIGFueSBmaWxlcyB0cmFu\n" - "c21pdHRlZCB3aXRoIGl0IG1heSBjb250YWluIGNvbmZpZGVudGlhbCBvciBsZWdhbGx5IHByaXZp\n" - "bGVnZWQgaW5mb3JtYXRpb24gb2YgV0RDIGFuZC9vciBpdHMgYWZmaWxpYXRlcywgYW5kIGFyZSBp\n" - "bnRlbmRlZCBzb2xlbHkgZm9yIHRoZSB1c2Ugb2YgdGhlIGluZGl2aWR1YWwgb3IgZW50aXR5IHRv\n" - "IHdoaWNoIHRoZXkgYXJlIGFkZHJlc3NlZC4gSWYgeW91IGFyZSBub3QgdGhlIGludGVuZGVkIHJl\n" - "Y2lwaWVudCwgYW55IGRpc2Nsb3N1cmUsIGNvcHlpbmcsIGRpc3RyaWJ1dGlvbiBvciBhbnkgYWN0\n" - "aW9uIHRha2VuIG9yIG9taXR0ZWQgdG8gYmUgdGFrZW4gaW4gcmVsaWFuY2Ugb24gaXQsIGlzIHBy\n" - "b2hpYml0ZWQuIElmIHlvdSBoYXZlIHJlY2VpdmVkIHRoaXMgZS1tYWlsIGluIGVycm9yLCBwbGVh\n" - "c2Ugbm90aWZ5IHRoZSBzZW5kZXIgaW1tZWRpYXRlbHkgYW5kIGRlbGV0ZSB0aGUgZS1tYWlsIGlu\n" - IGl0cyBlbnRpcmV0eSBmcm9tIHlvdXIgc3lzdGVtLgo= + "Hello Tejun,\n" + "\n" + "The 10/13/2016 19:22, Tejun Heo wrote:\n" + "> Hello, Adam.\n" + "> \n" + "> Sorry about late reply. Was on vacation.\n" + "\n" + "NP I was on vacation at the end of the week last week.\n" + "\n" + "> \n" + "> On Thu, Oct 13, 2016 at 04:00:31PM -0700, Adam Manzanares wrote:\n" + "> > Add a sysfs entry to turn on priority information being passed\n" + "> > to a ATA device. By default this feature is turned off.\n" + "> > \n" + "> > This patch depends on ata: Enabling ATA Command Priorities\n" + "> \n" + "> Looks generally good but can we please use a device attribute name\n" + "> which is more specific - ie. enable_ncq_prio?\n" + "\n" + "Will do, I'll also double check the naming scheme of functions and variables \n" + "also. The functions that check if the device has the ncq prio capability might\n" + "be too similar to the function that checks if the device attribute is \n" + "enabled.\n" + "\n" + "> \n" + "> Thanks.\n" + "> \n" + "> -- \n" + "> tejun\n" + "\n" + "Take care,\n" + Adam -dfcbdfe4e2fdfa4fbfde468ba4aad640d2be1eb322c07e637cb75dbf0c42c993 +1b24f501c75f3f1f2de06d80f30e1ec83d1d1f79c9fb50158341783425467f51
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.