From: Nitin Gupta <ngupta@vflare.org>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Greg KH <greg@kroah.com>, Pekka Enberg <penberg@cs.helsinki.fi>,
Minchan Kim <minchan.kim@gmail.com>, Ed Tomlinson <edt@aei.ca>,
Hugh Dickins <hugh.dickins@tiscali.co.uk>, Cyp <cyp561@gmail.com>,
driverdev <devel@driverdev.osuosl.org>,
linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/4] Rename ramzswap files to zram
Date: Sat, 05 Jun 2010 12:30:00 +0530 [thread overview]
Message-ID: <4C09F5F0.6020902@vflare.org> (raw)
In-Reply-To: <20100604123246.3e11022e.akpm@linux-foundation.org>
On 06/05/2010 01:02 AM, Andrew Morton wrote:
> On Tue, 1 Jun 2010 13:31:24 +0530
> Nitin Gupta <ngupta@vflare.org> wrote:
>
>> Related changes:
>> - Modify revelant Kconfig and Makefile accordingly.
>> - Change include filenames in code.
>> - Remove dependency on CONFIG_SWAP in Kconfig as zram usage
>> is no longer limited to swap disks.
>>
>> Signed-off-by: Nitin Gupta <ngupta@vflare.org>
>> ---
>> drivers/staging/Kconfig | 2 +-
>> drivers/staging/Makefile | 2 +-
>> drivers/staging/ramzswap/Kconfig | 21 ---------------
>> drivers/staging/ramzswap/Makefile | 3 --
>> drivers/staging/zram/Kconfig | 28 ++++++++++++++++++++
>> drivers/staging/zram/Makefile | 3 ++
>> drivers/staging/{ramzswap => zram}/xvmalloc.c | 0
>> drivers/staging/{ramzswap => zram}/xvmalloc.h | 0
>> drivers/staging/{ramzswap => zram}/xvmalloc_int.h | 0
>> .../{ramzswap/ramzswap.txt => zram/zram.txt} | 0
>> .../{ramzswap/ramzswap_drv.c => zram/zram_drv.c} | 2 +-
>> .../{ramzswap/ramzswap_drv.h => zram/zram_drv.h} | 2 +-
>> .../ramzswap_ioctl.h => zram/zram_ioctl.h} | 0
>> 13 files changed, 35 insertions(+), 28 deletions(-)
>> delete mode 100644 drivers/staging/ramzswap/Kconfig
>> delete mode 100644 drivers/staging/ramzswap/Makefile
>> create mode 100644 drivers/staging/zram/Kconfig
>> create mode 100644 drivers/staging/zram/Makefile
>> rename drivers/staging/{ramzswap => zram}/xvmalloc.c (100%)
>> rename drivers/staging/{ramzswap => zram}/xvmalloc.h (100%)
>> rename drivers/staging/{ramzswap => zram}/xvmalloc_int.h (100%)
>> rename drivers/staging/{ramzswap/ramzswap.txt => zram/zram.txt} (100%)
>> rename drivers/staging/{ramzswap/ramzswap_drv.c => zram/zram_drv.c} (99%)
>> rename drivers/staging/{ramzswap/ramzswap_drv.h => zram/zram_drv.h} (99%)
>> rename drivers/staging/{ramzswap/ramzswap_ioctl.h => zram/zram_ioctl.h} (100%)
>
> This is a git-specific thing and doesn't do anything useful for people
> who use plain-old-patch(1) (eg: Greg and me).
>
> Obviously we can manually type in the commands and put the massive
> file-moving diff together, but it's a bit of a pain.
>
> Maybe Greg has some magic automated way of turning these into regular
> patches, but I don't.
>
>
I used 'git-format-patch -M' as it should be easier to review
(as suggested by Arnd).
Anyways, I'm was not sure if I should be sending normal (non-git-specific)
version of these patches again. So, uploaded them here instead:
single monolithic patch:
http://compcache.googlecode.com/hg/sub-projects/mainline/zramv2_2.6.35-rc1.patch
or, if you prefer patch series:
http://compcache.googlecode.com/hg/sub-projects/mainline/zramv2_patch_series
Thanks,
Nitin
next prev parent reply other threads:[~2010-06-05 6:59 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-01 8:01 [PATCH 0/4] zram: generic RAM based compressed R/W block devices (v2) Nitin Gupta
2010-06-01 8:01 ` [PATCH 1/4] Support generic I/O requests Nitin Gupta
2010-06-02 6:20 ` Minchan Kim
2010-06-02 7:36 ` Nitin Gupta
2010-06-04 19:19 ` Andrew Morton
2010-06-05 8:28 ` Pekka Enberg
2010-06-05 8:36 ` Pekka Enberg
2010-06-05 8:35 ` Pekka Enberg
2010-06-05 13:15 ` Nitin Gupta
2010-06-06 23:32 ` Minchan Kim
2010-06-07 5:35 ` Pekka Enberg
2010-06-04 19:10 ` Andrew Morton
2010-06-05 7:44 ` Nitin Gupta
2010-06-01 8:01 ` [PATCH 2/4] Rename ramzswap files to zram Nitin Gupta
2010-06-04 19:32 ` Andrew Morton
2010-06-04 21:07 ` Greg KH
2010-06-05 7:00 ` Nitin Gupta [this message]
2010-06-01 8:01 ` [PATCH 3/4] Rename ramzswap to zram in code Nitin Gupta
2010-06-01 8:01 ` [PATCH 4/4] Rename ramzswap to zram in documentation Nitin Gupta
2010-06-01 10:25 ` [PATCH 0/4] zram: generic RAM based compressed R/W block devices (v2) Pekka Enberg
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=4C09F5F0.6020902@vflare.org \
--to=ngupta@vflare.org \
--cc=akpm@linux-foundation.org \
--cc=cyp561@gmail.com \
--cc=devel@driverdev.osuosl.org \
--cc=edt@aei.ca \
--cc=greg@kroah.com \
--cc=hugh.dickins@tiscali.co.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=minchan.kim@gmail.com \
--cc=penberg@cs.helsinki.fi \
/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