Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 2 Aug 2017 13:11:48 +0200
From:      Hans Petter Selasky <hps@selasky.org>
To:        Konstantin Belousov <kostikbel@gmail.com>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r321920 - head/sys/sys
Message-ID:  <760e0077-499e-fffb-a86b-979e410b35e1@selasky.org>
In-Reply-To: <20170802110607.GC1700@kib.kiev.ua>
References:  <201708021014.v72AEHEk061037@repo.freebsd.org> <7b1d274f-a531-f788-742a-a8375fe76763@selasky.org> <20170802110607.GC1700@kib.kiev.ua>

next in thread | previous in thread | raw e-mail | index | archive | help
On 08/02/17 13:06, Konstantin Belousov wrote:
> So linuxkpi was broken before this commit as well, as I noted in my
> other reply ?

Hi,

The LinuxKPI uses minor/major/makedev internally.

y = makedev(z,t)

Then the LinuxKPI assumes z and t can be retrieved through minor and 
major: minor(y) == z and major(y) == t

If "y" is an "int" and it is assigned a 64-bit value, it is obvious that 
this code no longer holds true. Same might be in userspace/ports.

So yes, this change breaks the LinuxKPI, at least some parts of it.

--HPS



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?760e0077-499e-fffb-a86b-979e410b35e1>