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

Re: [Xen-devel] [PATCH] pci: clear host_maskall field on assign


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Tue, 8 Oct 2019 15:14:25 +0200
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none; spf=None smtp.pra=roger.pau@xxxxxxxxxx; spf=Pass smtp.mailfrom=roger.pau@xxxxxxxxxx; spf=None smtp.helo=postmaster@xxxxxxxxxxxxxxx
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Stanislav Spassov <stanspas@xxxxxxxxx>, Chao Gao <chao.gao@xxxxxxxxx>
  • Delivery-date: Tue, 08 Oct 2019 13:14:47 +0000
  • Ironport-sdr: ftRRPaAf4dLRrO/sWsl7/zuVT2DIQZ8M7/ubLXKQRR9s0UVTY8dyVLhC/aUkDfIuqM8g0L3rmr XH/G/Thg1LxIdERkhcNA0Uf8oIQ3ibV0mh9pMdEHX//xnjZ7ZNoQXax04OV87R84zSVVNvofm+ b6Y4zWV6z1N/T+bWtsmhuY0X6MBizEpA6P6RZvMU7NUTY7yP7+WBMjNMI2920oafRSH+DGZ5Eh V2dLOyZLlHl+jVLC7AVSQ8ELRq8W3pf7u4OaSYLrRJDpxPmd8SGcv9gpZvNJyrJGGhLDrOSEIE ToU=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Tue, Oct 08, 2019 at 01:28:49PM +0200, Jan Beulich wrote:
> On 08.10.2019 13:09, Roger Pau Monné  wrote:
> > Given that as you correctly point out maskall is unset after device
> > reset, I feel that option 4 is the best one since it matches the state
> > of the hardware after reset.
> 
> Right, that's the variant coming closest to what hardware state
> ought to be at that point. We'd need to double check that the
> per-entry mask bits are all set at that point.

I'm not saying such check is not worth doing, but why do it in this
case but not when also clearing the maskall (in msix_capability_init)
when called from prepare_msix?

Thanks, Roger.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.