On 2017年10月25日 15:43, Roger Pau Monné wrote:
> On Wed, Oct 25, 2017 at 03:30:39PM +0800, Lan Tianyu wrote:
>> On 2017年10月19日 23:42, Roger Pau Monné wrote:
>>> On Thu, Sep 21, 2017 at 11:02:01PM -0400, Lan Tianyu wrote:
>>>
>>>>
>>>> struct viommu_ops {
>>>> @@ -28,6 +29,9 @@ struct viommu_ops {
>>>> int (*destroy)(struct viommu *viommu);
>>>> int (*handle_irq_request)(struct domain *d,
>>>> struct arch_irq_remapping_request *request);
>>>> + int (*get_irq_info)(struct domain *d,
>>>> + struct arch_irq_remapping_request *request,
>>>
>>> AFAICT d and request should be constified.
>>
>> Did you mean to keep d and request in the same line? This will exceed 80
>> chars.
>
> No, I meant that the parameters of the function should be "const struct
> domain *d" and "const struct arch_irq_remapping_request *request".
> AFAICT you should never modify them inside of get_irq_info.
>
OK. I got it. This makes sense and will update.
--
Best regards
Tianyu Lan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
