diff for duplicates of <1470845455.2576.6.camel@intel.com> diff --git a/a/1.txt b/N1/1.txt index 93670e0..6707cf3 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,17 +1,28 @@ -T24gV2VkLCAyMDE2LTA4LTEwIGF0IDA3OjM0ICswMjAwLCBDaHJpc3RvcGhlIEpBSUxMRVQgd3Jv -dGU6DQo+IEl0IGlzIGxpa2VseSB0aGF0IGNoZWNraW5nIHRoZSByZXN1bHQgb2YgJ3NldHVwX2N0 -eHQnIGlzIGV4cGVjdGVkDQo+IGhlcmUuDQo+IA0KPiBTaWduZWQtb2ZmLWJ5OiBDaHJpc3RvcGhl -IEpBSUxMRVQgPGNocmlzdG9waGUuamFpbGxldEB3YW5hZG9vLmZyPg0KPiAtLS0NCj4gwqBkcml2 -ZXJzL2luZmluaWJhbmQvaHcvaGZpMS9maWxlX29wcy5jIHwgMiArLQ0KPiDCoDEgZmlsZSBjaGFu -Z2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQ0KPiANCj4gZGlmZiAtLWdpdCBhL2Ry -aXZlcnMvaW5maW5pYmFuZC9ody9oZmkxL2ZpbGVfb3BzLmMNCj4gYi9kcml2ZXJzL2luZmluaWJh -bmQvaHcvaGZpMS9maWxlX29wcy5jDQo+IGluZGV4IDFlY2JlYzE5MjM1OC4uZWQ3NmJlMzliMjBi -IDEwMDY0NA0KPiAtLS0gYS9kcml2ZXJzL2luZmluaWJhbmQvaHcvaGZpMS9maWxlX29wcy5jDQo+ -ICsrKyBiL2RyaXZlcnMvaW5maW5pYmFuZC9ody9oZmkxL2ZpbGVfb3BzLmMNCj4gQEAgLTIyMiw3 -ICsyMjIsNyBAQCBzdGF0aWMgbG9uZyBoZmkxX2ZpbGVfaW9jdGwoc3RydWN0IGZpbGUgKmZwLA0K -PiB1bnNpZ25lZCBpbnQgY21kLA0KPiDCoAkJcmV0ID0gYXNzaWduX2N0eHQoZnAsICZ1aW5mbyk7 -DQo+IMKgCQlpZiAocmV0IDwgMCkNCj4gwqAJCQlyZXR1cm4gcmV0Ow0KPiAtCQlzZXR1cF9jdHh0 -KGZwKTsNCj4gKwkJcmV0ID0gc2V0dXBfY3R4dChmcCk7DQo+IMKgCQlpZiAocmV0KQ0KPiDCoAkJ -CXJldHVybiByZXQ7DQo+IMKgCQlyZXQgPSB1c2VyX2luaXQoZnApOw0KDQpUaGFua3MgZm9yIHRo -ZSBwYXRjaC4NCg0KQWNrZWQtYnk6IERlbm5pcyBEYWxlc3NhbmRybyA8ZGVubmlzLmRhbGVzc2Fu -ZHJvQGludGVsLmNvbT4 +On Wed, 2016-08-10 at 07:34 +0200, Christophe JAILLET wrote: +> It is likely that checking the result of 'setup_ctxt' is expected +> here. +> +> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> +> --- +> drivers/infiniband/hw/hfi1/file_ops.c | 2 +- +> 1 file changed, 1 insertion(+), 1 deletion(-) +> +> diff --git a/drivers/infiniband/hw/hfi1/file_ops.c +> b/drivers/infiniband/hw/hfi1/file_ops.c +> index 1ecbec192358..ed76be39b20b 100644 +> --- a/drivers/infiniband/hw/hfi1/file_ops.c +> +++ b/drivers/infiniband/hw/hfi1/file_ops.c +> @@ -222,7 +222,7 @@ static long hfi1_file_ioctl(struct file *fp, +> unsigned int cmd, +> ret = assign_ctxt(fp, &uinfo); +> if (ret < 0) +> return ret; +> - setup_ctxt(fp); +> + ret = setup_ctxt(fp); +> if (ret) +> return ret; +> ret = user_init(fp); + +Thanks for the patch. + +Acked-by: Dennis Dalessandro <dennis.dalessandro@intel.com> diff --git a/a/content_digest b/N1/content_digest index cb69510..68e0908 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -16,22 +16,33 @@ " kernel-janitors@vger.kernel.org <kernel-janitors@vger.kernel.org>\0" "\00:1\0" "b\0" - "T24gV2VkLCAyMDE2LTA4LTEwIGF0IDA3OjM0ICswMjAwLCBDaHJpc3RvcGhlIEpBSUxMRVQgd3Jv\n" - "dGU6DQo+IEl0IGlzIGxpa2VseSB0aGF0IGNoZWNraW5nIHRoZSByZXN1bHQgb2YgJ3NldHVwX2N0\n" - "eHQnIGlzIGV4cGVjdGVkDQo+IGhlcmUuDQo+IA0KPiBTaWduZWQtb2ZmLWJ5OiBDaHJpc3RvcGhl\n" - "IEpBSUxMRVQgPGNocmlzdG9waGUuamFpbGxldEB3YW5hZG9vLmZyPg0KPiAtLS0NCj4gwqBkcml2\n" - "ZXJzL2luZmluaWJhbmQvaHcvaGZpMS9maWxlX29wcy5jIHwgMiArLQ0KPiDCoDEgZmlsZSBjaGFu\n" - "Z2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQ0KPiANCj4gZGlmZiAtLWdpdCBhL2Ry\n" - "aXZlcnMvaW5maW5pYmFuZC9ody9oZmkxL2ZpbGVfb3BzLmMNCj4gYi9kcml2ZXJzL2luZmluaWJh\n" - "bmQvaHcvaGZpMS9maWxlX29wcy5jDQo+IGluZGV4IDFlY2JlYzE5MjM1OC4uZWQ3NmJlMzliMjBi\n" - "IDEwMDY0NA0KPiAtLS0gYS9kcml2ZXJzL2luZmluaWJhbmQvaHcvaGZpMS9maWxlX29wcy5jDQo+\n" - "ICsrKyBiL2RyaXZlcnMvaW5maW5pYmFuZC9ody9oZmkxL2ZpbGVfb3BzLmMNCj4gQEAgLTIyMiw3\n" - "ICsyMjIsNyBAQCBzdGF0aWMgbG9uZyBoZmkxX2ZpbGVfaW9jdGwoc3RydWN0IGZpbGUgKmZwLA0K\n" - "PiB1bnNpZ25lZCBpbnQgY21kLA0KPiDCoAkJcmV0ID0gYXNzaWduX2N0eHQoZnAsICZ1aW5mbyk7\n" - "DQo+IMKgCQlpZiAocmV0IDwgMCkNCj4gwqAJCQlyZXR1cm4gcmV0Ow0KPiAtCQlzZXR1cF9jdHh0\n" - "KGZwKTsNCj4gKwkJcmV0ID0gc2V0dXBfY3R4dChmcCk7DQo+IMKgCQlpZiAocmV0KQ0KPiDCoAkJ\n" - "CXJldHVybiByZXQ7DQo+IMKgCQlyZXQgPSB1c2VyX2luaXQoZnApOw0KDQpUaGFua3MgZm9yIHRo\n" - "ZSBwYXRjaC4NCg0KQWNrZWQtYnk6IERlbm5pcyBEYWxlc3NhbmRybyA8ZGVubmlzLmRhbGVzc2Fu\n" - ZHJvQGludGVsLmNvbT4 + "On Wed, 2016-08-10 at 07:34 +0200, Christophe JAILLET wrote:\n" + "> It is likely that checking the result of 'setup_ctxt' is expected\n" + "> here.\n" + "> \n" + "> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>\n" + "> ---\n" + "> \302\240drivers/infiniband/hw/hfi1/file_ops.c | 2 +-\n" + "> \302\2401 file changed, 1 insertion(+), 1 deletion(-)\n" + "> \n" + "> diff --git a/drivers/infiniband/hw/hfi1/file_ops.c\n" + "> b/drivers/infiniband/hw/hfi1/file_ops.c\n" + "> index 1ecbec192358..ed76be39b20b 100644\n" + "> --- a/drivers/infiniband/hw/hfi1/file_ops.c\n" + "> +++ b/drivers/infiniband/hw/hfi1/file_ops.c\n" + "> @@ -222,7 +222,7 @@ static long hfi1_file_ioctl(struct file *fp,\n" + "> unsigned int cmd,\n" + "> \302\240\t\tret = assign_ctxt(fp, &uinfo);\n" + "> \302\240\t\tif (ret < 0)\n" + "> \302\240\t\t\treturn ret;\n" + "> -\t\tsetup_ctxt(fp);\n" + "> +\t\tret = setup_ctxt(fp);\n" + "> \302\240\t\tif (ret)\n" + "> \302\240\t\t\treturn ret;\n" + "> \302\240\t\tret = user_init(fp);\n" + "\n" + "Thanks for the patch.\n" + "\n" + Acked-by: Dennis Dalessandro <dennis.dalessandro@intel.com> -596da7a9cbfcfdd856957ef2fd7a545ac90ecb12761b1d91958ea3eb8692b9f5 +77e19b31671fcd56a2792d8960249856089a779c2bcef7bb31445ff9f12b8779
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.