Re: [Xen-devel] [PATCH] Remove redundant code in branch MAP_PIRQ_TYPE_MSI

[Joe Jin <joe.jin@xxxxxxxxxx>]

Looks good for me.

Reviewed-by: Joe Jin <joe.jin@xxxxxxxxxx>

On 10/09/2017 02:00 PM, Zhenzhong Duan wrote:
> Same code is already in allocate_and_map_msi_pirq()
> 
> Signed-off-by: Zhenzhong Duan <zhenzhong.duan@xxxxxxxxxx>
> ---
>  xen/arch/x86/physdev.c |    2 --
>  1 files changed, 0 insertions(+), 2 deletions(-)
> 
> diff --git a/xen/arch/x86/physdev.c b/xen/arch/x86/physdev.c
> index 0eb4097..aa02d18 100644
> --- a/xen/arch/x86/physdev.c
> +++ b/xen/arch/x86/physdev.c
> @@ -122,8 +122,6 @@ int physdev_map_pirq(domid_t domid, int type, int *index, 
> int *pirq_p,
>          break;
>  
>      case MAP_PIRQ_TYPE_MSI:
> -        if ( !msi->table_base )
> -            msi->entry_nr = 1;
>          /* fallthrough */
>      case MAP_PIRQ_TYPE_MULTI_MSI:
>          ret = allocate_and_map_msi_pirq(d, *index, pirq_p, type, msi);
> 


-- 
Oracle <http://www.oracle.com>
Joe Jin | Software Development Director | +8610.6106.5624
ORACLE | Linux and Virtualization
No. 24 Zhongguancun Software Park, Haidian District | 100193 Beijing 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel