[lvc-project] [PATCH 4/4] gntdev: Unchecked finction result

[Fedor Pchelkin]

Здравствуйте!

On Tue, 02. Apr 14:51, Andrey Shumilin wrote:
> The dma_coerce_mask_and_coherent function may refuse
> to apply a mask due to incorrectly passed parameters.

Желательно бы чуть подробнее раскрыть, каковыми могут быть некорректные
параметры. Может быть эти данные где-то раньше по коду проверялись?
Такой вопрос могут задать сопровождающие xen-подсистемы.

> But since this function returns the error -EIO, this
> is an abnormal situation and the error must be written to the log.
> Do not return an error from the function.

Спасибо, что учли предыдущие замечания.

Однако тема письма, к сожалению, по-прежнему неправильна!

[PATCH 4/4] означает четвертый патч в серии из четырех патчей.
В данном случае должно быть просто [PATCH].

Что такое "finction"?

Название патча следует продумать получше, чтобы оно по крайней мере
отражало, у какой функции предлагается проверять возвращаемое значение.

> 
> Found by Linux Verification Center (linuxtesting.org) with SVACE.
> 
> Signed-off-by: Andrey Shumilin <shum.sdl at nppct.ru>
>
> ---

Пустую строку между последним Signed-off-by и `---` желательно убрать.

>  drivers/xen/gntdev.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/xen/gntdev.c b/drivers/xen/gntdev.c
> index 61faea1f0663..8a448dbbca86 100644
> --- a/drivers/xen/gntdev.c
> +++ b/drivers/xen/gntdev.c
> @@ -597,7 +597,8 @@ static int gntdev_open(struct inode *inode, struct file *flip)
>  	flip->private_data = priv;
>  #ifdef CONFIG_XEN_GRANT_DMA_ALLOC
>  	priv->dma_dev = gntdev_miscdev.this_device;
> -	dma_coerce_mask_and_coherent(priv->dma_dev, DMA_BIT_MASK(64));
> +	if (dma_coerce_mask_and_coherent(priv->dma_dev, DMA_BIT_MASK(64)) == -EIO)
> +		pr_err("The mask was applied unsuccessfully");
>  #endif
>  	pr_debug("priv %p\n", priv);
>  
> -- 
> 2.30.2