On Mon, 10 Sep 2012 16:31:59 -0700 Kent Overstreet
wrote:
> cc'ing Neil
>
> On Mon, Sep 10, 2012 at 04:06:54PM -0700, Tejun Heo wrote:
> > Hello, again.
> >
> > cc'ing Kent and Vivek. The original thread is at
> >
> > http://thread.gmane.org/gmane.linux.network.drbd.devel/2130
> >
> > On
On Thu, Aug 30, 2012 at 2:26 AM, Mike Turquette wrote:
> Quoting Chao Xie (2012-08-26 18:19:02)
>> On Sat, Aug 25, 2012 at 8:21 AM, Mike Turquette
>> wrote:
>> > Quoting Chao Xie (2012-08-19 19:55:10)
>> >> From: Chao Xie
>> >> arch/arm/mach-mmp/Kconfig|3 +
>> >> drivers/clk/Makefile
On Tue, Sep 4, 2012 at 6:38 PM, Hebbar, Gururaja wrote:
> From: Vaibhav Bedia
>
> In some cases mmc_suspend_host() is not able to claim the
> host and proceed with the suspend process. The core returns
> -EBUSY to the host controller driver. Unfortunately, the
> host controller driver does not
On 10.09.2012 22:08, Kurt Van Dijck wrote:
> On Mon, Sep 10, 2012 at 09:01:21PM +0200, Oliver Hartkopp wrote:
> + if (msg == NETDEV_CHANGENAME) {
>>
>>
>> I do have a second remark.
>>
>> What about always using the notifiers to name the LED triggers?
>>
>> Looking into
>>
>>
Hi Tim,
On Mon, Sep 10, 2012 at 09:19:20AM -0700, Tim Chen wrote:
> This is the second version of the patch series. Thanks to Matthew Wilcox
> for many valuable suggestions on improving the patches.
>
> To do page reclamation in shrink_page_list function, there are two
> locks taken on a page
At 09/11/2012 01:18 PM, Jerry Wrote:
> Hi Kim,
>
> Thank you for your kindness. Let me clarify this:
>
> On ARM architecture, there are 32 bits physical addresses space. However,
> the addresses space is divided into 8 banks normally. Each bank
> disabled/enabled by a chip selector signal. In my
The ACPI spec doesn't provide for a way for the bios to pass down
recommended thresholds to the OS on a _per-bank_ basis. This patch adds
a new boot option, which if passed, allows bios to initialize the CMCI
threshold. In such a case, we simply skip programming any threshold
value.
As fail-safe,
On Mon, Sep 10, 2012 at 09:50:22PM +0200, Rafael J. Wysocki wrote:
> drivers/acpi/bus.c | 11 ++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
> Index: linux/drivers/acpi/bus.c
> ===
> ---
On Mon, Sep 10, 2012 at 09:19:29AM -0700, Tim Chen wrote:
> We reorganize the page_referenced and try_to_unmap code to determine
> explicitly if mapping->i_mmap_mutex needs to be acquired. This allows
> us to acquire the mutex for multiple pages in batch. We can call
> __page_referenced or
On Tue, Sep 11, 2012 at 5:21 AM, Sachin Kamat wrote:
> Path to gpio.txt (dt) document was broken.
>
> Signed-off-by: Sachin Kamat
Applied, thanks!
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More
On Mon, 10 Sep 2012 17:22:11 -0700 Kent Overstreet
wrote:
> Random assortment of refactoring and trivial cleanups;
>
> Immutable bio vecs and efficient bio splitting require auditing and
> removing pretty much all bi_idx uses, among other things.
>
> The reason is that with immutable bio vecs
Hi,
On 09/10/2012 08:38 AM, Wang Sheng-Hui wrote:
To check the duplicated super blocks, use BTRFS_SUPER_MIRROR_MAX
as the loops limit.
Signed-off-by: Wang Sheng-Hui
---
fs/btrfs/disk-io.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/btrfs/disk-io.c
Hi Rafael,
I've been investigating runtime PM support for some use-cases on GPUs.
In some laptops we have a secondary GPU (optimus) that can be powered
up for certain 3D tasks and then turned off when finished with. Now I
did an initial pass on supporting it without using the kernel runtime
PM
Hi. Wu.
I will update documentation as per your suggestion.
Thank you.
2012/9/11, Fengguang Wu :
> On Mon, Aug 27, 2012 at 12:01:41PM -0400, Namjae Jeon wrote:
>> From: Namjae Jeon
>>
>> This commit adds dirty_background_time description in bdi sysfs
>> documentation.
>>
>> Signed-off-by:
On Mon, 10 Sep 2012 17:22:24 -0700 Kent Overstreet
wrote:
> Had to shuffle the code around a bit (where bi_rw and bi_end_io were
> set), but shouldn't really be anything tricky here
>
> Signed-off-by: Kent Overstreet
> CC: Jens Axboe
> CC: NeilBrown
> ---
> drivers/md/raid5.c | 28
2012/9/11, Fengguang Wu :
> Hi Namjae,
Hi. Wu.
>
> On Mon, Aug 27, 2012 at 12:01:14PM -0400, Namjae Jeon wrote:
>> From: Namjae Jeon
>>
>> This patch is based on suggestion by Wu Fengguang:
>> https://lkml.org/lkml/2011/8/19/19
>>
>> kernel has mechanism to do writeback as per dirty_ratio and
>>
On Mon, Sep 10, 2012 at 11:51:48PM +0200, Lars-Peter Clausen wrote:
> On 09/10/2012 02:05 PM, Thierry Reding wrote:
> > This commit moves the driver to drivers/pwm and converts it to the new
> > PWM framework.
> >
> > Signed-off-by: Thierry Reding
>
> Seems to work, thanks a lot. This one and
On Mon, Sep 10, 2012 at 22:29:21, Cousson, Benoit wrote:
> On 09/10/2012 06:52 PM, Matt Porter wrote:
> > On Mon, Sep 10, 2012 at 06:34:20PM +0200, Benoit Cousson wrote:
> >> + Tony
> >>
> >> Hi Matt,
> >>
> >> 30 minutes too late for my pull request :-(
> >>
> >> There are a couple of am33xx
On Mon, 10 Sep 2012 17:22:23 -0700 Kent Overstreet
wrote:
> I couldn't figure out what sbio->bi_end_io in process_checks() was
> supposed to be, so I took the easy way out.
Almost.
You save 'sbio->bi_end_io' to 'bi_end_io', then do nothing with it...
A little way above the 'fixup the bio for
On Mon, Sep 10, 2012 at 21:50:20, Porter, Matt wrote:
> On AM33xx, the datasheet and TRM refer to four GPIO instances that
> are 0-based, GPIO0-3.
>
Thanks Matt,
I think Anil labeled it as gpio1-4 due to hwmod naming convention, as you
can not have gpioo id = 0 (refer to
From: Wei WANG
Realtek PCI-E SD/MMC card host driver is used to access SD/MMC card,
with the help of Realtek PCI-E card reader MFD driver.
Signed-off-by: Wei WANG
Reviewed-by: Arnd Bergmann
Tested-by: Borislav Petkov
---
drivers/mmc/host/Kconfig |7 +
drivers/mmc/host/Makefile
From: Wei WANG
Realtek PCI-E Memstick card host driver is used to access Memstick
card, with the help of Realtek PCI-E card reader MFD driver.
Signed-off-by: Wei WANG
---
drivers/memstick/host/Kconfig | 10 +
drivers/memstick/host/Makefile |1 +
From: Wei WANG
Support for Realtek PCI-Express driver-based card readers including rts5209 and
rts5229.
v2:
1. Using platform device to replace realtek slot bus
v3:
1. Fix a bug that DMA out of SW-IOMMU space in Lenovo Thinkpad x121e
2. Tested by Borislav Petkov
v4:
1. Fulfill power
On Thu, 6 Sep 2012 16:48:05 -0700 Tejun Heo wrote:
> Hello, guys.
>
> (cc'ing Jens, Alasdair and Neil)
>
> On Thu, Sep 06, 2012 at 03:34:54PM -0700, Kent Overstreet wrote:
> > Screwed up the bio_reset() patch in the last patch series when I went to
> > edit
> > the description, fixed that
On Tue, Sep 11, 2012 at 05:01:09AM +0200, Andi Kleen wrote:
> >
> > -c should be kept because it means --stdout and --keep
>
> It seems to work without it.
Yeah I see: it's used in the pipe anyway:
tar cf - . $opts | ${compress} > "${tarball}${file_ext}"
Thanks,
Fengguang
--
To
Hi Linus:
This push fixes the authenc self-test crash as well as a missing
export of a symbol used by a module.
Please pull from
git://git.kernel.org/pub/scm/linux/kernel/git/herbert/crypto-2.6.git
or
master.kernel.org:/pub/scm/linux/kernel/git/herbert/crypto-2.6.git
Ben Collins (1):
On Mon, 2012-09-10 at 20:44 -0700, Greg KH wrote:
> On Mon, Sep 10, 2012 at 08:40:11PM -0700, Joe Perches wrote:
> > On Mon, 2012-09-10 at 19:52 -0700, Greg KH wrote:
> > > On Tue, Sep 11, 2012 at 04:50:15AM +0200, Bruce Humphrey wrote:
> > > > Change 5 instances of printk(KERN_INFO, KERN_WARN and
On Mon, Sep 10, 2012 at 08:40:11PM -0700, Joe Perches wrote:
> On Mon, 2012-09-10 at 19:52 -0700, Greg KH wrote:
> > On Tue, Sep 11, 2012 at 04:50:15AM +0200, Bruce Humphrey wrote:
> > > Change 5 instances of printk(KERN_INFO, KERN_WARN and KERN_ERROR for the
> > > prefered pr_info, pr_warn and
On 09/11/2012 02:49 AM, Al Viro wrote:
On Mon, Sep 10, 2012 at 11:40:11PM +1000, Greg Ungerer wrote:
Hi Al,
On 09/08/2012 04:20 AM, Al Viro wrote:
To architecture maintainers: please, review the current
situation in git.kernel.org/pub/scm/linux/kernel/git/viro/signal #execve2
and
On Mon, 2012-09-10 at 19:52 -0700, Greg KH wrote:
> On Tue, Sep 11, 2012 at 04:50:15AM +0200, Bruce Humphrey wrote:
> > Change 5 instances of printk(KERN_INFO, KERN_WARN and KERN_ERROR for the
> > prefered pr_info, pr_warn and pr_error
[]
> > diff --git a/drivers/staging/comedi/drivers/fl512.c
>
Path to gpio.txt (dt) document was broken.
Signed-off-by: Sachin Kamat
---
Documentation/devicetree/bindings/gpio/led.txt |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/Documentation/devicetree/bindings/gpio/led.txt
b/Documentation/devicetree/bindings/gpio/led.txt
The freescale V2 SATA controller checks if the received data length matches
the programmed length 'ttl', if not, it assumes that this is an error.
In ATAPI, the 'ttl' is based on max allocation length and not the actual
data transfer length, controller will raise 'DLM' (Data length Mismatch)
error
On 09/11/2012 09:37 AM, Yasuaki Ishimatsu wrote:
> Hi Lai,
>
> 2012/09/11 10:22, Lai Jiangshan wrote:
>> On 09/11/2012 08:40 AM, Yasuaki Ishimatsu wrote:
>>> Hi Lai,
>>>
>>> Using memory_online to hot-added node's memory, the following kernel
>>> messages
>>> were shown. Is this a known issue?
>
> -c should be kept because it means --stdout and --keep
It seems to work without it.
-Andi
--
a...@linux.intel.com -- Speaking for myself only.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo
On Tue, Sep 11, 2012 at 04:50:15AM +0200, Bruce Humphrey wrote:
> Change 5 instances of printk(KERN_INFO, KERN_WARN and KERN_ERROR for the
> prefered pr_info, pr_warn and pr_error
>
> Signed-off-by: Bruce Humphrey Ventura
> ---
> drivers/staging/comedi/drivers/fl512.c | 10 +-
> 1
On 09/08/2012 04:39 PM, Pekka Enberg wrote:
> On Fri, Sep 7, 2012 at 1:29 AM, Paul E. McKenney
> wrote:
>> On Thu, Sep 06, 2012 at 11:05:11AM +0800, Michael Wang wrote:
>>> On 09/05/2012 09:55 PM, Christoph Lameter wrote:
On Wed, 5 Sep 2012, Michael Wang wrote:
> Since the cachep
At 09/10/2012 04:58 PM, Lai Jiangshan Wrote:
> Currently memory_hotplug only manages the node_states[N_HIGH_MEMORY],
> it forgot to manage node_states[N_NORMAL_MEMORY]. fix it.
>
> Add check_nodemasks_changes_online() and check_nodemasks_changes_offline
> to detect do node_states[N_HIGH_MEMORY]
Change 5 instances of printk(KERN_INFO, KERN_WARN and KERN_ERROR for the
prefered pr_info, pr_warn and pr_error
Signed-off-by: Bruce Humphrey Ventura
---
drivers/staging/comedi/drivers/fl512.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git
Hi, ishimatsu
At 09/05/2012 05:25 PM, we...@cn.fujitsu.com Wrote:
> From: Yasuaki Ishimatsu
>
> If system supports memory hot-remove, online_pages() may online removed pages.
> So online_pages() need to check whether onlining pages are present or not.
Because we use memory_block_change_state()
On Fri, Sep 07, 2012 at 11:00:22AM -0500, Justin M. Forbes wrote:
> On Fri, 2012-09-07 at 16:44 +0100, Jan Beulich wrote:
> >
> > All of this still doesn't provide evidence that a plain upstream
> > kernel is actually having any problems in the first place. Further,
> > if you say EC2 has a
At 09/11/2012 10:24 AM, Yasuaki Ishimatsu Wrote:
> Hi Wen,
>
> 2012/09/11 11:15, Wen Congyang wrote:
>> Hi, ishimatsu
>>
>> At 09/05/2012 05:25 PM, we...@cn.fujitsu.com Wrote:
>>> From: Yasuaki Ishimatsu
>>>
>>> If system supports memory hot-remove, online_pages() may online
>>> removed pages.
Hi Tim,
On Mon, Sep 10, 2012 at 09:19:25AM -0700, Tim Chen wrote:
> We gather the pages that need to be unmapped in shrink_page_list. We batch
> the unmap to reduce the frequency of acquisition of
> the tree lock protecting the mapping's radix tree. This is
> possible as successive pages likely
Hi Wen,
2012/09/11 11:15, Wen Congyang wrote:
Hi, ishimatsu
At 09/05/2012 05:25 PM, we...@cn.fujitsu.com Wrote:
From: Yasuaki Ishimatsu
If system supports memory hot-remove, online_pages() may online removed pages.
So online_pages() need to check whether onlining pages are present or not.
Hi Linus,
Please pull blackfin updates for 3.6-rc6, one kbuild and a smp build fix.
Thanks,
-Bob
The following changes since commit 0d7614f09c1ebdbaa1599a5aba7593f147bf96ee:
Linux 3.6-rc1 (2012-08-02 16:38:10 -0700)
are available in the git repository at:
On Mon, Aug 27, 2012 at 12:01:41PM -0400, Namjae Jeon wrote:
> From: Namjae Jeon
>
> This commit adds dirty_background_time description in bdi sysfs
> documentation.
>
> Signed-off-by: Namjae Jeon
> Signed-off-by: Vivek Trivedi
> ---
> Documentation/ABI/testing/sysfs-class-bdi | 13
> What it's caused by is do_alignment_t32_to_handler(), and its assignment
> through a pointer of this variable. You can see that this is the cause
> because the patch below fixes the warning.
We couldn't fix the warning with your patch. the patch below fixes it instead.
IMHO, gcc can't seems to
Hi Namjae,
On Mon, Aug 27, 2012 at 12:01:14PM -0400, Namjae Jeon wrote:
> From: Namjae Jeon
>
> This patch is based on suggestion by Wu Fengguang:
> https://lkml.org/lkml/2011/8/19/19
>
> kernel has mechanism to do writeback as per dirty_ratio and dirty_background
> ratio. It also maintains
Hi Lai,
2012/09/11 10:22, Lai Jiangshan wrote:
On 09/11/2012 08:40 AM, Yasuaki Ishimatsu wrote:
Hi Lai,
Using memory_online to hot-added node's memory, the following kernel messages
were shown. Is this a known issue?
Thank you for your report.
What operations did you have performed ?
My
At 09/11/2012 08:52 AM, Minchan Kim Wrote:
> Hello Wen,
>
> On Fri, Sep 07, 2012 at 03:28:22PM +0800, Wen Congyang wrote:
>> At 09/06/2012 10:53 AM, Minchan Kim Wrote:
>>> Normally, MIGRATE_ISOLATE type is used for memory-hotplug.
>>> But it's irony type because the pages isolated would exist
>>>
'pcie_type'
Caused by commit ac161fbfdaa2 ("powerpc/powernv: I/O and memory alignment
for P2P bridges"). pcie_type was removed in commit b2ef39be5744 ("PCI:
Remove unused field pcie_type from struct pci_dev") (also in the pci
tree).
I have used the pci tree from next-20120910 f
On 09/10/2012 11:26 PM, Peter Zijlstra wrote:
> On Mon, 2012-09-10 at 15:10 +0800, Alex Shi wrote:
>> There is no load_balancer to be selected now. It just set state of
>> nohz tick stopping.
>>
>> So rename the function, pass the 'cpu' from parameter and then
>> remove the useless calling from
Hi Jerry,
On Tue, Sep 11, 2012 at 08:27:40AM +0800, Jerry wrote:
> Hi Wen,
>
> I have been arranged a job related memory hotplug on ARM architecture.
> Maybe I know some new issues about memory hotplug on ARM architecture. I
> just enabled it on ARM, and it works well in my Android tablet now.
>
On 09/11/2012 08:40 AM, Yasuaki Ishimatsu wrote:
> Hi Lai,
>
> Using memory_online to hot-added node's memory, the following kernel messages
> were shown. Is this a known issue?
Thank you for your report.
What operations did you have performed ?
Thanks.
Lai
>
> [ 327.837408] [
At 09/11/2012 08:27 AM, Jerry Wrote:
> Hi Wen,
>
> I have been arranged a job related memory hotplug on ARM architecture.
> Maybe I know some new issues about memory hotplug on ARM architecture. I
> just enabled it on ARM, and it works well in my Android tablet now.
> However, I have not send out
On Fri, Sep 07, 2012 at 09:47:01AM -0400, Konrad Rzeszutek Wilk wrote:
> On Fri, Sep 07, 2012 at 01:40:43PM +0200, Stefan Bader wrote:
> > When writing unsupported flags into CR4 (for some time the
> > xen_write_cr4 function would refuse to do anything at all)
> > older Xen hypervisors (and patch
Hi Michal,
On Thu, Sep 06, 2012 at 02:59:53PM +0200, Michal Nazarewicz wrote:
> > On Wed, Sep 05, 2012 at 07:28:23PM +0200, Michal Nazarewicz wrote:
> >> If you ask me, I'm not convinced that this improves anything.
>
> On Thu, Sep 06 2012, Minchan Kim wrote:
> > At least, it removes
Hello Wen,
On Fri, Sep 07, 2012 at 03:28:22PM +0800, Wen Congyang wrote:
> At 09/06/2012 10:53 AM, Minchan Kim Wrote:
> > Normally, MIGRATE_ISOLATE type is used for memory-hotplug.
> > But it's irony type because the pages isolated would exist
> > as free page in
Hi Bart,
On Thu, Sep 06, 2012 at 06:34:35PM +0200, Bartlomiej Zolnierkiewicz wrote:
>
> Hi,
>
> On Thursday 06 September 2012 04:53:38 Minchan Kim wrote:
> > Normally, MIGRATE_ISOLATE type is used for memory-hotplug.
> > But it's irony type because the pages isolated would exist
> > as free
On Sat, Sep 08, 2012 at 12:47:59PM -0700, Andi Kleen wrote:
> From: Andi Kleen
>
> For large kernel configurations (like a distribution kernel)
> targz-pkg takes a quite long time to just do the compression.
> I clocked it at 15+mins for a SUSE kernel like config on a fast
> system. And tarxz
Hi Lai,
Using memory_online to hot-added node's memory, the following kernel messages
were shown. Is this a known issue?
[ 327.837408] [ cut here ]
[ 327.892556] kernel BUG at mm/page_alloc.c:553!
[ 327.945621] invalid opcode: [#1] SMP
[ 327.994748] Modules
This patch drops clean cache pages instead of migration during
alloc_contig_range() to minimise allocation latency by reducing the amount
of migration is necessary. It's useful for CMA because latency of migration
is more important than evicting the background processes working set.
In addition,
On Mon, Sep 10, 2012 at 06:26:33PM +0200, Jan Kara wrote:
> On Sat 08-09-12 23:49:00, Wang Sheng-Hui wrote:
> > The function scans @delaying_queue and stops at the first inode
> > whose dirtied_when is after *work->older_than_this. So the expired
> > ones being moved are those before
This adds a pointer to the bvec array to struct bio_integrity_payload,
instead of the bvecs always being inline; then the bvecs are allocated
with bvec_alloc_bs().
This is needed eventually for immutable bio vecs - immutable bvecs
aren't useful if we still have to copy them, hence the need for
Without this patch, da8xx_register_spi() registers the SPI board info,
the SPI controller, and sets its number of chipselect to the size of the
static spi_board_info array. This is bad because a SPI board info may
declare devices for different SPI buses, and because other code can also
call
This is prep work for immutable bio vecs; we first want to centralize
where bvecs are modified.
Next two patches convert some existing code to use this function.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
---
fs/bio.c| 41 +
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: NeilBrown
---
drivers/md/md.c | 19 +--
1 file changed, 5 insertions(+), 14 deletions(-)
diff --git a/drivers/md/md.c b/drivers/md/md.c
index 7a2b079..51ce48c 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -190,25
Just a little convenience macro - main reason to add it now is preparing
for immutable bio vecs, it'll reduce the size of the patch that puts
bi_sector/bi_size/bi_idx into a struct bvec_iter.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
---
block/blk-core.c | 2 +-
On Mon, Sep 10, 2012 at 05:44:49PM +0100, Lee Jones wrote:
> Ola poke.
Please don't send contentless top-posted pings - if you think things
have been forgotten you should generally resend them, there's a good
chance that if things have been lost people don't have the original any
more. Replying
These were harmless but uneccessary,andt getting rid of them makes the
code easier to audit since most of them need to be removed.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
---
drivers/block/floppy.c | 1 -
drivers/md/dm-verity.c | 2 +-
drivers/md/raid10.c| 1 -
fs/buffer.c
More prep work for immutable bvecs/effecient bio splitting - usage of
bi_vcnt has to be auditing, so getting rid of all the unnecessary usage
makes that easier.
Plus, bio_segments() is really what this code wanted, as it respects the
current value of bi_idx.
Signed-off-by: Kent Overstreet
CC:
Random cleanup - this code was duplicated and it's not really specific
to md.
Also added the ability to return the actual error code.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: NeilBrown
---
drivers/md/raid1.c | 19 ---
drivers/md/raid10.c | 19 ---
More prep work for immutable bio vecs, mainly getting rid of references
to bi_idx.
bio_reset was being open coded in a few places. The one in sync_request
was a bit nontrivial to convert, so could use some extra eyeballs.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: NeilBrown
---
I couldn't figure out what sbio->bi_end_io in process_checks() was
supposed to be, so I took the easy way out.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: NeilBrown
---
drivers/md/raid1.c | 22 +-
1 file changed, 5 insertions(+), 17 deletions(-)
diff --git
Had to shuffle the code around a bit (where bi_rw and bi_end_io were
set), but shouldn't really be anything tricky here
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: NeilBrown
---
drivers/md/raid5.c | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff
More bi_idx removal. This code was just open coding bio_clone(). This
could probably be further improved by using bio_advance() instead of
skipping over null pages, but that'd be a larger rework.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: NeilBrown
---
drivers/md/raid1.c | 36
In the short term this'll help with code auditing, and if this code ever
gets used now it's converted :)
Signed-off-by: Kent Overstreet
CC: Jiri Kosina
---
drivers/block/pktcdvd.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/block/pktcdvd.c
This doesn't really delete any code _yet_, but once immutable bvecs are
done we can just delete the rest of the code in that loop.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: NeilBrown
---
drivers/md/raid1.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
A bunch of what __blk_queue_bounce() was doing was problematic for the
immutable bvec work; this cleans that up and the code is quite a bit
smaller, too.
The __bio_for_each_segment() in copy_to_high_bio_irq() was changed
because that one's looping over the original bio, not the bounce bio -
since
A few places in the code were either open coding or using the wrong
version - fix.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: NeilBrown
---
drivers/md/dm-crypt.c | 3 +--
drivers/md/raid1.c| 10 +++---
fs/bio.c | 20 ++--
fs/direct-io.c| 8
This is part of the immutable bvec prep work; bio_for_each_segment() is
going to have a different implementation so these need to be split
apart.
This change is also to better document the intent of code that's using
it - bio_for_each_segment_all() is only legal to use for code that owns
the bio.
More utility code to replace stuff that's getting open coded.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
---
fs/bio.c| 28
include/linux/bio.h | 1 +
2 files changed, 29 insertions(+)
diff --git a/fs/bio.c b/fs/bio.c
index d88ad77..65e6eac 100644
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: NeilBrown
---
drivers/md/raid1.c | 16 +++-
1 file changed, 3 insertions(+), 13 deletions(-)
diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c
index d30b4cb..18b743a 100644
--- a/drivers/md/raid1.c
+++ b/drivers/md/raid1.c
@@
This is for the new bio splitting code. When we split a bio, if the
split occured on a bvec boundry we reuse the bvec for the new bio. But
that means bio_free() can't free it, hence the explicit flag.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
Acked-by: Tejun Heo
---
fs/bio.c
This was the only real user of BIO_CLONED, which didn't have very clear
semantics. Convert to its own flag so we can get rid of BIO_CLONED.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: Martin K. Petersen
---
fs/bio-integrity.c | 5 ++---
include/linux/bio.h | 1 +
2 files changed, 3
BIO_CLONED wasn't very useful, and didn't have very clear semantics, so
kill it.
Replace it with a more useful flag - BIO_SUBMITTED means the bio has
been passed to generic_make_request() and the bvec can no longer be
modified.
Roll both changes into the same patch so we can steal the old bit
Signed-off-by: Kent Overstreet
CC: Jens Axboe
CC: Jiri Kosina
---
drivers/block/pktcdvd.c | 79 -
1 file changed, 12 insertions(+), 67 deletions(-)
diff --git a/drivers/block/pktcdvd.c b/drivers/block/pktcdvd.c
index 0824627..1079a77 100644
---
This gets open coded quite a bit and it's tricky to get right, so make a
generic version and convert some existing users over to it instead.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
---
fs/bio.c| 70 +
include/linux/bio.h |
Bunch of places in the code weren't using it where they could be -
this'll reduce the size of the patch that puts bi_sector/bi_size/bi_idx
into a struct bvec_iter.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
---
drivers/block/aoe/aoeblk.c | 2 +-
drivers/block/aoe/aoecmd.c | 2 +-
Prep work for immutable bio_vecs/efficient bio splitting: they require
auditing and removing most uses of bi_idx.
So here we convert bio_split() to respect the current value of bi_idx
and use the bio_iovec() macro, instead of assuming bi_idx will be 0.
Signed-off-by: Kent Overstreet
CC: Jens
Converts it to use bio_advance(), simplifying it quite a bit in the
process.
Signed-off-by: Kent Overstreet
CC: Jens Axboe
---
block/blk-core.c | 84 +++-
1 file changed, 16 insertions(+), 68 deletions(-)
diff --git a/block/blk-core.c
Random assortment of refactoring and trivial cleanups;
Immutable bio vecs and efficient bio splitting require auditing and
removing pretty much all bi_idx uses, among other things.
The reason is that with immutable bio vecs we can't use the bvec array
directly; if we have a partially completed
On 09/10/2012 12:45 PM, Daniel Lezcano wrote:
On 09/10/2012 07:14 PM, John Stultz wrote:
In the meantime, I'll try to reproduce on my T61. If you could send me
your .config, I'd appreciate it.
http://pastebin.com/qSxqfdDK
The header of the config file shows for a v3.5-rc7 because it is the
On Mon, Sep 10, 2012 at 10:53:29PM +0200, Luis Gonzalez Fernandez wrote:
> linux-3.6/drivers/regulator/tps62360-regulator.c: In function
> ‘tps62360_probe’:
> linux-3.6/drivers/regulator/tps62360-regulator.c:364:13: warning: cast from
> poi$
>
> Fix incorrect cast in tps62360_probe() function.
On Sun, Sep 9, 2012 at 10:54 AM, Mathias Krause wrote:
> On Sun, Sep 2, 2012 at 11:37 PM, Mathias Krause
> wrote:
>> It's redundant and makes sparse complain about it.
>>
>> Signed-off-by: Mathias Krause
>> ---
>> include/linux/pci.h |2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
On Mon, Sep 10, 2012 at 4:55 PM, Fengguang Wu wrote:
> On Mon, Sep 10, 2012 at 02:44:32PM -0600, Bjorn Helgaas wrote:
>> On Mon, Sep 10, 2012 at 1:36 PM, Jiri Slaby wrote:
>> > Commit "PCI: Use pci_device_id on stack for pci_get_subsys/class() to
>> > avoid kmalloc" changed heap allocations to
On Mon, Sep 10, 2012 at 04:35:02PM -0700, Tejun Heo wrote:
> debug. Would it be difficult to convert dm drivers to collect size
> requirements and use front-pad for all per-bio data?
I can't give a quick answer because a single bio may require a variable
number (depending on the bio content) of
On Mon, 10 Sep 2012 12:28:35 +0200 (CEST) Thomas Gleixner
wrote:
> On Mon, 10 Sep 2012, NeilBrown wrote:
> >
> > The IRQCHIP_MASK_ON_SUSPEND flag seems to be hard to use correctly, so
> > either
> > I'm understanding it wrongly, or it could be made easier to use.
> > If the first case, I'm
On Mon, Sep 10, 2012 at 01:40:36PM -0600, Stephen Warren wrote:
> On 09/10/2012 09:03 AM, Daniel Mack wrote:
> > + - "enabled-at-boot": If set, the regulator has been enabled
> > at boot
> > + time
> Isn't that regulator-boot-on, as defined in
On Thu, Sep 06, 2012 at 10:25:24PM +0400, Evgeniy Polyakov wrote:
> Hi
>
> On Wed, Sep 05, 2012 at 02:07:58PM -0700, Greg KH (g...@kroah.com) wrote:
> > > Greg, please pull it into your tree
> > > Acked-by: Evgeniy Polyakov
> >
> > I can't seem to find the original patch here, can someone
Hello, Alasdair.
On Tue, Sep 11, 2012 at 12:09:17AM +0100, Alasdair G Kergon wrote:
> On Mon, Sep 10, 2012 at 04:01:01PM -0700, Tejun Heo wrote:
> > IIUC, Kent posted a patch which converts all of them to use front-pad
>
> The only patch I saw posted here only handles one of the easier cases so
1 - 100 of 1382 matches
Mail list logo