Thread (3 messages) 3 messages, 2 authors, 2020-09-29

Re: [PATCH] bcache: remove unused function closure_set_ret_ip()

From: Jason Yan <yanaijie@huawei.com>
Date: 2020-09-29 07:31:56

在 2020/9/29 14:49, Coly Li 写道:
On 2020/9/29 14:47, Jason Yan wrote:
quoted
This function has no caller after commit e4bf791937d8 ("bcache: Fix,
improve efficiency of closure_sync()").

Signed-off-by: Jason Yan <yanaijie@huawei.com>
NACK. Indeed, it is necessary for closure debug, you don't want to make
current life even worse.
Get it. Thanks.

Jason
Coly Li

quoted
---
  drivers/md/bcache/closure.h | 7 -------
  1 file changed, 7 deletions(-)
diff --git a/drivers/md/bcache/closure.h b/drivers/md/bcache/closure.h
index c88cdc4ae4ec..35f3f87f74e4 100644
--- a/drivers/md/bcache/closure.h
+++ b/drivers/md/bcache/closure.h
@@ -205,13 +205,6 @@ static inline void closure_set_ip(struct closure *cl)
  #endif
  }
  
-static inline void closure_set_ret_ip(struct closure *cl)
-{
-#ifdef CONFIG_BCACHE_CLOSURES_DEBUG
-	cl->ip = _RET_IP_;
-#endif
-}
-
  static inline void closure_set_waiting(struct closure *cl, unsigned long f)
  {
  #ifdef CONFIG_BCACHE_CLOSURES_DEBUG
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help