Re: [PATCH v2] scsi: target: Save a few cycles in transport_lookup_[cmd|tmr]_lun()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
Subject: Re: [PATCH v2] scsi: target: Save a few cycles in transport_lookup_[cmd|tmr]_lun()
From: Chaitanya Kulkarni <chaitanyak@xxxxxxxxxx>
Date: Tue, 16 Aug 2022 06:06:50 +0000
Accept-language: en-US
Cc: "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, "michael.christie@xxxxxxxxxx" <michael.christie@xxxxxxxxxx>, "kernel-janitors@xxxxxxxxxxxxxxx" <kernel-janitors@xxxxxxxxxxxxxxx>, "linux-scsi@xxxxxxxxxxxxxxx" <linux-scsi@xxxxxxxxxxxxxxx>, "target-devel@xxxxxxxxxxxxxxx" <target-devel@xxxxxxxxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
In-reply-to: <03705222390bfa3b48ad7658f693fc0fc030b3ae.1660596679.git.christophe.jaillet@wanadoo.fr>
References: <03705222390bfa3b48ad7658f693fc0fc030b3ae.1660596679.git.christophe.jaillet@wanadoo.fr>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.0

On 8/15/22 13:52, Christophe JAILLET wrote:
> Use percpu_ref_tryget_live_rcu() instead of percpu_ref_tryget_live() to
> save a few cycles when it is known that the rcu lock is already
> taken/released.
> 
> Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
> ---

do you have a quantitative data that shows actual savings of cycles?

-ck

Follow-Ups:
- Re: [PATCH v2] scsi: target: Save a few cycles in transport_lookup_[cmd|tmr]_lun()
  - From: Christophe JAILLET

References:
- [PATCH v2] scsi: target: Save a few cycles in transport_lookup_[cmd|tmr]_lun()
  - From: Christophe JAILLET

Prev by Date: [PATCH v2] scsi: target: Save a few cycles in transport_lookup_[cmd|tmr]_lun()
Next by Date: Re: [PATCH v2] scsi: target: Save a few cycles in transport_lookup_[cmd|tmr]_lun()
Previous by thread: [PATCH v2] scsi: target: Save a few cycles in transport_lookup_[cmd|tmr]_lun()
Next by thread: Re: [PATCH v2] scsi: target: Save a few cycles in transport_lookup_[cmd|tmr]_lun()
Index(es):
- Date
- Thread

[Index of Archives] [Linux SCSI] [Kernel Newbies] [Linux SCSI Target Infrastructure] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Device Mapper]