>>> On 24.05.18 at 22:26, <sameer.g...@linaro.org> wrote:
> On 05/24/2018 01:57 AM, Jan Beulich wrote:
>>>>> On 24.05.18 at 02:46, <sameer.g...@linaro.org> wrote:
>>> --- /dev/null
>>> +++ b/xen/include/xen/linux_compat.h
>> I continue to dislike the idea of having a header with these contents in 
> this location.
> As explained previously this header can be used for the any driver that 
> we want to port from Linux. This is not arm specific. This seemed like 
> the best location for the file.

Please take into consideration that with lots of code originally having come
from Linux, in all the years we've never had a need to have such a header.
With that history, it needs good reasoning to introduce one now in such a
global fashion.

> Which other common location should I pick?

None - that's the whole point of my comment: I don't see this as something
that should go in a common location.

Jan



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

Reply via email to