From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Date: Wed, 07 Oct 2020 09:28:58 +0000 Subject: Re: [PATCH 1/3] fbdev/sbuslib: remove FBIOSCURSOR/FBIOGCURSOR leftovers Message-Id: List-Id: References: <20201007074447.797968-1-hch@lst.de> <20201007074447.797968-2-hch@lst.de> <20201007085911.GA17356@lst.de> In-Reply-To: <20201007085911.GA17356@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Christoph Hellwig Cc: Geert Uytterhoeven , "David S. Miller" , Bartlomiej Zolnierkiewicz , linux-m68k , sparclinux , Linux Fbdev development list , Sam Ravnborg On Wed, Oct 7, 2020 at 10:59 AM Christoph Hellwig wrote: > On Wed, Oct 07, 2020 at 10:54:19AM +0200, Arnd Bergmann wrote: > > On Wed, Oct 7, 2020 at 9:44 AM Christoph Hellwig wrote: > > > > > > There are no actual implementations of FBIOSCURSOR/FBIOGCURSOR left, so > > > don't bother with a compat handler for it, and remove the remaining > > > definitions as well. > > > > > > Signed-off-by: Christoph Hellwig > > > > I had submitted a similar patch earlier, and Sam Ravnborg applied it to the > > drm-misc tree, but it doesn't seem to be in linux-next, so I don't know > > what the state is. > > > > My version only removed the compat handling, not the data structures, > > so I'm happy to see your version used instead if mine got lost. > > Oh, sorry. I thought in your summary you decided to give up on > the sbuslib ones. Here is what I have pending at the moment for set_fs() and compat_alloc_user_space(): https://git.kernel.org/pub/scm/linux/kernel/git/arnd/playground.git/log/?h=compat-alloc-user-space-2 The only one I have actually given up on is the atomisp staging driver, which uses an awful hack, copying the x86 implementation of copy_in_user()/compat_alloc_user_space() into the driver. This is based on last week's linux-next, as I plan to resubmit after the merge window. Arnd