hi Lkml https://bit.ly/2KhYIvQ
Badari Pulavarty
Good morning lkml
http://www.drewlin.me/pitch.php?cause=1va2u3y80rqgv
Badari Pulavarty
64. But on ppc64 is
contains ONLY iomem related. Paulus didn't want to export all the system
memory into /proc/iomem on ppc64. So I had to workaround by providing
arch-specific walk_memory_resource() function for ppc64.
Thanks,
Badari
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
On Wed, 2008-02-13 at 14:09 +0900, Yasunori Goto wrote:
> Thanks Badari-san.
>
> I understand what was occured. :-)
>
> > On Tue, 2008-02-12 at 13:56 -0800, Badari Pulavarty wrote:
> > > > > + /*
> > > > > +* Its ugly, but this is the best I
On Tue, 2008-02-12 at 14:15 -0800, Dave Hansen wrote:
> On Tue, 2008-02-12 at 14:07 -0800, Badari Pulavarty wrote:
> > On Tue, 2008-02-12 at 13:57 -0800, Dave Hansen wrote:
> > > On Tue, 2008-02-12 at 13:56 -0800, Badari Pulavarty wrote:
> > > >
> > > >
On Tue, 2008-02-12 at 14:15 -0800, Dave Hansen wrote:
> On Tue, 2008-02-12 at 14:07 -0800, Badari Pulavarty wrote:
> > On Tue, 2008-02-12 at 13:57 -0800, Dave Hansen wrote:
> > > On Tue, 2008-02-12 at 13:56 -0800, Badari Pulavarty wrote:
> > > >
> > > >
On Tue, 2008-02-12 at 13:57 -0800, Dave Hansen wrote:
> On Tue, 2008-02-12 at 13:56 -0800, Badari Pulavarty wrote:
> >
> > +static void __remove_section(struct zone *zone, unsigned long
> > section_nr)
> > +{
> > + if (!valid_section_nr(sectio
On Tue, 2008-02-12 at 12:59 -0800, Dave Hansen wrote:
> On Tue, 2008-02-12 at 09:22 -0800, Badari Pulavarty wrote:
> > +static void __remove_section(struct zone *zone, unsigned long
> > phys_start_pfn)
> > +{
> > + if (!pfn_valid(phys_start_pfn))
> > +
On Tue, 2008-02-12 at 17:06 +0900, Yasunori Goto wrote:
> > On Mon, 2008-02-11 at 11:48 -0800, Andrew Morton wrote:
> > > On Mon, 11 Feb 2008 09:23:18 -0800
> > > Badari Pulavarty <[EMAIL PROTECTED]> wrote:
> > >
> > > > Hi Andrew,
> > >
On Mon, 2008-02-11 at 11:48 -0800, Andrew Morton wrote:
> On Mon, 11 Feb 2008 09:23:18 -0800
> Badari Pulavarty <[EMAIL PROTECTED]> wrote:
>
> > Hi Andrew,
> >
> > While testing hotplug memory remove against -mm, I noticed
> > that unregister_mem
On Mon, 2008-02-11 at 09:54 -0800, Greg KH wrote:
> On Mon, Feb 11, 2008 at 09:23:18AM -0800, Badari Pulavarty wrote:
> > Hi Andrew,
> >
> > While testing hotplug memory remove against -mm, I noticed
> > that unregister_memory() is not cleaning up /sysfs entrie
issue.
Could you please include it in -mm ?
Thanks,
Badari
register_memory()/unregister_memory() never gets called with
"root". unregister_memory() is accessing kobject_name of
the object just freed up. Since no one uses the code,
lets take the code out. And also, make register_memor
On Thu, 2008-02-07 at 20:55 -0800, Greg KH wrote:
> On Thu, Feb 07, 2008 at 05:25:46PM -0800, Badari Pulavarty wrote:
> > On Thu, 2008-02-07 at 16:38 -0800, Greg KH wrote:
> > > On Thu, Feb 07, 2008 at 03:56:58PM -0800, Badari Pulavarty wrote:
> > > > Hi Greg,
>
On Thu, 2008-02-07 at 21:41 -0800, Greg KH wrote:
> On Thu, Feb 07, 2008 at 09:08:42PM -0800, Badari Pulavarty wrote:
> >
> > On Thu, 2008-02-07 at 20:55 -0800, Greg KH wrote:
> > > On Thu, Feb 07, 2008 at 05:25:46PM -0800, Badari Pulavarty wrote:
> > > > On
On Thu, 2008-02-07 at 20:55 -0800, Greg KH wrote:
> On Thu, Feb 07, 2008 at 05:25:46PM -0800, Badari Pulavarty wrote:
> > On Thu, 2008-02-07 at 16:38 -0800, Greg KH wrote:
> > > On Thu, Feb 07, 2008 at 03:56:58PM -0800, Badari Pulavarty wrote:
> > > > Hi Greg,
>
On Thu, 2008-02-07 at 16:38 -0800, Greg KH wrote:
> On Thu, Feb 07, 2008 at 03:56:58PM -0800, Badari Pulavarty wrote:
> > Hi Greg,
> >
> > While playing with hotplug memory remove on 2.6.24-mm1, I
> > noticed that /sysfs directory entries are not getting removed.
?
With this change, the directories are getting removed
correctly. Comments ?
Thanks,
Badari
sysdev_unregister() should call kobject_del() to remove
the object.
Signed-off-by: Badari Pulavarty <[EMAIL PROTECTED]>
---
drivers/base/sys.c |1 +
1 file changed, 1 insertion(+)
Index: linux-
em EXPORT_SYMBOL_GPL() ?
Otherwise, looks good to me.
I have been planning to send this as part of my next update
with ppc64 arch-specific remove support and generic __remove_pages()
support. If this is blocking your work, lets get this in.
Acked-by: Badari Pulavarty <[EMAIL PROTECTED]>
Thank
ins the bhs while committing journal so
> > try_to_release_page fails when jbd
> > is committing the transaction.
>
> Yeah. It sure would be fantastic if some ext3 expert could stop this
> from happening somehow. But that hasn't happened in.. uh.. Badari, for
> how many y
003f0dbd90] [c02f4bb0] .__pci_register_driver+0x5c/0xcc
[c0003f0dbe20] [c0604b38] .ipr_init+0x38/0x50
[c0003f0dbea0] [c05d6428] .kernel_init+0x214/0x3ec
[c0003f0dbf90] [c0026734] .kernel_thread+0x4c/0x68
Instruction dump:
e8410028 3921 38210080 7d234b78 e801001
ond
> that point. I have not yet found time to debug it though.
Are you running into same issue, I am getting on my machine ? Are you
using IPR driver ?
Thanks,
Badari
e100: Intel(R) PRO/100 Network Driver, 3.5.23-k4-NAPI
e100: Copyright(c) 1999-2006 Intel Corporation
ipr: IBM Power RAID SCSI
On Tue, 2007-11-06 at 02:33 -0800, [EMAIL PROTECTED] wrote:
> The mm snapshot broken-out-2007-11-06-02-32.tar.gz has been uploaded to
>
>
> ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-11-06-02-32.tar.gz
>
> It contains the following patches against 2.6.24-rc1:
Getti
50): undefined reference to
> `devices_subsys'
> make: *** [.tmp_vmlinux1] Error 1
I sent a fix for this 30 minutes ago ..
Here it is anyway.
Thanks,
Badari
---
arch/powerpc/kernel/vio.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Index: linux-2
t-in.o(.toc+0x1548): undefined reference to
`devices_subsys'
make: *** [.tmp_vmlinux1] Error 1
Here is the patch, Is this correct usage ?
Thanks,
Badari
---
arch/powerpc/kernel/vio.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Index: linux-2.6.24-rc1/ar
; make[2]: *** [drivers/pci/hotplug] Error 2
> make[1]: *** [drivers/pci] Error 2
> make: *** [drivers] Error 2
>
> The patch,
> gregkh-driver-kset-convert-pci-hotplug-to-use-kset_create_and_register.patch
> is causing the build failure.
Here is the fix (against 24-rc1 mm-broke
gt;
> The above patch causes the build failure, because the from_rtc_time() and
> to_rtc_time() are
> moved under the ifdef CONFIG_ADB_CUDA, but they are begin called in the
> pmu_set_rtc_time() and
> pmac_get_rtc_time() under CONFIG_ADB_PMU.
Yes. My fault. This patch needs to be dropp
;
> if (request_resource(&iomem_resource, res) < 0) {
> printk("System RAM resource %llx - %llx cannot be added\n",
> (unsigned long long)res->start, (unsigned long long)res->end);
>
Not quite.. You need following patch on
Simple compile warning fix. (against 2.6.23-git12)
Thanks,
Badari
vortex_up() should initialize 'err' for a successful return.
drivers/net/3c59x.c: In function `vortex_up':
drivers/net/3c59x.c:1494: warning: `err' might be used uninitialized in this
function
Signed-off-
Fix to clean up compile warnings (against 2.6.23-git12)
Thanks,
Badari
to_rtc_time() and from_rtc_time() seems to be used only if CONFIG_ADB_CUDA
defined. Moving them under that ifdef.
arch/powerpc/platforms/powermac/time.c:88: warning: `to_rtc_time' defined but
not used
arch/powerpc/plat
Simple error handling fix (against 2.26.23-git12).
Thanks,
Badari
Need to initialize "err" in case of skb_clone() failure.
net/ipv4/ip_fragment.c: In function `ip_defrag':
net/ipv4/ip_fragment.c:540: warning: `err' might be used uninitialized in this
function
Signed-off-
On Wed, 2007-10-17 at 18:38 +0300, Ismail Dönmez wrote:
> Wednesday 17 October 2007 Tarihinde 18:33:18 yazmıştı:
> > Known issue ?
> >
> >
> > Thanks,
> > Badari
> >
> > CHK include/linux/version.h
> > CHK include/linux/utsrelease.h
Andrew Morton wrote:
On Fri, 13 Jul 2007 18:36:54 -0700
Badari Pulavarty <[EMAIL PROTECTED]> wrote:
More statfs() improvements for ext2. ext2 already maintains
percpu counters for free blocks and inodes. Derive free
block count and inode count by summing up percpu counters,
inst
On Fri, 2007-10-05 at 15:41 +0200, Valerie Clement wrote:
> Badari Pulavarty wrote:
> > On Thu, 2007-10-04 at 18:13 +0200, Valerie Clement wrote:
> >> While running ffsb tests on my ext4 filesystem, I got an Oops in
> >> cache_alloc_refill().
> >> I turn
On Thu, 2007-10-04 at 18:13 +0200, Valerie Clement wrote:
> While running ffsb tests on my ext4 filesystem, I got an Oops in
> cache_alloc_refill().
> I turned on SLAB debugging and here is the message I got:
>
> slab: Internal list corruption detected in cache 'buffer_head'(30),
> slabp 810
3_ I made
> objdump -x -D -s super.o
> (the only place where I found this function in the source code). If you
> want something else, let me know, but a bit more specific. Can I do the
> objdump directly from the kernel image file?
>
objdump -DlS balloc.o
would give us e
Hi Andy,
One the patch you created in -mm is causing compile warning.
Here is the fix. Please verify.
Thanks,
Badari
arch/powerpc/mm/init_64.c: In function `vmemmap_populated':
arch/powerpc/mm/init_64.c:211: warning: passing arg 1 of
`vmemmap_section_start' makes pointer from integer
k_action':
> /root/scrap/linux-2.6.23-rc8/drivers/base/memory.c:188: undefined reference
> to `.remove_memory'
> make: *** [.tmp_vmlinux1] Error 1
>
I ran into the same thing earlier. Here is the fix I made.
Thanks,
Badari
Memory hotplug remove is currently supported only on
On Tue, 2007-09-25 at 10:02 -0700, Andrew Morton wrote:
> On Tue, 25 Sep 2007 09:57:01 -0700 Badari Pulavarty <[EMAIL PROTECTED]> wrote:
>
> > On Mon, 2007-09-24 at 17:46 -0700, Linus Torvalds wrote:
> > > Ok, I think I'm getting close to releasing a real 2.6.23. T
but one of my machines doesn't
like 2.6.23-rc8. Infact, 2.6.23-rc1 was the first kernel this is broken.
(I didn't get hands on this machine till now).
Since my other x86-64 machines are doing fine, I am going to blame it
on my machine specific config :)
Thanks,
Badari
elm3a242:/usr/src/linux
On Mon, 2007-09-24 at 13:04 -0700, Linus Torvalds wrote:
>
> On Mon, 24 Sep 2007, Badari Pulavarty wrote:
> >
> > Whats happening on my machine is ..
> >
> > dbench forks of 4 children and sends them a signal to start the work.
> > 3 out of 4 children gets
On Mon, 2007-09-24 at 13:04 -0700, Linus Torvalds wrote:
>
> On Mon, 24 Sep 2007, Badari Pulavarty wrote:
> >
> > Whats happening on my machine is ..
> >
> > dbench forks of 4 children and sends them a signal to start the work.
> > 3 out of 4 children gets
-rc3 doesn't seem
to have this problem). I am going to do bi-sect and find out which
patch caused this.
I am using dbench-2.0 which consistently reproduces the problem on
my x86-64 box. Did you find anything new with your setup ?
Thanks,
Badari
-
To unsubscribe from this list: send the
T4_FEATURE_INCOMPAT_META_BG0x0010
> #define EXT4_FEATURE_INCOMPAT_EXTENTS0x0040 /* extents
> support */
> #define EXT4_FEATURE_INCOMPAT_64BIT 0x0080
> +#define EXT4_FEATURE_INCOMPAT_FLEX_BG0x0200
Any reason why 0x100 is skipped ?
Thanks,
On Tue, 2007-09-18 at 15:21 -0700, Badari Pulavarty wrote:
> Hi Balbir,
>
> I get following panic from SLUB, while doing simple fsx tests.
> I haven't used any container/memory controller stuff except
> that I configured them in :(
>
> Looks like slub doesn't
Hi Balbir,
I get following panic from SLUB, while doing simple fsx tests.
I haven't used any container/memory controller stuff except
that I configured them in :(
Looks like slub doesn't like one of the flags passed in ?
Known issue ? Ideas ?
Thanks,
Badari
CONFIG_CO
ee_pages() waste
> 1/3-1/2 page space.
>
> What was the originally intention to set up slabs for committed_data(and
> frozen_buffer) in JBD? Why not using kmalloc?
>
> Mingming
Looks good. Small suggestion is to get rid of all kmalloc() usages and
consistently use jbd_kmallo
On Tue, 2007-07-31 at 15:34 -0700, Andrew Morton wrote:
> On Tue, 31 Jul 2007 15:25:11 -0700
> Badari Pulavarty <[EMAIL PROTECTED]> wrote:
>
> > + dio->map_bh.b_state = 0;
>
> ho hum, thanks.
>
> We zero out so many fields in there now that a kzalloc() mig
IO size < blocksize case). It checks if the buffer is
newly allocation by doing buffer_new(bh). map_bh is NOT initialized and
never went through get_block() code. So, its possible to pass the check
and end up submitting a page wrongly and causing the oops.
Fix is to initialize the buffer
. looks lile my machine crashed too while running the tests. I
will take a look.
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
P
job10]
ioengine=mmap
rw=randwrite
bs=1k
filename=file1:file2
[job11]
ioengine=mmap
rw=randwrite
direct=1
filename=file1:file2
Hmm.. in this config file, whats causing DIO to panic ? Which test actually
passing faulty buffer ?
Thanks,
Badari
On Mon, 2007-07-30 at 16:38 -0700, Zach Brown wrote:
> On Jul 30, 2007, at 2:58 PM, Badari Pulavarty wrote:
>
> > On Mon, 2007-07-30 at 14:45 -0700, Zach Brown wrote:
> >>> I am also taking a look at it right now.
> >>
> >> Are we having a race to w
On Mon, 2007-07-30 at 16:38 -0700, Zach Brown wrote:
> On Jul 30, 2007, at 2:58 PM, Badari Pulavarty wrote:
>
> > On Mon, 2007-07-30 at 14:45 -0700, Zach Brown wrote:
> >>> I am also taking a look at it right now.
> >>
> >> Are we having a race to w
e done.
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
. Which is still many weeks away so there
> is
> plenty of time to prepare something which was carefully reviewed and
> well-tested,
> thanks.
I am also taking a look at it right now. Unfortunately, I don't think
fix is that simple - since we need to return success, in case of a
parti
On Fri, 2007-07-20 at 14:29 +1000, Nick Piggin wrote:
> Andrew Morton wrote:
> > On Thu, 19 Jul 2007 08:51:49 -0700 Badari Pulavarty <[EMAIL PROTECTED]>
> > wrote:
> >
> >
> >>>>+ }
> >>>>+
> >>>>+
On Wed, 2007-07-18 at 22:19 -0700, Andrew Morton wrote:
> On Fri, 13 Jul 2007 18:23:33 -0700 Badari Pulavarty <[EMAIL PROTECTED]> wrote:
>
> > Hi Andrew,
> >
> > Here is the patch to support read() for hugetlbfs, needed to get
> > oprofile working
On Wed, 2007-07-18 at 20:18 -0700, Andrew Morton wrote:
> On Fri, 13 Jul 2007 18:36:54 -0700 Badari Pulavarty <[EMAIL PROTECTED]> wrote:
>
> > More statfs() improvements for ext2. ext2 already maintains
> > percpu counters for free blocks and inodes. Derive free
> >
Andrew,
Can you include it in -mm ?
BTW, this patch is against mainline, won't apply cleanly to -mm, due to
other statfs() improvements.
Thanks,
Badari
More statfs() improvements for ext2. ext2 already maintains
percpu counters for free blocks and inodes. Derive free
block count and
Hi Andrew,
Here is the patch to support read() for hugetlbfs, needed to get
oprofile working on executables backed by largepages.
If you plan to consider Christoph Lameter's pagecache cleanup patches,
I will re-write this. Otherwise, please consider this for -mm.
Thanks,
Badari
Suppor
On Tue, 2007-07-10 at 15:30 +0400, Pavel Emelianov wrote:
> Cedric Le Goater wrote:
> > Badari Pulavarty wrote:
> >> On Fri, 2007-07-06 at 12:01 +0400, Pavel Emelianov wrote:
> >>> This is "submition for inclusion" of hierarchical, not kconfig
> >>
On Tue, 2007-07-10 at 17:06 +0400, Pavel Emelianov wrote:
> > Not able to boot my ppc64 machine with the patchset :(
> >
> > Thanks,
> > Badari
>
> That's the hunk lost during the split:
>
> --- ./fs/proc/root.c.procfix 2007-07-10 13:52:08.0 +0
On Mon, 2007-07-09 at 22:06 +0200, Cedric Le Goater wrote:
> Badari Pulavarty wrote:
> > On Fri, 2007-07-06 at 12:01 +0400, Pavel Emelianov wrote:
> >> This is "submition for inclusion" of hierarchical, not kconfig
> >> configurable, zero overheaded ;) pid nam
On Fri, 2007-07-06 at 12:01 +0400, Pavel Emelianov wrote:
> This is "submition for inclusion" of hierarchical, not kconfig
> configurable, zero overheaded ;) pid namespaces.
Not able to boot my ppc64 machine with the patchset :(
Thanks,
Badari
Unable to handle kernel paging req
t=/tmp/bench-static after “cp -a /dev/hda /tmp/”.
Hmm.. I didn't have any luck booting my machine with the patchset
(with 8k pagesize) :(
It fails to find the partition table on my hard drive.
Thanks,
Badari
AMD8111: IDE controller at PCI slot :00:07.1
AMD8111: chipset revision 3
ike to support for it.
I would like to play with, please keep the patchset uptodate.
Here is the small nit fix ..
Thanks,
Badari
mm/migrate.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-2.6.22-rc7/mm/migrate.c
=
On Thu, 2007-06-28 at 03:43 -0700, Andrew Morton wrote:
> ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.22-rc6/2.6.22-rc6-mm1/
>
fs/xfs/linux-2.6/xfs_ioctl32.c: In function ‘xfs_ioc_bulkstat_compat’:
fs/xfs/linux-2.6/xfs_ioctl32.c:334: error: ‘xfs_inumbers_fmt_compat’
undeclar
the BUG_ON condition is a
> good one.
I will ask them to try that.
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
). But unfortunately, our perf. team is able reproduce the
problem. Debug indicated that, the ret2 == 1 :(
Not sure how that can happen. Ideas ?
Thanks,
Badari
>
> ---
>
> dio: remove bogus refcounting BUG_ON
>
> Badari Pulavarty reported a case of this BUG_ON is trig
et this to work, without any hacks.
Thanks,
Badari
fs/hugetlbfs/inode.c | 117 +++
1 file changed, 117 insertions(+)
Index: linux/fs/hugetlbfs/inode.c
===
--- linux.orig/fs/huge
r generic_write_end(). :(
Mingming, can you take a look at the current & proposed order ?
I ran into bunch of races when I tried to change the order for
->writepages() support earlier :(
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
elease, unless Albert really wants to extend proc-ps utils for
hugetlbfs segments too.
But, its very simple patch - you might as well push this too.
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
meaningful
errors instead of NULL.
Signed-off-by: Cedric Le Goater <[EMAIL PROTECTED]>
Cc: Serge E. Hallyn <[EMAIL PROTECTED]>
Cc: Badari Pulavarty <[EMAIL PROTECTED]>
Cc: Pavel Emelianov <[EMAIL PROTECTED]>
Cc: Herbert Poetzl <[EMAIL PROTECTED]>
Cc: Eric W.
ntion as regular
shmem for its
name. This is not absolutely needed, its a nice to have. Currently, user
space tools
can't depend on the filename alone, since its not unique (based on kry).
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel&qu
Andrew,
Can you include this in -mm ?
Thanks,
Badari
shmid used to be stored as inode# for shared memory segments. Some of
the proc-ps tools use this from /proc/pid/maps. Recent cleanups
to newseg() changed it. This patch sets inode number back to shared
memory id to fix breakage.
Signed
an only
depend
on inode# to match shmid. (since key is not unique esp. for key=0x).
BTW, I agree that shmid is not unique even without namespaces as its
based on
seq# and we wrap seq#.
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel"
Serge E. Hallyn wrote:
Quoting Serge E. Hallyn ([EMAIL PROTECTED]):
Quoting Badari Pulavarty ([EMAIL PROTECTED]):
On Thu, 2007-06-07 at 15:37 -0500, Serge E. Hallyn wrote:
Quoting Badari Pulavarty ([EMAIL PROTECTED]):
On Thu, 2007-06-07 at 12:48 -0700, Andrew Morton wrote:
On Thu
Serge E. Hallyn wrote:
Quoting Badari Pulavarty ([EMAIL PROTECTED]):
On Thu, 2007-06-07 at 15:37 -0500, Serge E. Hallyn wrote:
Quoting Badari Pulavarty ([EMAIL PROTECTED]):
On Thu, 2007-06-07 at 12:48 -0700, Andrew Morton wrote:
On Thu, 07 Jun 2007 10:06:37 -0700
Badari Pulavarty
On Thu, 2007-06-07 at 15:37 -0500, Serge E. Hallyn wrote:
> Quoting Badari Pulavarty ([EMAIL PROTECTED]):
> > On Thu, 2007-06-07 at 12:48 -0700, Andrew Morton wrote:
> > > On Thu, 07 Jun 2007 10:06:37 -0700
> > > Badari Pulavarty <[EMAIL PROTECTED]> wrote:
> &
On Thu, 2007-06-07 at 12:48 -0700, Andrew Morton wrote:
> On Thu, 07 Jun 2007 10:06:37 -0700
> Badari Pulavarty <[EMAIL PROTECTED]> wrote:
>
> > On Thu, 2007-06-07 at 12:43 -0400, Albert Cahalan wrote:
> > > On 6/7/07, Badari Pulavarty <[EMAIL PROTECTED]> wro
On Thu, 2007-06-07 at 12:43 -0400, Albert Cahalan wrote:
> On 6/7/07, Badari Pulavarty <[EMAIL PROTECTED]> wrote:
>
> > BTW, I agree with Eric that its would be nice to use shmid as part
> > of name instead of forcing to be as inode number. It should be
> > possible
On Thu, 2007-06-07 at 00:53 -0400, Albert Cahalan wrote:
> On 6/6/07, Andrew Morton <[EMAIL PROTECTED]> wrote:
> > On Wed, 6 Jun 2007 23:27:01 -0400 "Albert Cahalan" <[EMAIL PROTECTED]>
> > wrote:
> > > Eric W. Biederman writes:
>
On Wed, 2007-06-06 at 11:02 -0600, Eric W. Biederman wrote:
> Badari Pulavarty <[EMAIL PROTECTED]> writes:
>
> > Hi Eric,
> >
> > Your recent cleanup to shm code, namely
> >
> > [PATCH] shm: make sysv ipc shared memory use stacked files
> >
&g
this done deliberately ? Anything wrong in setting this back ?
Comments ?
Thanks,
Badari
Without patch:
--
# ipcs -m
-- Shared Memory Segments
keyshmid owner perms bytes nattch status
0x 884737 db2inst1 76733554432
74: warning: `__kfree_section_memmap' defined but not used
Here is the patch.
Thanks,
Badari
__kmalloc_section_memmap(), vaddr_in_vmalloc_area() and
__kfree_section_memmap() are used only for MEMORY_HOTPLUG.
Moved them under CONFIG_MEMORY_HOTPLUG.
Signed-off-by: Badari Pulavarty <[EMAIL PROTECTED]>
Inde
; > [c47bba90] [c000c178] .do_IRQ+0x90/0xec
> > > [c47bbb10] [c0004790] hardware_interrupt_entry+0x18/0x1c
> >
> > Not good. The qla changes are non-trivial (that hardware has a really
> > funky sg setup), so I may have botched a part of it.
On Mon, 2007-05-21 at 08:35 +0200, Jens Axboe wrote:
> On Mon, May 21 2007, Jens Axboe wrote:
> > On Fri, May 18 2007, Badari Pulavarty wrote:
> > > On Fri, 2007-05-18 at 09:35 +0200, Jens Axboe wrote:
> > > > On Thu, May 17 2007, Badari Pulavarty wrote:
> > >
On Mon, 2007-05-21 at 08:14 +0200, Jens Axboe wrote:
> On Fri, May 18 2007, Badari Pulavarty wrote:
> > On Fri, 2007-05-18 at 09:35 +0200, Jens Axboe wrote:
> > > On Thu, May 17 2007, Badari Pulavarty wrote:
> > > > On Thu, 2007-05-17 at 08:27 +0200, Jens Axboe wrote:
On Fri, 2007-05-18 at 09:35 +0200, Jens Axboe wrote:
> On Thu, May 17 2007, Badari Pulavarty wrote:
> > On Thu, 2007-05-17 at 08:27 +0200, Jens Axboe wrote:
> > > On Wed, May 16 2007, Badari Pulavarty wrote:
> > > > On Tue, 2007-05-15 at 19:50 +0200, Jens Axboe wrote:
On Fri, 2007-05-18 at 19:03 +0200, Jens Axboe wrote:
> On Fri, May 18 2007, Badari Pulavarty wrote:
> > On Fri, 2007-05-18 at 09:33 +0200, Jens Axboe wrote:
> > > On Thu, May 17 2007, Badari Pulavarty wrote:
> > > > On Thu, 2007-05-17 at
On Fri, 2007-05-18 at 09:33 +0200, Jens Axboe wrote:
> On Thu, May 17 2007, Badari Pulavarty wrote:
> > On Thu, 2007-05-17 at 08:27 +0200, Jens Axboe wrote:
> > ..
> > > > >
> > > > > Ah ok, you need the updated patch series for ppc64 support. Builds
On Thu, 2007-05-17 at 08:27 +0200, Jens Axboe wrote:
> On Wed, May 16 2007, Badari Pulavarty wrote:
> > On Tue, 2007-05-15 at 19:50 +0200, Jens Axboe wrote:
> > > On Tue, May 15 2007, Badari Pulavarty wrote:
> > > > On Tue, 2007-05-15 at 19:20 +0200, Jens Axboe wrote:
ots fine with following backed out.
Looking at the code ide_probe.c: hwif_init() is doing
hwif->sg_table = kmalloc(sizeof(struct scatterlist)*hwif->sg_max_nents,
GFP_KERNEL);
blk_rq_map_sg() is looking for the chaining info and going over end of the
a
g/git-new/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=70674f95c0a2ea694d5c39f4e514f538a09be36f
>
> And I think it makes sense for the compat code to do it too.
>
> Anton
Here it is ..
Should I do one for poll() also ?
Thanks,
Badari
Optimize select by a using stack space
On Tue, 2007-05-15 at 19:50 +0200, Jens Axboe wrote:
> On Tue, May 15 2007, Badari Pulavarty wrote:
> > On Tue, 2007-05-15 at 19:20 +0200, Jens Axboe wrote:
> > > On Tue, May 15 2007, Badari Pulavarty wrote:
> > > > On Fri, 2007-05-11 at 15:51 +0200,
On Tue, 2007-05-15 at 19:50 +0200, Jens Axboe wrote:
> On Tue, May 15 2007, Badari Pulavarty wrote:
> > On Tue, 2007-05-15 at 19:20 +0200, Jens Axboe wrote:
> > > On Tue, May 15 2007, Badari Pulavarty wrote:
> > > > On Fri, 2007-05-11 at 15:51 +0200,
On Tue, 2007-05-15 at 10:44 -0700, Andrew Morton wrote:
> On Tue, 15 May 2007 10:29:18 -0700
> Badari Pulavarty <[EMAIL PROTECTED]> wrote:
>
> > Hi,
> >
> > Is select(0, ..) is a valid operation ?
>
> Probably - it becomes an elaborate way of doing a sl
On Tue, 2007-05-15 at 19:20 +0200, Jens Axboe wrote:
> On Tue, May 15 2007, Badari Pulavarty wrote:
> > On Fri, 2007-05-11 at 15:51 +0200, Jens Axboe wrote:
> > > Hi,
> > >
> > > Updated version of the patch - this time I'll just attach the patch
>
Hi,
Is select(0, ..) is a valid operation ?
I see that there is no check to prevent this or return
success early, without doing any work. Do we need one ?
slub code is complaining that we are doing kmalloc(0).
Thanks,
Badari
[ cut here ]
Badness at include/linux
cb.c: In function ‘qla24xx_build_scsi_iocbs’:
drivers/scsi/qla2xxx/qla_iocb.c:678: warning: implicit declaration of
function ‘for_each_sg’
drivers/scsi/qla2xxx/qla_iocb.c:678: error: expected ‘;’ before ‘{’
token
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-
spin_unlock(&dentry->d_lock);
> + union_lock(dentry);
> + }
> +}
> +
> +static inline struct union_info *union_lock_and_get(struct dentry *dentry)
> +{
> + union_lock(dentry);
> + return union_get(dentry->d_union);
> +}
> +
> +/* Shouldn't be called with last reference to union_info */
> +static inline void union_put_and_unlock(struct union_info *uinfo)
> +{
> + union_put(uinfo);
> + __union_unlock(&uinfo->u_mutex);
^^^
It should be
__union_unlock(uinfo);
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
1 - 100 of 200 matches
Mail list logo