Re: [PATCH v2 4/5] soc: qcom: Remote FS memory driver

From	Christoph Hellwig <hch@infradead.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 4/5] soc: qcom: Remote FS memory driver
Date	2017-08-11 12:10 +0200
Message-ID	<udf3c-2bo-1@gated-at.bofh.it> (permalink)
References	<uaewF-224-7@gated-at.bofh.it> <uaewH-224-31@gated-at.bofh.it> <ub42d-3iq-3@gated-at.bofh.it> <ubT4L-2E7-45@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Mon, Aug 07, 2017 at 09:23:27AM -0700, Bjorn Andersson wrote:
> Up until this patch the user space tool that implements the message
> handler just mapped the reserved memory region though /dev/mem, but this
> requires /dev/mem access and for the later platforms we need to make a
> call into TrustZone to provide the remote permission to access this
> memory region. So we need some code in the kernel to represent this.
> 
> > Else: NAK as the scheme looks completely brain dead and bonkers.
> 
> I can rework the commit message in an attempt to better explain the
> setup and hope you/people find it slightly less bonkers.
> 
> Does this sound reasonable?

Please rework the commit message.  And it should probably grow a saner
module name, too.

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/5] Qualcomm remote filesystem shared memory driver Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-08-03 05:00 +0200
  [PATCH v2 4/5] soc: qcom: Remote FS memory driver Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-08-03 05:00 +0200
    Re: [PATCH v2 4/5] soc: qcom: Remote FS memory driver Christoph Hellwig <hch@infradead.org> - 2017-08-05 12:00 +0200
      Re: [PATCH v2 4/5] soc: qcom: Remote FS memory driver Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-08-07 18:30 +0200
        Re: [PATCH v2 4/5] soc: qcom: Remote FS memory driver Christoph Hellwig <hch@infradead.org> - 2017-08-11 12:10 +0200

csiph-web