Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-12 Thread Jan Beulich
On 12.02.2024 16:38, Roger Pau Monné wrote: > On Mon, Feb 12, 2024 at 11:45:33AM +0100, Jan Beulich wrote: >> On 12.02.2024 10:39, Roger Pau Monné wrote: >>> On Thu, Feb 08, 2024 at 04:49:46PM +0100, Jan Beulich wrote: On 08.02.2024 12:49, Roger Pau Monné wrote: > On Mon, Feb 05, 2024 at

Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-12 Thread Roger Pau Monné
On Mon, Feb 12, 2024 at 11:45:33AM +0100, Jan Beulich wrote: > On 12.02.2024 10:39, Roger Pau Monné wrote: > > On Thu, Feb 08, 2024 at 04:49:46PM +0100, Jan Beulich wrote: > >> On 08.02.2024 12:49, Roger Pau Monné wrote: > >>> On Mon, Feb 05, 2024 at 02:55:43PM +0100, Jan Beulich wrote: >

Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-12 Thread Jan Beulich
On 12.02.2024 10:39, Roger Pau Monné wrote: > On Thu, Feb 08, 2024 at 04:49:46PM +0100, Jan Beulich wrote: >> On 08.02.2024 12:49, Roger Pau Monné wrote: >>> On Mon, Feb 05, 2024 at 02:55:43PM +0100, Jan Beulich wrote: Make the variable a tristate, with (as done elsewhere) a negative value

Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-12 Thread Roger Pau Monné
On Thu, Feb 08, 2024 at 04:49:46PM +0100, Jan Beulich wrote: > On 08.02.2024 12:49, Roger Pau Monné wrote: > > On Mon, Feb 05, 2024 at 02:55:43PM +0100, Jan Beulich wrote: > >> Make the variable a tristate, with (as done elsewhere) a negative value > >> meaning "default". Since all use sites need

Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-08 Thread Jan Beulich
On 08.02.2024 12:49, Roger Pau Monné wrote: > On Mon, Feb 05, 2024 at 02:55:43PM +0100, Jan Beulich wrote: >> Make the variable a tristate, with (as done elsewhere) a negative value >> meaning "default". Since all use sites need looking at, also rename it >> to match our usual "opt_*" pattern.

Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-08 Thread Roger Pau Monné
On Mon, Feb 05, 2024 at 02:55:43PM +0100, Jan Beulich wrote: > Make the variable a tristate, with (as done elsewhere) a negative value > meaning "default". Since all use sites need looking at, also rename it > to match our usual "opt_*" pattern. While touching it, also move it to >

[PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-05 Thread Jan Beulich
Make the variable a tristate, with (as done elsewhere) a negative value meaning "default". Since all use sites need looking at, also rename it to match our usual "opt_*" pattern. While touching it, also move it to .data.ro_after_init. The only place it retains boolean nature is pci_ats_device(),