Re: [PATCH] xen: Use 'unsigned int' instead of 'unsigned'

[Boris Ostrovsky]

On 3/3/20 3:30 AM, Jan Beulich wrote:
> On 02.03.2020 23:18, Yan Yankovskyi wrote:
>> On Mon, Mar 2, 2020 at 10:11 Jan Beulich wrote:
>>> ... evtchn_port_t here and elsewhere.
>> There are some interfaces with signed int as a type for port, e.g. in
>> include/xen/events.h.
>> Should I create additional patch to resolve inconsistency with evtchn
>> interface?
>> Or you suggest combining these changes into the existing patch?
> Signed <-> unsigned conversions would perhaps better go into a
> separate patch. But note I'm not the maintainer of this code.
>
>> Also as I understand 'evtchn' and 'port' are essentially the same
>> entities from perspective of local domain, related to each other roughly
>> like connection and file descriptor pair. What do you think about
>> renaming all 'evtchn' arguments and variables to 'port'?
>> It will eliminate inconsistencies in the code, for example
>> in include/xen/interface/event_channel.h and include/xen/events.h.
> I'd welcome this, but the maintainers will have the final say.

TBH I'd rather see the things that Jan pointed to fixed than replacing
'unsigned' with 'unsigned int.'

The latter is purely a kernel style requirement and is typically fixed
when the line is modified (in fact, we did just that last week).

-boris