[Devel] [PATCH VZ9 1/4] fs/fuse kio: remove unused PCS_IREQ_CUSTOM handling

Konstantin Khorenko khorenko at virtuozzo.com
Wed Nov 12 20:56:25 MSK 2025


Liu, please use branch-rh9-5.14.0-427.92.1.vz9.88.x-ovz for vz9 development
and do not forget to send patches for vz10 kernel branch-rh10-6.12.0-55.13.1.2.x.vz10-ovz as well if 
needed.

i will handle this series.

--
Best regards,

Konstantin Khorenko,
Virtuozzo Linux Kernel Team

On 11/11/25 03:02, Liu Kui wrote:
> The PCS_IREQ_CUSTOM request type is not used in the kio module,
> remove the associated dead code.
> 
> Related to: #VSTOR-117505
> 
> Signed-off-by: Liu Kui <kui.liu at virtuozzo.com>
> ---
>   fs/fuse/kio/pcs/pcs_cluster.c | 32 ++++++++++++--------------------
>   1 file changed, 12 insertions(+), 20 deletions(-)
> 
> diff --git a/fs/fuse/kio/pcs/pcs_cluster.c b/fs/fuse/kio/pcs/pcs_cluster.c
> index 7e878d7befc3..7aaf88b8bd26 100644
> --- a/fs/fuse/kio/pcs/pcs_cluster.c
> +++ b/fs/fuse/kio/pcs/pcs_cluster.c
> @@ -48,15 +48,13 @@ void pcs_sreq_complete(struct pcs_int_request *sreq)
>   			if (ireq_check_redo(sreq)) {
>   				ireq_retry_inc(ireq);
>   				sreq->flags &= ~IREQ_F_REQUEUED;
> -				if (sreq->type != PCS_IREQ_CUSTOM) {
> -					map_notify_soft_error(sreq);
> -
> -					if (!(sreq->flags & IREQ_F_ONCE)) {
> -						sreq->flags |= IREQ_F_ONCE;
> -						pcs_clear_error(&sreq->error);
> -						pcs_cc_submit(sreq->cc, sreq);
> -						return;
> -					}
> +				map_notify_soft_error(sreq);
> +
> +				if (!(sreq->flags & IREQ_F_ONCE)) {
> +					sreq->flags |= IREQ_F_ONCE;
> +					pcs_clear_error(&sreq->error);
> +					pcs_cc_submit(sreq->cc, sreq);
> +					return;
>   				}
>   				pcs_clear_error(&sreq->error);
>   				ireq_delay(sreq);
> @@ -65,16 +63,13 @@ void pcs_sreq_complete(struct pcs_int_request *sreq)
>   			pcs_copy_error_cond_atomic(&ireq->error, &sreq->error);
>   		}
>   
> -		if (sreq->type != PCS_IREQ_CUSTOM)
> -			map_notify_iochunk_error(sreq);
> +		map_notify_iochunk_error(sreq);
>   	}
>   
> -	if (sreq->type != PCS_IREQ_CUSTOM) {
> -		if (!(sreq->flags & IREQ_F_CACHED))
> -			ireq->flags &= ~IREQ_F_CACHED;
> -		pcs_deaccount_ireq(sreq, &sreq->error);
> -	} else if (sreq->custom.destruct)
> -		sreq->custom.destruct(sreq);
> +
> +	if (!(sreq->flags & IREQ_F_CACHED))
> +		ireq->flags &= ~IREQ_F_CACHED;
> +	pcs_deaccount_ireq(sreq, &sreq->error);
>   
>   	if (!pcs_sreq_detach(sreq))
>   		ireq_complete(ireq);
> @@ -489,9 +484,6 @@ static void ireq_process_(struct pcs_int_request *ireq)
>   	case PCS_IREQ_FLUSH:
>   		process_flush_req(ireq);
>   		break;
> -	case PCS_IREQ_CUSTOM:
> -		ireq->custom.action(ireq);
> -		break;
>   	case PCS_IREQ_TOKEN:
>   		process_ireq_token(ireq);
>   		break;



More information about the Devel mailing list