diff for duplicates of <87d08axb7k.fsf@vitty.brq.redhat.com> diff --git a/a/1.txt b/N1/1.txt index e95082d..0d30bb1 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -17,7 +17,7 @@ Colin 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; @@ -27,7 +27,7 @@ Colin 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; @@ -42,7 +42,7 @@ If I'm looking at the right source (5.7-rc1) it *is* beeing used: * 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 9642ad9..9f24228 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,7 +1,7 @@ "ref\020200414152343.243166-1-colin.king@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\0Tue, 14 Apr 2020 16:51:11 +0000\0" + "Date\0Tue, 14 Apr 2020 18:51:11 +0200\0" "To\0Colin King <colin.king@canonical.com>\0" "Cc\0kernel-janitors@vger.kernel.org" linux-kernel@vger.kernel.org @@ -31,7 +31,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" @@ -41,7 +41,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" @@ -56,7 +56,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" @@ -64,4 +64,4 @@ "-- \n" Vitaly -0b8a4b397a57a30cbd58bc331111c6c79dde3400846720426a1ab722c2edd6e7 +5b10c802b0d6ac1edf98b8d697668f2f3e22e04bf20372f486dff9ca4ba2ba87
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.