Re: [PATCH 1/2] staging: android: ion: Remove file ion_carveout_heap.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jul 03, 2019 at 07:32:27AM -0400, Laura Abbott wrote:
> On 7/3/19 5:50 AM, Daniel Vetter wrote:
> > On Wed, Jul 3, 2019 at 10:37 AM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > > 
> > > On Wed, Jul 03, 2019 at 01:48:41PM +0530, Nishka Dasgupta wrote:
> > > > Remove file ion_carveout_heap.c as its functions and definitions are not
> > > > used anywhere.
> > > > Issue found with Coccinelle.
> > > > 
> > > > Signed-off-by: Nishka Dasgupta <nishkadg.linux@xxxxxxxxx>
> > > > ---
> > > >   drivers/staging/android/ion/Kconfig           |   9 --
> > > >   drivers/staging/android/ion/Makefile          |   1 -
> > > >   .../staging/android/ion/ion_carveout_heap.c   | 133 ------------------
> > > 
> > > I keep trying to do this, but others point out that the ion code is
> > > "going to be fixed up soon" and that people rely on this interface now.
> > > Well, "code outside of the kernel tree" relies on this, which is not ok,
> > > but the "soon" people keep insisting on it...
> > > 
> > > Odds are I should just delete all of ION, as there hasn't been any
> > > forward progress on it in a long time.
> > > 
> > > Hopefully that wakes some people up...
> > 
> > John Stultz has done a steady stream on ion destaging patch series
> > past few months, und the heading of "DMA-BUF Heaps", targeting
> > drivers/dma-buf. I'm not following the details, and it seems a bit a
> > crawl, but there's definitely work going on ... Just probably not
> > in-place in staging I think.
> > -Daniel
> > 
> 
> 
> https://lists.freedesktop.org/archives/dri-devel/2019-June/223705.html
> 
> It is making slow and steady progress. Part of this is trying to
> make sure we actually get this right before moving anything
> out of staging.
> 
> That said, I think we're at the point where nobody wants the
> carveout and chunk heaps so I'd actually be okay with removing
> those files. Just to be explicit:
> 
> Acked-by: Laura Abbott <labbott@xxxxxxxxxx>

Hey, if you agree to delete these, I'll not object at all!

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux