diff for duplicates of <87wo6hvxkz.fsf@vitty.brq.redhat.com> diff --git a/a/1.txt b/N1/1.txt index d6cf6c1..fe0d180 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -20,7 +20,7 @@ Colin Ian King <colin.king@canonical.com> writes: >>> +++ b/drivers/hv/channel_mgmt.c >>> @@ -425,8 +425,6 @@ void hv_process_channel_removal(struct vmbus_channel *channel) >>> ->>> if (channel->primary_channel = NULL) { +>>> if (channel->primary_channel == NULL) { >>> list_del(&channel->listentry); >>> - >>> - primary_channel = channel; @@ -30,7 +30,7 @@ Colin Ian King <colin.king@canonical.com> writes: >> >> If I'm looking at the right source (5.7-rc1) it *is* beeing used: >> ->> if (channel->primary_channel = NULL) { +>> if (channel->primary_channel == NULL) { >> list_del(&channel->listentry); >> >> primary_channel = channel; @@ -45,7 +45,7 @@ Colin Ian King <colin.king@canonical.com> writes: >> * We need to free the bit for init_vp_index() to work in the case >> * of sub-channel, when we reload drivers like hv_netvsc. >> */ ->> if (channel->affinity_policy = HV_LOCALIZED) +>> if (channel->affinity_policy == HV_LOCALIZED) >> cpumask_clear_cpu(channel->target_cpu, >> &primary_channel->alloced_cpus_in_node); >> ^^^^^ HERE ^^^^^ diff --git a/a/content_digest b/N1/content_digest index edb5a43..0fb2fe8 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,7 +3,7 @@ "ref\0606c442c-1923-77d4-c350-e06878172c44@canonical.com\0" "From\0Vitaly Kuznetsov <vkuznets@redhat.com>\0" "Subject\0Re: [PATCH][next] drivers: hv: remove redundant assignment to pointer primary_channel\0" - "Date\0Wed, 15 Apr 2020 10:43:08 +0000\0" + "Date\0Wed, 15 Apr 2020 12:43:08 +0200\0" "To\0Colin Ian King <colin.king@canonical.com>\0" "Cc\0kernel-janitors@vger.kernel.org" linux-kernel@vger.kernel.org @@ -36,7 +36,7 @@ ">>> +++ b/drivers/hv/channel_mgmt.c\n" ">>> @@ -425,8 +425,6 @@ void hv_process_channel_removal(struct vmbus_channel *channel)\n" ">>> \n" - ">>> \tif (channel->primary_channel = NULL) {\n" + ">>> \tif (channel->primary_channel == NULL) {\n" ">>> \t\tlist_del(&channel->listentry);\n" ">>> -\n" ">>> -\t\tprimary_channel = channel;\n" @@ -46,7 +46,7 @@ ">> \n" ">> If I'm looking at the right source (5.7-rc1) it *is* beeing used:\n" ">> \n" - ">> \tif (channel->primary_channel = NULL) {\n" + ">> \tif (channel->primary_channel == NULL) {\n" ">> \t\tlist_del(&channel->listentry);\n" ">> \n" ">> \t\tprimary_channel = channel;\n" @@ -61,7 +61,7 @@ ">> \t * We need to free the bit for init_vp_index() to work in the case\n" ">> \t * of sub-channel, when we reload drivers like hv_netvsc.\n" ">> \t */\n" - ">> \tif (channel->affinity_policy = HV_LOCALIZED)\n" + ">> \tif (channel->affinity_policy == HV_LOCALIZED)\n" ">> \t\tcpumask_clear_cpu(channel->target_cpu,\n" ">> \t\t\t\t &primary_channel->alloced_cpus_in_node);\n" ">> ^^^^^ HERE ^^^^^\n" @@ -82,4 +82,4 @@ "-- \n" Vitaly -d3edbae7b744fac0e8d544b88e9d39ca2760f78929c0ec6d58477ebab6f3ad46 +2a12642a4171b97f19f0bced2d48d8166847e8bbd13bf1b25b3f84a1ceb482d9
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.