Commit 5acf32ee authored by Andrew Morton's avatar Andrew Morton Committed by Linus Torvalds

[PATCH] misc fixes

- i460-agp linkage fix ("Luck, Tony" <tony.luck@intel.com>)

- Don't reimplement offsetof() in hfs

- NBD warning fix

- Remove unneeded null-pointer test in journal_stop (Andreas Gruenbacher)

- remove debug stuff in journal_dirty_metadata()

- slab.c typo fixes (Lev Makhlis <mlev@despammed.com>)

- In devfs_mk_cdev() error path, don't print `buf' until we've written
  something into it.  (Reported by Gergely Nagy <algernon@gandalph.mad.hu>)

- Two ISA sound drivers had their kmalloc() args reversed (Spotted by Steve
  French)
parent c07f63b3
...@@ -258,7 +258,8 @@ void nbd_send_req(struct nbd_device *lo, struct request *req) ...@@ -258,7 +258,8 @@ void nbd_send_req(struct nbd_device *lo, struct request *req)
dprintk(DBG_TX, "%s: request %p: sending control (%s@%llu,%luB)\n", dprintk(DBG_TX, "%s: request %p: sending control (%s@%llu,%luB)\n",
lo->disk->disk_name, req, lo->disk->disk_name, req,
nbdcmd_to_ascii(nbd_cmd(req)), nbdcmd_to_ascii(nbd_cmd(req)),
req->sector << 9, req->nr_sectors << 9); (unsigned long long)req->sector << 9,
req->nr_sectors << 9);
result = sock_xmit(sock, 1, &request, sizeof(request), result = sock_xmit(sock, 1, &request, sizeof(request),
(nbd_cmd(req) == NBD_CMD_WRITE)? MSG_MORE: 0); (nbd_cmd(req) == NBD_CMD_WRITE)? MSG_MORE: 0);
if (result <= 0) { if (result <= 0) {
......
...@@ -608,7 +608,7 @@ static struct pci_driver agp_intel_i460_pci_driver = { ...@@ -608,7 +608,7 @@ static struct pci_driver agp_intel_i460_pci_driver = {
.name = "agpgart-intel-i460", .name = "agpgart-intel-i460",
.id_table = agp_intel_i460_pci_table, .id_table = agp_intel_i460_pci_table,
.probe = agp_intel_i460_probe, .probe = agp_intel_i460_probe,
.remove = agp_intel_i460_remove, .remove = __devexit_p(agp_intel_i460_remove),
}; };
static int __init agp_intel_i460_init(void) static int __init agp_intel_i460_init(void)
......
...@@ -1106,16 +1106,6 @@ int journal_dirty_metadata(handle_t *handle, struct buffer_head *bh) ...@@ -1106,16 +1106,6 @@ int journal_dirty_metadata(handle_t *handle, struct buffer_head *bh)
if (jh->b_transaction == handle->h_transaction && if (jh->b_transaction == handle->h_transaction &&
jh->b_jlist == BJ_Metadata) { jh->b_jlist == BJ_Metadata) {
JBUFFER_TRACE(jh, "fastpath"); JBUFFER_TRACE(jh, "fastpath");
console_verbose();
if (jh->b_transaction != journal->j_running_transaction) {
printk("jh->b_transaction=%p\n", jh->b_transaction);
printk("journal->j_running_transaction=%p\n",
journal->j_running_transaction);
printk("handle->h_transaction=%p\n",
handle->h_transaction);
printk("journal->j_committing_transaction=%p\n",
journal->j_committing_transaction);
}
J_ASSERT_JH(jh, jh->b_transaction == J_ASSERT_JH(jh, jh->b_transaction ==
journal->j_running_transaction); journal->j_running_transaction);
goto out_unlock_bh; goto out_unlock_bh;
...@@ -1328,9 +1318,6 @@ int journal_stop(handle_t *handle) ...@@ -1328,9 +1318,6 @@ int journal_stop(handle_t *handle)
journal_t *journal = transaction->t_journal; journal_t *journal = transaction->t_journal;
int old_handle_count, err; int old_handle_count, err;
if (!handle)
return 0;
J_ASSERT(transaction->t_updates > 0); J_ASSERT(transaction->t_updates > 0);
J_ASSERT(journal_current_handle() == handle); J_ASSERT(journal_current_handle() == handle);
......
...@@ -28,9 +28,6 @@ ...@@ -28,9 +28,6 @@
extern struct timezone sys_tz; extern struct timezone sys_tz;
#undef offsetof
#define offsetof(TYPE, MEMB) ((size_t) &((TYPE *)0)->MEMB)
/* Typedefs for integer types by size and signedness */ /* Typedefs for integer types by size and signedness */
typedef __u8 hfs_u8; typedef __u8 hfs_u8;
typedef __u16 hfs_u16; typedef __u16 hfs_u16;
......
...@@ -2482,11 +2482,11 @@ static void *s_start(struct seq_file *m, loff_t *pos) ...@@ -2482,11 +2482,11 @@ static void *s_start(struct seq_file *m, loff_t *pos)
seq_puts(m, "slabinfo - version: 2.0\n"); seq_puts(m, "slabinfo - version: 2.0\n");
#endif #endif
seq_puts(m, "# name <active_objs> <num_objs> <objsize> <objperslab> <pagesperslab>"); seq_puts(m, "# name <active_objs> <num_objs> <objsize> <objperslab> <pagesperslab>");
seq_puts(m, " : tunables <batchcount> <limit <sharedfactor>"); seq_puts(m, " : tunables <batchcount> <limit> <sharedfactor>");
seq_puts(m, " : slabdata <active_slabs> <num_slabs> <sharedavail>"); seq_puts(m, " : slabdata <active_slabs> <num_slabs> <sharedavail>");
#if STATS #if STATS
seq_puts(m, " : globalstat <listallocs> <maxobjs> <grown> <reaped> <error> <maxfreeable> <freelimit>"); seq_puts(m, " : globalstat <listallocs> <maxobjs> <grown> <reaped> <error> <maxfreeable> <freelimit>");
seq_puts(m, " : cpustat <allochit <allocmiss <freehit <freemiss>"); seq_puts(m, " : cpustat <allochit> <allocmiss> <freehit <freemiss>");
#endif #endif
seq_putc(m, '\n'); seq_putc(m, '\n');
} }
......
...@@ -293,7 +293,7 @@ static int __devinit snd_cmi8330_pnp(int dev, struct snd_cmi8330 *acard, ...@@ -293,7 +293,7 @@ static int __devinit snd_cmi8330_pnp(int dev, struct snd_cmi8330 *acard,
const struct pnp_card_device_id *id) const struct pnp_card_device_id *id)
{ {
struct pnp_dev *pdev; struct pnp_dev *pdev;
struct pnp_resource_table * cfg = kmalloc(GFP_ATOMIC, sizeof(struct pnp_resource_table)); struct pnp_resource_table *cfg = kmalloc(sizeof(*cfg), GFP_ATOMIC);
int err; int err;
acard->cap = pnp_request_card_device(card, id->devs[0].id, NULL); acard->cap = pnp_request_card_device(card, id->devs[0].id, NULL);
......
...@@ -1966,7 +1966,7 @@ static int __devinit snd_audiodrive_pnp(int dev, struct snd_audiodrive *acard, ...@@ -1966,7 +1966,7 @@ static int __devinit snd_audiodrive_pnp(int dev, struct snd_audiodrive *acard,
const struct pnp_card_device_id *id) const struct pnp_card_device_id *id)
{ {
struct pnp_dev *pdev; struct pnp_dev *pdev;
struct pnp_resource_table * cfg = kmalloc(GFP_ATOMIC, sizeof(struct pnp_resource_table)); struct pnp_resource_table *cfg = kmalloc(sizeof(*cfg), GFP_ATOMIC);
int err; int err;
if (!cfg) if (!cfg)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment