Hi,
The tag says '2/5' but I don't see a thread. Is the series meant to
contain more patches?
Also, the title is not very specific about where the assignment is
removed. I have committed with the following title:
xen/evtchn: Remove useful assignment in evtchn_alloc_unbound
On 05/02/2024 20:41, Stefano Stabellini wrote:
On Mon, 5 Feb 2024, Frediano Ziglio wrote:
The variable is assigned later, that value is never used.
I also add 'rc' just for clarity.
Signed-off-by: Frediano Ziglio <[email protected]>
Reviewed-by: Stefano Stabellini <[email protected]>
---
xen/common/event_channel.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/xen/common/event_channel.c b/xen/common/event_channel.c
index a7a004a084..15aec5dcbb 100644
--- a/xen/common/event_channel.c
+++ b/xen/common/event_channel.c
@@ -324,8 +324,6 @@ int evtchn_alloc_unbound(evtchn_alloc_unbound_t *alloc,
evtchn_port_t port)
goto out;
}
- rc = 0;
-
This looks like a missing clean-up from e5ba5165cae6 ("xen/evtchn: Purge
ERROR_EXIT{,_DOM}()").
Cheers,
--
Julien Grall
