public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jon Masters <jonathan@jonmasters.org>
To: Mark Rutland <mark.rutland@arm.com>, Xishi Qiu <qiuxishi@huawei.com>
Cc: Steve Capper <steve.capper@linaro.org>,
	zhong jiang <zhongjiang@huawei.com>,
	LKML <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	catalin.marinas@arm.com, Jim Perrin <jperrin@centos.org>
Subject: Re: [RFC] arm64: failed when run the command: timedatectl set-timezone Asia/Shanghai
Date: Mon, 18 Jan 2016 01:23:50 -0600	[thread overview]
Message-ID: <569C9306.7090904@jonmasters.org> (raw)
In-Reply-To: <20160113110900.GA23370@leverpostej>

Hi Mark, Xishi,

On 1/13/16, 5:09 AM, Mark Rutland wrote:
> On Wed, Jan 13, 2016 at 09:16:43AM +0800, Xishi Qiu wrote:
>> On 2016/1/12 18:59, Steve Capper wrote:
>>
>>> On 12 January 2016 at 03:32, Xishi Qiu <qiuxishi@huawei.com> wrote:
>>>> On 2016/1/12 10:47, Xishi Qiu wrote:
>>>>
>>>>> Failed when run the command: timedatectl set-timezone Asia/Shanghai
>>>>> But CONFIG_PGTABLE_LEVELS=3 is OK, and CONFIG_PGTABLE_LEVELS=4 is failed.
>>>>> The kernel is v4.1, and this command need the lib polikit.
>>>>>
>>>>> Is this the bug of kernel?

As noted by others, this is a problem with interpretation by certain 
userspace (probably not restricted to mozjs) applications of their 
ability to use the high order bits of a VA. It's actually been flagged a 
few times by others in both RHEL(SA) and Fedora as well, but I've 
pointed out (at least in the former) that we're not building with a 
48-bit VA currently so it won't be as visible for the moment. That means 
if you're consuming a Cent7 kernel from Jim/etc. it'll probably not fall 
over out of the box (copying him so he knows about this - it's been 
coming up a bit over the past few days in some other conversation).

This definitely needs fixing in upstream polkit builds, and a general 
education exercise needs to be initiated so that people don't get this 
wrong in future (but it's guaranteed that they will). FWIW my personal 
opinion is that tagged pointers are dangerous and should never have 
happened (because you can't trust people/there will be code out there 
for years that will have broken assumptions and never be fixed).

Jon.

      parent reply	other threads:[~2016-01-18  7:24 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-12  2:47 [RFC] arm64: failed when run the command: timedatectl set-timezone Asia/Shanghai Xishi Qiu
2016-01-12  3:32 ` Xishi Qiu
2016-01-12 10:59   ` Steve Capper
2016-01-13  1:16     ` Xishi Qiu
2016-01-13 11:09       ` Mark Rutland
2016-01-13 17:00         ` Christopher Covington
2016-01-13 18:47           ` Mark Rutland
2016-01-14  1:48         ` Xishi Qiu
2016-01-14 10:34           ` Mark Rutland
2016-01-18  7:23         ` Jon Masters [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=569C9306.7090904@jonmasters.org \
    --to=jonathan@jonmasters.org \
    --cc=catalin.marinas@arm.com \
    --cc=jperrin@centos.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=qiuxishi@huawei.com \
    --cc=steve.capper@linaro.org \
    --cc=zhongjiang@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox