From: Jaya K. <jay...@gm...> - 2009-06-25 18:08:27
|
On Thu, Jun 25, 2009 at 2:06 PM, Magnus Damm<mag...@gm...> wrote: > > The code is fbmem.c is currently filled with #ifdefs today, want me > create inline versions for fb_deferred_io_open() and > fb_deferred_io_fsync() as well? > The patch looks good. I was going to suggest that it might be attractive to use __attribute__(weak) for each of the dummy functions instead of ifdefs in this case, but I can't remember if there was a consensus about attribute-weak versus ifdefs. Thanks, jaya |